マルチスレッド環境でクラスインスタンスの値が壊れているのはなぜですか？

Question

はここ

class Program 
{ 
    static void Main(string[] args) 
    { 
     for (int i = 0; i < 100; i++) 
     { 
      Thread thread = new Thread(new ThreadStart(()=> SimpleClass.Instance.weird.SetHello(i))); 
      thread.Start(); 
     } 

     Console.Read(); 
    } 
} 

interface IClass 
{ 
    WeirdClass weird{ get; set; } 
} 

class SimpleClass : IClass { 
    public static SimpleClass Instance = new SimpleClass(); 

    public WeirdClass weird{ get; set; } 

    public SimpleClass() 
    { 
     weird= new WeirdClass(); 
    } 
} 

class WeirdClass 
{ 
    public int hello; 

    public void SetHello(int i) 
    { 
     this.hello = i; 
     Console.WriteLine(this.hello); 
    } 
} 

我々はWeirdClassの「ハロー」の値がマルチスレッドで正しく行われていません見ることができ、値は単に静的インスタンスのようですが、そうではありません、単純なコードです。

おそらく、魔法はSimpleClass.Instance.asyncで起こるかもしれません。ありがとう

Answer 1

私はいくつかの興味深い研究に私をもたらしたので、私はあなたの質問をupvoted。 （例えば、入れ

var ii = i; 
Thread thread = new Thread(
    new ThreadStart(()=> SimpleClass.Instance.async.SetHello(ii))); 
thread.Start(); 

私は異なる構成と異なるコンテキストでそれをテストしてみた：ループのために一時iiにi変数を割り当てるとスレッドの初期化にiiを使用し、このように

helloの割り当てとConsole.WriteLineの間のランダムThread.Sleep）。

私の解決方法はこれです：すべてのスレッドは独自のiを持っており、Consoleに一度だけ書き込まれます。 を作成すると、スレッドが順番に開始されないため、数字は順番に書き込まれません。

ジャストサイドノート：@JonSkeetが推奨されているように、識別子としてasync使用していない、それは予約語です。使用する代わりにAsyncWeird、InnerClass、HelloContainer、または何でもそれは、公式のC＃のトークンではありませんそれ以外の場合は、あなたのコードはあまり明らかになりました。

UPDATE - 解決

これ：

for(int i = 0; i < 10; i++) 

これに相当する：

int i; 
for(i = 0; i < 10; i++) 

両方の場合においてiをforループの外部で作成された、であること、およびループの異なるサイクル間で何らかの形で共有されます。あなたが提供するリンク@CodeCasterをチェックすると、あなたは、C＃の以前のバージョンとforeachループで同じ問題が表示されます。あなたが入力としてiでサイクルでスレッドを宣言すると

、あなたがスレッドに言っている：は「OK、あなたが起動したとき、i値を取得します！」。その瞬間、iは、適切な値を持っていますが、スレッドは、それが効果的に開始した唯一のiを要求し、その値が既にメインスレッドが既にループの新しいサイクルを開始しましたし、既に再割り当てしているので、その間に変更されました次の反復のためi。

ループ内にの中に一時変数を作成すると、そのサイクルに残り、外部には見えません。

これは、ゲーム全体のデモンストレーションです：ループの外iiを宣言しようとする...行動は、あなたがi変数を示したことが、再び間違ったものです。