LINQ + Foreach vs Foreach + Si

Question

Necesito iterar sobre una Lista de objetos, haciendo algo solo para los objetos que tienen una propiedad booleana establecida en verdadero. Me estoy debatiendo entre este código

foreach (RouteParameter parameter in parameters.Where(p => p.Condition)) 
{ //do something } 

y este código

foreach (RouteParameter parameter in parameters) 
{ 
    if !parameter.Condition 
    continue; 
    //do something 
} 

El primer código es, obviamente, más limpio, pero sospecho que va a un bucle sobre la lista dos veces - una vez para la consulta y una vez para el foreach. Esta no será una gran lista, así que no estoy demasiado preocupado por el rendimiento, pero la idea de repetir dos veces solo errores me.

Pregunta: ¿Existe una manera clara/bonita de escribir esto sin bucles dos veces?

Answer 1

Jon Skeet a veces hace una demostración LINQ de acción real para explicar cómo funciona esto. Imagina que tienes tres personas en el escenario. A la izquierda tenemos a un tipo que tiene una baraja barajada. En el medio tenemos a un tipo que solo pasa las tarjetas rojas, y a la derecha, tenemos un tipo que quiere cartas.

El tipo de la derecha golpea al tipo en el medio. El tipo en el medio golpea al tipo de la izquierda. El hombre de la izquierda le da una carta al chico del medio. Si es negro, el tipo que está en el medio lo tira al suelo y lo golpea de nuevo hasta que recibe una tarjeta roja, que luego le entrega al tipo de la derecha. Entonces el tipo de la derecha golpea al tipo en el medio otra vez.

Esto continúa hasta que el hombre de la izquierda se queda sin cartas.

La plataforma no se ha recorrido de principio a fin más de una vez. Sin embargo, tanto el hombre de la izquierda como el del medio manejaban 52 cartas, y el de la derecha manejaba 26 cartas. Hubo un total de 52 + 52 + 26 operaciones en las tarjetas, pero el mazo solo se conectó una vez.

Su versión "LINQ" y la versión "continuar" son la misma cosa; si tuviera

foreach(var card in deck) 
{ 
    if (card.IsBlack) continue; 
    ... use card ... 

entonces hay 52 operaciones que recuperan cada carta de la baraja, 52 operaciones que probar para ver si cada tarjeta es de color negro, y 26 operaciones que actúan sobre la tarjeta roja. Lo mismo exactamente.

Answer 2

La mayoría de los operadores de Linq como Where están implementados para admitir la ejecución diferida y diferida . En su ejemplo, la lista se repetirá solo una vez porque el enumerador que se encuentra detrás del IEnumerable devuelto por Where enumerará la lista hasta que encuentre un elemento que coincida con el predicado, lo cederá y solo continuará cuando se le solicite el siguiente elemento.

Desde el punto de vista del código, preferiría la variante utilizando where, aunque podría argumentarse que podría declarar un local para parameters.Where(p => p.Condition).

La serie Edulinq de Jon Skeet es muy recomendada, la lectura de algunas partes de esto debería ayudarle con su comprensión de los operadores LINQ.

Answer 3

En realidad, no es "bucle dos veces". La cláusula .Where utiliza ejecución diferida. En otras palabras, prácticamente no se realiza ningún trabajo cuando llama al .Where, pero cuando itera sobre el resultado, iterará sobre la lista original y solo pasará elementos que coincidan con su condición. Si se piensa en ello en términos de cómo el código es ejecutado, que está haciendo efectivamente esto:

Func<Parameter, bool> matchesCondition = p => p.Condition; 
foreach(var parameter in parameters) 
{ 
    if(matchesCondition(parameter)) 
    { 
     ... 
    } 
} 

Como una cuestión de estilo, yo personalmente prefiero algo más como:

var matchingParameters = parameters.Where(p => p.Condition); 
foreach(var parameter in matchingParameters) 
{ 
} 

Answer 4

Yo prefiero esta:

theList.Where(itm => itm.Condition).ToList().ForEach(itmFE => { itmFe.DoSomething(); });