Declarar una variable dentro o fuera de un bucle foreach: ¿cuál es más rápido/mejor?

Question

¿Cuál de estos es el más rápido/mejor?

Ésta:

List<User> list = new List<User>(); 
User u; 

foreach (string s in l) 
{ 
    u = new User(); 
    u.Name = s; 
    list.Add(u); 
} 

O esta otra:

List<User> list = new List<User>(); 

foreach (string s in l) 
{ 
    User u = new User(); 
    u.Name = s; 
    list.Add(u); 
} 

Mis habilidades novato-desarrollo me dice que el primero es mejor, pero un amigo mío me dice im mal, pero podría No me da una buena razón por la cual el segundo es mejor.

¿Hay alguna diferencia en el rendimiento en absoluto?

Answer 1

En cuanto a rendimiento, ambos ejemplos se compilan en la misma IL, por lo que no hay diferencia.

El segundo es mejor, porque expresa más claramente su intención si u solo se usa dentro del bucle.

Answer 2

El segundo es mejor. Quiere decir tener un nuevo usuario en cada iteración.

Answer 3

No debe haber diferencias perceptibles en el rendimiento.

Answer 4

Técnicamente, el primer ejemplo ahorrará algunos nanosegundos porque no será necesario mover el marco de la pila para asignar una nueva variable, pero esta es una cantidad tan pequeña de tiempo de CPU que no lo notará, eso es si el compilador no optimiza ninguna diferencia de ninguna manera.

Answer 5

Una declaración no hace que se ejecute ningún código, por lo que no es un problema de rendimiento.

El segundo es lo que quieres decir, y es menos probable que cometas un error estúpido si lo haces de la segunda manera, así que úsalo. Intente siempre declarar las variables en el alcance más pequeño que sea necesario.

Y, además, la mejor manera es utilizar LINQ:

List<User> users = l.Select(name => new User{ Name = name }).ToList(); 

Answer 6

En este escenario, la segunda versión es mejor.

En general, si solo necesita acceder al valor dentro del cuerpo de la iteración, entonces elija la segunda versión. Por otro lado, si hay algún estado final, la variable se mantendrá más allá del cuerpo del ciclo, entonces declara y luego usa la primera versión.

Answer 7

En cualquier caso, la mejor manera sería utilizar un constructor que toma un nombre ... o, de otro modo, explotar rizado corsé notación:

foreach (string s in l) 
{ 
    list.Add(new User(s)); 
} 

o

foreach (string s in l) 
{ 
    list.Add(new User() { Name = s }); 
} 

o aún mejor, LINQ:

var list = l.Select(s => new User { Name = s}); 

Ahora, mientras el primer ejemplo podría, en algunos casos, ser unperceptibly más rápido, el segundo en e es mejor porque es más legible y el compilador puede descartar la variable (y omitirla por completo) ya que no se usa fuera del alcance de foreach.

Answer 8

Cuando tenga una pregunta sobre el rendimiento, lo único que debe hacer es medir, ejecutar un ciclo alrededor de su prueba y medir el tiempo.

Para responder a su pregunta, sin medir :-) o mirar las ilasm generadas, cualquier diferencia no se notaría en un número significativo de iteraciones y la operación más cara en su código probablemente sea la asignación del usuario por unos pocos órdenes de magnitud, así que concéntrate en la claridad del código (como deberías en general) y ve con 2.

Oh, es tarde y creo que solo estoy tratando de decir que no te preocupes por este tipo de cosa o quedar atrapado en detalles como este.

K

Answer 9

Otra referencia que se parece más arriba:

http://social.msdn.microsoft.com/forums/en-US/csharplanguage/thread/d43aaba5-a58b-4610-bea4-5bc5d6741f98