スレッドパラメータが変更されています

Question

複数のスレッドを起動すると、解析中のパラメータidが間違っていることがあります。ここに私のスタートアップは、次のとおりです。

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

そして、私のスレッド関数：

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

このコードの出力は次のとおりです。

私の心には、このコードが持つすべてのスレッドを作成する必要があります

[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 

上記のように重複ではなくユニークなidです。

コンパイラ情報：

プラットフォームターゲット：x64の

ターゲットフレームワーク：.NET Frameworkの4.5

Answer 1

あなたは、誤ってスレッドを開始した後iのように撮影した変数の変更について注意する必要がありますので、 iは共有です。 i変数は、ループの存続期間全体にわたって同じメモリ位置を参照します。 ソリューションは、このような一時変数を使用することです：

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

読むよりおよそクロージャここ：The Beauty of Closures from (Jon Skeet)とLambda expressions and captured variables from (Joseph Albahari).

Answer 2

問題はi変数が同じメモリ位置を指していることですループの寿命を通して。したがって、各スレッドは、実行中に値が変化する可能性のある変数を呼び出します。解決策は、ユーザに一時変数int temp = iを渡すことです。 @ S.Akbariは言った。