Parametri della filettatura modificati

Question

Quando si avviano più thread, il parametro id viene analizzato a volte errato. Qui è la mia startup:

for (int i = 0; i < _threadCount; i++) 
{ 
    Thread thread = new Thread(() => WorkerThread(i)); 
    thread.Start(); 
    _threads.Add(thread); 
} 

E la mia funzione di discussione:

private void WorkerThread(int id) 
{ 
    Console.WriteLine("[{0}] Thread started {1}", DateTime.Now.ToLongTimeString(), id); 
} 

L'uscita di questo codice è:

[19:10:54] Thread start 3 
[19:10:54] Thread start 9 
[19:10:54] Thread start 4 
[19:10:54] Thread start 12 
[19:10:54] Thread start 11 
[19:10:54] Thread start 3 
[19:10:54] Thread start 12 
[19:10:54] Thread start 6 
[19:10:54] Thread start 9 
[19:10:54] Thread start 6 
[19:10:54] Thread start 13 
[19:10:54] Thread start 2 
[19:10:54] Thread start 15 
[19:10:54] Thread start 9 
[19:10:54] Thread start 15 

Dove nella mia mente, questo codice dovrebbe creare ogni thread con un unico id invece di duplicati come visto sopra.

Compiler informazioni: obiettivo

Piattaforma: x64

target Framework: .NET Framework 4.5

Answer 1

Si dovrebbe fare attenzione su come modificare accidentalmente catturati variabili come i dopo l'inizio del filo, in quanto il i è condiviso. La variabile i si riferisce alla stessa posizione di memoria per tutta la durata del ciclo. La soluzioneè utilizzare un temporaneo variabile simili:

for (int i = 0; i < _threadCount; i++) 
{ 
     var i1 = i; 
     Thread thread = new Thread(() => WorkerThread(i1)); 
     thread.Start(); 
     _threads.Add(thread); 
} 

più informazioni Chiusure qui: The Beauty of Closures from (Jon Skeet) e Lambda expressions and captured variables from (Joseph Albahari).

Answer 2

Il problema è che la variabile i si riferisce alla stessa posizione di memoria attraverso la vita del ciclo. Pertanto, ogni thread chiama una variabile il cui valore può cambiare mentre è in esecuzione. La soluzione consiste nell'utilizzare una variabile temporanea int temp = i. come ha detto @ S.Akbari.