C# 2.0 Threading Frage (anonyme Methoden)

Question

Ich habe eine einfache Anwendung mit dem folgenden Code:

FileInfo[] files = (new DirectoryInfo(initialDirectory)).GetFiles(); 
    List<Thread> threads = new List<Thread>(files.Length); 

    foreach (FileInfo f in files) 
    { 
     Thread t = new Thread(delegate() 
     { 
      Console.WriteLine(f.FullName); 
     }); 
     threads.Add(t); 
    } 

    foreach (Thread t in threads) 
     t.Start(); 

Können sagen, in 'I = Initial' Verzeichnis Ich habe 3 Dateien. Diese Anwendung sollte dann 3 Threads erstellen, wobei jeder Thread einen der Dateinamen ausdruckt. Stattdessen wird jeder Thread den Namen der letzten Datei im Array 'files' ausgeben.

Warum ist das? Warum wird die aktuelle Datei 'f' Variable nicht richtig in der anonymen Methode eingerichtet?

Answer 1

Die anonyme Methode behält eine Referenz auf die Variable im umschließenden Block - nicht den tatsächlichen Wert der Variablen.

Zu dem Zeitpunkt, zu dem die Methoden tatsächlich ausgeführt werden (wenn Sie die Threads starten) wurde f zugewiesen, um auf den letzten Wert in der Auflistung zu zeigen, so dass alle 3 Threads den letzten Wert drucken.

Answer 2

Es ist, weil f.FullName ist eine Referenz auf eine Variable, und kein Wert (wie Sie es versucht haben, zu verwenden). Zu dem Zeitpunkt, zu dem Sie tatsächlich die Threads starten, wurde f.FullName bis zum Ende des Arrays inkrementiert.

Wie auch immer, warum durchlaufen Sie die Dinge hier zweimal?

foreach (FileInfo f in files) 
{ 
    Thread t = new Thread(delegate() 
    { 
     Console.WriteLine(f.FullName); 
    }); 
    threads.Add(t); 
    t.Start(); 
} 

Dies ist jedoch immer noch falsch, und vielleicht noch schlimmer, da Sie jetzt eine Race-Bedingung zu sehen haben, welcher Thread geht schneller: die Konsole Artikel zu schreiben oder auf die nächste Fileinfo laufen.

Answer 3

Hier sind ein paar nette Artikel über anonyme Methoden in C# und dem Code, der vom Compiler generiert wird:

http://blogs.msdn.com/oldnewthing/archive/2006/08/02/686456.aspx
http://blogs.msdn.com/oldnewthing/archive/2006/08/03/687529.aspx
http://blogs.msdn.com/oldnewthing/archive/2006/08/04/688527.aspx

Ich denke, wenn du getan hast:

 
    foreach (FileInfo f in files) 
    { 
     FileInfo f2 = f; //variable declared inside the loop 
     Thread t = new Thread(delegate() 
     { 
      Console.WriteLine(f2.FullName); 
     }); 
     threads.Add(t); 
    } 

es würde funktionieren, wie Sie es wollten.

Answer 4

Dies liegt daran, dass der zugrunde liegende Code für Iterator (foreach) bereits durch alle Werte in der Liste "iteriert" hat, bevor die Threads beginnen ... Wenn sie gestartet werden, ist der vom Iterator "angedeutete" Wert der letzte in der Liste ...

den Faden innerhalb der Iteration stattdessen starten ....

foreach (FileInfo f in files) 
{ 
    string filName = f.FullName; 
    Thread t = new Thread(delegate() 
       { Console.WriteLine(filName); }); 
    t.Start(); 
} 

ich glaube nicht, ein Rennen hier möglich ist, da von allen Threads gibt es keinen gemeinsamen Speicher zugegriffen werden.

Answer 5

Folgendes würde auch funktionieren.

Thread t = new Thread(delegate() 
    { 
     string name = f.Name; 
     Console.WriteLine(name); 
    });