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)); 
     } 
    } 
} 

Вот LazyAsync

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

Каждый Client содержит socket, так что я не могу с трудом Clone это. Проблема заключается в том, что когда я делаю Invoke до c.WriteMessage, так как исполнение задерживается, оно обычно не срабатывает по первой паре в списке, и иногда на самом деле будет срабатывать только целая куча на самом последнем элементе.

Я знаю, что это связано с тем, что c является ссылкой, которая изменяется до того, как Invoke действительно вызван, но есть ли способ избежать этого?

Выполнение общего цикла 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)); 
     } 
    } 
} 

ли веб-поиска: «Доступ к модифицированному закрытия», если вы хотите получить больше информации.

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

установка Try с локальным переменным и вызова LazyAsync.Invoke на том, что, чтобы избежать гр переподчиняемого к петле Еогеаспа перед Invoke происходит. Когда LazyAsync.Invoke делает c.WriteMessage, он звонит WriteMessage на то, что с происходит с теперь указывают на то, что, не это было, когда LazyAsync.Invoke (() => c.WriteMessage (см)) оценивали

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

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