Lancer un thread à partir d'une boucle et passer l'ID de boucle

Question

Je viens de commencer à jouer avec du filetage aujourd'hui et j'ai rencontré quelque chose que je ne comprends pas.

public void Main() 
{ 
    int maxValue = 5; 
    for (int ID = 0; ID < maxValue; ID++) 
    { 
     temp(ID); 
    } 
} 

public void temp(int i) 
{ 
    MessageBox.Show(i.ToString()); 
} 

Comme base qu'il obtient qui fonctionne très bien, mais lorsque je tente de créer un nouveau fil pour chacun, il ne passe que valeurmax. S'il vous plaît ne tenez pas compte de la gravité de cette situation, je l'ai seulement écrit comme un exemple simpliste.

public void Main() 
{ 
    int maxValue = 5; 
    for (int ID = 0; ID < maxValue; ID++) 
    { 
     threads.Add(new Thread(() => temp(myString, rowID))); 
     threads[rowID].Start(); 
    } 
} 

public void temp(string myString, int i) 
{ 
    string _myString = myString; 

    MessageBox.Show(i.ToString()); 
} 

Compte tenu de cela, j'ai deux questions: 1) Pourquoi la méthode ne marche pas un se fait appel à un nouveau thread qui passe l'ID? 2) Comment devrait-il être correctement codé?

Answer 1

Le problème est que vous n'avez qu'une seule variable ID et qu'elle est en cours de capture. La variable est seulement read lorsque le code dans le nouveau thread est réellement exécuté, ce qui sera souvent après que votre thread principal a terminé sa boucle, laissant ID à maxValue. Prenez une copie de chaque itération de boucle, de sorte que vous capturez une variable différente à chaque fois:

for (int ID = 0; ID < maxValue; ID++) 
{ 
    int copy = ID; 
    threads.Add(new Thread(() => temp(myString, copy))); 
    threads[rowID].Start(); 
} 

Ceci est une erreur commune avec des fermetures. Lisez my article comparing C# and Java closures pour plus d'informations. La même chose se produit avec foreach, en passant - et qui est encore plus déroutant, car il lit comme vous avez une nouvelle à chaque fois que la variable:

foreach (string url in urls) 
{ 
    // Aargh, bug! Don't do this! 
    new Thread(() => Fetch(url)).Start(); 
} 

Encore une fois, vous finissez seulement avec une variable. Vous avez besoin de chaque délégué pour capturer une variable distincte, de sorte que vous utilisez à nouveau une copie:

foreach (string url in urls) 
{ 
    string urlCopy = url; 
    new Thread(() => Fetch(urlCopy)).Start(); 
}