C＃でのスレッディング：ループインデックスをスレッドセーフにするには？次の例では

Question

は、forループのインデックスiが正しくループをマルチスレッド化された方法DoWork_Threaded()

に（さえSystem.Environment.ProcessorCount-1より大きい値、倍数）iの奇妙な値に至る各スレッドによって独立して修飾されると思われますCで行われた＃？

// Prepare all threads 
Thread[] threads = new Thread[System.Environment.ProcessorCount]; 

// Start all threads 
for (int i = 0; i < System.Environment.ProcessorCount; i++) 
{ 
    threads[i] = new Thread(() => DoWork_Threaded(i)); 
    threads[i].Start(); 
} 

// Wait for completion of all threads 
for (int i = 0; i < System.Environment.ProcessorCount; i++) 
{ 
    threads[i].Join(); 
} 

Answer 1

まず、スレッドスケジューリングサービスをローリングするのではなく、なぜタスク並列ライブラリを使用しないのですか？すでにスレッドをスケジュールするプロセッサの数を決定するロジックを持っています。

実際の質問にお答えするには、ループ変数でを締めてください。それは間違っている理由について私の記事を参照してください。

要するに

https://ericlippert.com/2009/11/12/closing-over-the-loop-variable-considered-harmful-part-one/

：iは変数と変数変更です。ラムダが実行されると、の現在のの値がiで実行されます。デリゲートが作成されたときの値ではありません。

Answer 2

匿名のメソッドとスレッドでローカル変数を使用するのは難しい場合があります。

for (int i = 0; i < System.Environment.ProcessorCount; i++) 
{ 
    int a = i; 
    threads[i] = new Thread(() => DoWork_Threaded(a)); 
    threads[i].Start(); 
} 

これは私のマシン上で合理的な出力になります：あなたはそれが変化しているとして匿名メソッドによって使用されている値をコピーする必要があります。