C＃foreach with Action.BeginInvoke

Question

申し訳ありませんが、ここでは少し問題があります。 ここにループがあります。

lock (ClientLocker) 
{ 
    Trace.WriteLine("#WriteAll: " + sm.Header); 
    foreach (Client c in Clients) 
    { 
     if (c.LoggedIn) 
     { 
      Trace.WriteLine("#TryWriteTo[" + c.Id + "](" + sm.Header + ")"); 
      LazyAsync.Invoke(() => c.WriteMessage(sm)); 
     } 
    } 
} 

はここでそれぞれが Clientが socket含まLazyAsync

public static class LazyAsync 
{ 
    public static void Invoke(Action a) 
    { 
     a.BeginInvoke(a.EndInvoke, null); 
    } 
} 

あるので、私はそれはほとんどCloneすることはできません。 問題は、Invokeをc.WriteMessageにすると、実行が遅れるため、通常はリストの最初のカップルでは起動せず、実際には最後のアイテムだけが一斉に起動することがあります。

これは、Invokeが実際に呼び出される前に変更される参照であるCと関係があることは知っていますが、これを避ける方法はありますか？

一般的に、for(int i=0 etcループを実行してもこの問題は解決されないようです。

誰でも私がこれをどのように修正することができますか？

CloneClientを忘れないでください。

Answer 1

は、このようなあなたのcへのローカル変数をコピー：あなたはより多くの情報を取得したい場合は、「修正閉鎖へのアクセス」：

lock (ClientLocker) 
{ 
    Trace.WriteLine("#WriteAll: " + sm.Header); 
    foreach (Client c in Clients) 
    { 
     if (c.LoggedIn) 
     { 
      Client localC = c; 
      Trace.WriteLine("#TryWriteTo[" + c.Id + "](" + sm.Header + ")"); 
      LazyAsync.Invoke(() => localC.WriteMessage(sm)); 
     } 
    } 
} 

がためのWeb検索を行います。

Answer 2

あなたの疑惑は正しいです：変数cはラムダ式によって取り込まれますが、後で評価されることはありません。

ラムダ変数内でループ変数を使用すると、ループ変数がループ外にスコープされ、ループの各繰り返しではスコープされないため、このエラーが発生します。

あなたは、foreachループに新しいローカル変数を作成することによってこの問題を回避それにcを割り当て、その後、ラムダ式にその新しいローカル変数を渡すことができます。ここでは

lock (ClientLocker) 
{ 
    Trace.WriteLine("#WriteAll: " + sm.Header); 
    foreach (Client c in Clients) 
    { 
     if (c.LoggedIn) 
     { 
      Trace.WriteLine("#TryWriteTo[" + c.Id + "](" + sm.Header + ")"); 

      Client copyOfC = c; 
      LazyAsync.Invoke(() => copyOfC.WriteMessage(sm)); 
     } 
    } 
} 

は、関連するいくつかのStackOverflowのです投稿：

• Access to Modified Closure
• Is there a reason for C#'s reuse of the variable in a foreach?
• C# lambda, local variable value not taken when you think?

Answer 3

呼び出しが起こる前に、foreachループによって再割り当てさcを避けるために、ローカル変数にCを設定し、その上LazyAsync.Invokeを呼び出してみてください。 LazyAsync.Invokeはc.WriteMessageをするとき、それは今まで、それはなかった何をするときLazyAsync.Invoke（（）=> c.WriteMessage（SM））を評価した

foreach (Client c in Clients) 
{ 
    if (c.LoggedIn) 
    { 
     Trace.WriteLine("#TryWriteTo[" + c.Id + "](" + sm.Header + ")"); 

     Client client = c; 
     LazyAsync.Invoke(() => client.WriteMessage(sm)); 
    } 
}