Está usando linq en esta situación overkill

Question

Estoy pensando en un nuevo factor, pero no sé si el resultado final es simplemente exagerado. Actualmente tengo

IList<myobjecttypebase> m_items; 

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

¿Sería excesivo y/o menos eficiente de utilizar LINQ para el ParaCada

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

¿El elenco ser una sobrecarga significativa aquí?

Answer 1

El método ToList es un método de extensión se utiliza con LINQ, pero el método ParaCada es sólo un método en la clase Lista.

La principal sobrecarga aquí es la llamada al método ToList, que crea una nueva lista de la colección. El ForEach también tiene una ligera sobrecarga, ya que tiene que llamar a un delegado para cada elemento.

Si desea utilizar métodos de LINQ, el método agregado parece más apropiado:

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

o Suma:

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

o bien tiene una ligera sobrecarga por encima de su versión original, así que si quieres el más eficiente, solo adhiérase a un simple bucle.

Answer 2

La sobrecarga consistirá en iterar la colección dos veces.

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

ToList no realiza la iteración de manera perezosa como la mayoría de las declaraciones de LINQ, para ello se obligará a la iterate código a través de la colección dos veces.

En general, el formulario LINQ parece más agradable de leer, pero si te preocupa el rendimiento, es mejor que lo evites.

Answer 3

¿Por qué no utilizar el operador SUM directamente en el IList?

Se tiene varias sobrecargas mediante Func delegados:

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

Answer 4

No hacerlo de esta manera.
El ToList() aquí asignará y rellenará una nueva lista, en mi humilde opinión no es necesario. Si bien es un ejercicio trivial escribir un método de extensión ForEach para iterar sobre cualquier objeto que implemente IEnumberable, recomendaría que no lo haga (consulte este artículo de Eric Lippert foreach V's ForEach).

I'd go with the foreach.

P.S. que necesita para inicializar incrementTot (lo siento, no pude evitarlo)

Answer 5

Como se ha mencionado Oded uso SUMA

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

Answer 6

ForEach no es parte de LINQ - es parte de la API List<T> y ha sido desde entonces. NET 2.0.

La razón por la que no forma parte de LINQ es porque es un método natural que causa efectos secundarios ... y LINQ no fomenta los efectos secundarios.

Usando suma es la solución aquí, pero no es necesario para crear una lista en primer lugar:

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