Interessanter C# Bug?

Exor

Hallo,

ich bin grad auf folgenden Bug? gestoßen

Code

List<Func<int>> doublers = new List<Func<int>>();


            for(int i=0;i<5;i++)
                doublers.Add( () => i * 2 );


            foreach (Func<int> f in doublers)
                Console.WriteLine( f() );


            Console.ReadKey();

Alles anzeigen

gibt aus:

Code

Nun ich kann mir denken das, das daran liegt das lambda expressions zur compile time erzeugt werden..

Allerdings bin ich absolut unfähig die Spezifikation dazu zu finden.

help?

Plantschkuh!

Hmm, Bug... Ich würds eher die Ineleganz, die man bei Sprachen mit destruktivem Update nunmal hat, nennen. Die erzeugten Funktionen sind wohl alle unterschiedliche Objekte, haben aber alle eine Referenz aufs selbe Objekt i. Zu dem Zeitpunkt, zu dem sie aufgerufen werden, hat dieses eine Objekt nunmal den Wert 5.

Ohne C# zu kennen oder mich irgendwie informiert zu haben: Was passiert, wenn du sowas in die Richtung probierst? Die Idee wär, über die zusätzliche Indirektion den Wert von i zu kopieren, nicht die Adresse. Und zwar zu dem Zeitpunkt, zu dem die Funktion in die Liste eingefügt wird, d.h. mit unterschiedlichen i-Werten.

Code

doublers.Add( ((int j) => (() => j * 2))(i) );

Ringding

Plantschkuh hat recht, closures funktionieren in vielen Sprachen so. Definitiv sagen kann ich es von Python, dort wird das Ergebnis genau gleich sein.

fabs

Zitat von Plantschkuh!
Code
doublers.Add( ((int j) => (() => j * 2))(i) );

Sollte funktionieren, eine verständlichere Version wäre aber:

Code

for(int i=0;i<5;i++)
{
 int j=i;
        doublers.Add( () => j * 2 );
}

damike

Such mal nach "Deferred Execution" und "Lazy Evaluation" - gibt einige MS Artikel dazu

LG

Jetzt mitmachen!