Wiele SUM przy użyciu LINQ

Question

Mam pętlę jak poniżej, czy mogę zrobić to samo za pomocą wielu SUM?

foreach (var detail in ArticleLedgerEntries.Where(pd => pd.LedgerEntryType == LedgerEntryTypeTypes.Unload && 
                   pd.InventoryType == InventoryTypes.Finished)) 
{ 
    weight += detail.GrossWeight; 
    length += detail.Length; 
    items += detail.NrDistaff; 
} 

Answer 1

Technicznie rzecz biorąc, co masz jest prawdopodobnie najbardziej skuteczny sposób rób to, o co prosisz. Jednak można utworzyć metody rozszerzenie na IEnumerable <T> nazwie Każda które mogą uprościć:

public static class EnumerableExtensions 
{ 
    public static void Each<T>(this IEnumerable<T> col, Action<T> itemWorker) 
    { 
     foreach (var item in col) 
     { 
      itemWorker(item); 
     } 
    } 
} 

i nazwać tak:

// Declare variables in parent scope 
double weight; 
double length; 
int items; 

ArticleLedgerEntries 
    .Where(
     pd => 
      pd.LedgerEntryType == LedgerEntryTypeTypes.Unload && 
      pd.InventoryType == InventoryTypes.Finished 
    ) 
    .Each(
     pd => 
     { 
      // Close around variables defined in parent scope 
      weight += pd.GrossWeight; 
      lenght += pd.Length; 
      items += pd.NrDistaff; 
     } 
    ); 

UPDATE: tylko jedna uwaga dodatkowa. Powyższy przykład opiera się na zamknięciu. Zmienne waga, długość i elementy powinny być zadeklarowane w zakresie nadrzędnym, pozwalając im zachować się poza każdym wywołaniem akcji itemWorker. Zaktualizowałem przykład, aby to odzwierciedlić dla jasności.

Answer 2

Możesz zadzwonić pod numer Sum trzy razy, ale będzie wolniej, ponieważ spowoduje utworzenie trzech pętli.

Na przykład:

var list = ArticleLedgerEntries.Where(pd => pd.LedgerEntryType == LedgerEntryTypeTypes.Unload 
            && pd.InventoryType == InventoryTypes.Finished)) 

var totalWeight = list.Sum(pd => pd.GrossWeight); 
var totalLength = list.Sum(pd => pd.Length); 
var items = list.Sum(pd => pd.NrDistaff); 

powodu opóźnionej realizacji, będzie to również ponownej oceny połączenia Where za każdym razem, mimo że nie jest takim problemem w Twoim przypadku. Można tego uniknąć, wywołując ToArray, ale spowoduje to przydzielenie tablicy. (I nadal będzie działał trzy pętle)

Jednak, jeśli nie masz dużej liczby wpisów lub uruchamiasz ten kod w ciasnej pętli, nie musisz martwić się o wydajność.

---

EDIT: Jeśli naprawdę chcesz używać LINQ, można nadużywać Aggregate, tak:

int totalWeight, totalLength, items; 

list.Aggregate((a, b) => { 
    weight += detail.GrossWeight; 
    length += detail.Length; 
    items += detail.NrDistaff; 
    return a; 
}); 

To fenomenalnie brzydki kod, ale należy wykonać niemal jak również prosta pętla.

Można również podsumować akumulator, (patrz przykład poniżej), ale to przydzieli tymczasowy obiekt dla każdego elementu na liście, co jest głupim pomysłem. (Typy anonimowych są niezmienne)

var totals = list.Aggregate(
    new { Weight = 0, Length = 0, Items = 0}, 
    (t, pd) => new { 
     Weight = t.Weight + pd.GrossWeight, 
     Length = t.Length + pd.Length, 
     Items = t.Items + pd.NrDistaff 
    } 
); 

Answer 3

Byłbyś w stanie zrobić tego przegubu stylu, przy użyciu odpowiedzi w tym temacie: Is it possible to Pivot data using LINQ?

Answer 4

Ok. Zdaję sobie sprawę, że nie ma prostego sposobu na to, używając LINQ. Wezmę pętlę foreach, ponieważ zrozumiałem, że nie jest tak źle. Dzięki wam wszystkim

Answer 5

Można też grupa przez prawdziwe - 1 (co faktycznie jest w tym jednego z przedmiotów, a następnie je policzone lub summered):

var results = from x in ArticleLedgerEntries 
         group x by 1 
         into aggregatedTable 
         select new 
            { 
             SumOfWeight = aggregatedTable.Sum(y => y.weight), 
             SumOfLength = aggregatedTable.Sum(y => y.Length), 
             SumOfNrDistaff = aggregatedTable.Sum(y => y.NrDistaff) 
            }; 

miarę Czas trwania jest prawie tak dobre, jak pętla (ze stałym dodatkiem).