Używa linq w tej sytuacji overkill

Question

Mam na myśli re-factor, ale nie wiem, czy efekt końcowy jest po prostu przesadzony. Obecnie mam

IList<myobjecttypebase> m_items; 

    public int GetIncrementTotal() 
    { 
    int incrementTot; 
    foreach(myobjecttypebase x in m_items) 
    { incrementTot += x.GetIncrement(); } 
    } 

byłoby przesadą i/lub mniej efektywne w użyciu LINQ do ForEach

m_items.ToList().ForEach(x => incrementTot += x.GetIncrement()); 

Czy obsada być znaczny narzut tutaj?

Answer 1

Sposób ToList jest metoda rozszerzenie stosowania z LINQ, ale metoda ForEach jest tylko metoda w klasie Lista.

Najważniejszym narzutem tutaj jest wywołanie metody ToList, która tworzy nową listę z kolekcji. ForEach ma również niewielki narzut, ponieważ musi wywołać delegata dla każdej pozycji.

Jeśli chcesz użyć metody LINQ, sposób sumaryczny wydaje się bardziej odpowiedni:

public int GetIncrementTotal() { 
    return m_items.Aggregate(0, (t, i) => t + i.GetIncrement()); 
} 

lub suma:

public int GetIncrementTotal() { 
    return m_items.Sum(i => i.GetIncrement()); 
} 

Albo ma niewielki narzut nad swoim pierwotnym brzmieniu, więc jeśli chcesz najbardziej wydajny, po prostu trzymaj się prostej pętli.

Answer 2

Narzutu będzie w iteracji kolekcji dwukrotnie.

m_items.ToList() // First iteration to add to the list all the items 
    .ForEach(x => incrementTot += x.GetIncrement()); /* Second iteration to 
                 perform logic */ 

ToList nie wykonuje iteracje w sposób jak najbardziej leniwym sprawozdania LINQ nich będzie zmusić powtórzyć, kod nad kolekcji dwukrotnie.

Generalnie formularz LINQ wydaje się być czytelniejszy, ale jeśli martwisz się wydajnością, lepiej go unikaj.

Answer 3

Dlaczego nie używać operatora SUM bezpośrednio na IList?

To ma kilka przeciążeń za pomocą Func delegatów:

m_items.Sum(x => x.GetIncrement()); 

Answer 4

nie rób tego w ten sposób.
Tutaj ToList() przydzieli i zapełni nową listę, IMHO nie jest konieczne. Mimo że jest to trywialne ćwiczenie polegające na napisaniu metody rozszerzenia ForEach do iterowania nad dowolnym obiektem, który implementuje IEnumberable, odradzam (patrz artykuł Eric Lipperta foreach V's ForEach).

Poszedłbym z foreach.

P.S. trzeba zainicjować incrementTot (przepraszam, nie mogłem się powstrzymać)

Answer 5

Jak wspomniano Oded use SUM

incrementTot = m_items.ToList().Sum(x => x.GetIncrement()); 

Answer 6

ForEach nie jest częścią LINQ - to część List<T> API i jest od tego czasu. NET 2.0.

Powodem, dla którego nie jest częścią LINQ jest to, że jest to metoda naturalnie uboczna ... a LINQ nie zachęca do działań niepożądanych.

Korzystanie Sum jest dobrym rozwiązaniem tutaj, ale nie trzeba utworzyć listę pierwsze:

return m_items.Sum(x => x.GetIncrement());