Eine Variable innerhalb oder außerhalb einer foreach-Schleife deklarieren: Was ist schneller/besser?

Question

Welcher davon ist der schnellere/bessere?

Dieses:

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

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

Oder diese:

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

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

Mein Neuling-Entwicklung von Fähigkeiten sagt mir, die erste ist besser, aber ein Freund von mir erzählt im falschen mich, konnte aber gib mir keinen guten Grund, warum der zweite besser ist.

Gibt es überhaupt einen Unterschied in der Leistung?

Answer 1

Leistungsmäßig werden beide Beispiele zu derselben IL kompiliert, also gibt es keinen Unterschied.

Die zweite ist besser, weil es deutlicher Ihre Absicht ausdrückt, wenn u nur innerhalb der Schleife verwendet wird.

Answer 2

Die zweite ist besser. Sie möchten in jeder Iteration einen neuen Benutzer haben.

Answer 3

Es sollte keinen wahrnehmbaren Unterschied in der Leistung geben.

Answer 4

Technisch betrachtet, spart das erste Beispiel ein paar Nanosekunden, weil der Stapelrahmen nicht verschoben werden muss, um eine neue Variable zuzuweisen, aber das ist eine so kleine Menge an CPU-Zeit, dass Sie es nicht bemerken Der Compiler optimiert keine Unterschiede.

Answer 5

Eine Deklaration führt nicht dazu, dass Code ausgeführt wird. Es handelt sich also nicht um ein Leistungsproblem.

Die zweite ist, was Sie meinen, und Sie sind weniger wahrscheinlich, einen dummen Fehler zu machen, wenn Sie es auf die zweite Art tun, also verwenden Sie das. Versuchen Sie immer Variablen im kleinstmöglichen Umfang zu deklarieren.

Und außerdem ist der bessere Weg ist Linq zu verwenden:

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

Answer 6

In diesem Szenario wird die zweite Version besser ist.

Wenn Sie nur auf den Wert im Rumpf der Iteration zugreifen müssen, wählen Sie im Allgemeinen die zweite Version. Auf der anderen Seite, wenn es einen Endzustand gibt, wird die Variable über den Körper der Schleife gehalten, dann deklariere dann die erste Version.

Answer 7

Auf jeden Fall der beste Weg, einen Konstruktor zu verwenden wäre, die einen Namen hat ... oder sonst ausnutzen geschweiften Klammer-Notation:

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

oder

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

oder noch besser, LINQ:

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

Jetzt, während Ihr erstes Beispiel könnte, in einigen Fällen unperceptibly schneller sein, der zweite auf e ist besser, weil es lesbarer ist, und der Compiler kann die Variable verwerfen (und sie ganz weglassen), da sie nicht außerhalb des Bereichs foreach verwendet wird.

Answer 8

Wenn Sie eine Frage zur Leistung haben, müssen Sie nur messen - führen Sie eine Schleife durch Ihren Test und geben Sie die Zeit ein.

Um Ihre Frage zu beantworten - ohne zu messen :-) oder Blick auf die generierte ilasm - keinen Unterschied in einer sinnvollen Anzahl von Iterationen nicht bemerkbar und die teuerste Operation in Ihrem Code ist wahrscheinlich die Benutzerzuweisung um ein paar Größenordnungen, so konzentrieren Sie sich auf Code Klarheit (wie Sie sollten im Allgemeinen) und gehen mit 2.

Oh, es ist spät und ich denke, ich versuche nur zu sagen, mach dir keine Sorgen über diese Art von Ding oder sich in Details wie diese verfangen.

K

Answer 9

Eine weitere Referenz, die wie oben aussieht:

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