C＃の奇妙なスレッディング

Question

C＃スレッディングで奇妙な問題が発生しました。

これはスレッドを使用してagentListの各エージェントでPrint（）関数を「アクティブにする」サンプルプログラムです。

class Program { 

    static void Main(string[] args) { 

     List<Agent> agentList = new List<Agent>(); 

     agentList.Add(new Agent("lion")); 
     agentList.Add(new Agent("cat")); 
     agentList.Add(new Agent("dog")); 
     agentList.Add(new Agent("bird")); 

     foreach (var agent in agentList) { 
      new Thread(() => agent.Print()).Start(); 
     } 

     Console.ReadLine(); 
    } 
} 

class Agent { 
    public string Name { get; set; } 

    public Agent(string name) { 
     this.Name = name; 
    } 

    public void Print() { 
     Console.WriteLine("Agent {0} is called", this.Name); 
    } 
} 

そして、ここでは、私は上記のプログラムを実行するときの結果である：

Agent cat is called 
Agent dog is called 
Agent bird is called 
Agent bird is called 

しかし、私は期待を何かが最も驚くべきものがあることである

Agent lion is called 
Agent cat is called 
Agent dog is called 
Agent bird is called 

のようにすべての4つの物質が含まれています私がforeachの外にスレッドを呼び出すと、動作します！

class Program { 
    static void Main(string[] args) { 
     List<Agent> agentList = new List<Agent>(); 

     agentList.Add(new Agent("leecom")); 
     agentList.Add(new Agent("huanlv")); 
     agentList.Add(new Agent("peter")); 
     agentList.Add(new Agent("steve")); 

     new Thread(() => agentList[0].Print()).Start(); 
     new Thread(() => agentList[1].Print()).Start(); 
     new Thread(() => agentList[2].Print()).Start(); 
     new Thread(() => agentList[3].Print()).Start(); 


     Console.ReadLine(); 
    } 
} 

上記のコードの結果は、まさに私が期待したものです。だからここで何が問題なの？

Answer 1

あなたが持っているものはクロージャです。 foreachループ内の変数をクローズしています。何が起こっているのは、スレッドが開始する前に変数が上書きされているため、同じ値を持つ2つの繰り返しがあるということです。

​​

Answer 2

これはスレッド内で同じrefrenceを持つ可能性があるvarエージェントのためです。 あなたのイヤラーのアプローチはお勧めできません。

Answer 3

は、このようなあなたのforeachループの変更

：

簡単な修正は、それを使用する前に、foreachループ内の値をキャプチャすることです（

foreach (var agent in agentList) 
{ 
    var agent1 = agent; 
    new Thread(() => agent1.Print()).Start();   
} 

ローカル変数に値をコピーしています少し愚かに見えます）は、実行時に変更される可能性のある変数を使用するスレッドを回避します。

Answer 4

クロージャでagentをキャプチャしていますが、これはマルチスレッドで問題になる可能性があります。最初のローカル変数に割り当てます。

foreach (var agent in agentList) { 
     var temp = agent; 
     new Thread(() => temp.Print()).Start(); 
    } 

Answer 5

（ManualResetEventのような）スレッド同期のいくつかの種類を使用しなければ、スレッドが処理される順序を保証することはできません。複数のステップを順番に実行したい場合は、ワークバンドルをバンドルして、すべてを単一のバックグラウンドスレッドで実行することをお勧めします。

List<Agent> agentList = new List<Agent>(); 

agentList.Add(new Agent("leecom")); 
agentList.Add(new Agent("huanlv")); 
agentList.Add(new Agent("peter")); 
agentList.Add(new Agent("steve")); 

BackgroundWorker worker = new BackgroundWorker(); 
worker.DoWork += (sender, e) => 
{ 
    foreach (var item in agentList) 
    { 
     item.Print(); 
    } 
}; 

worker.RunWorkerAsync();