Stosując metody anonimowe wewnątrz opóźnionego zadania wewnątrz pętli

Question

Mam następujący -

for (int i = 0; i < N; ++i) 
{ 
    var firstTask = DoSomething(...); 

    var task = anotherTask.ContinueWith(
     (t) => ProcessResult(i, t.Result)); 
} 

Problem jest, że wartość i przeszedł do ProcessResult wydaje się być wartość, kiedy zaczyna, nie wartość iteracja po jej utworzeniu.

Jaki jest najlepszy sposób ochrony przed tym?

Answer 1

Musisz przechwycić wartość i w swojej własnej zmiennej.

for (int i = 0; i < N; ++i) 
{ 
    var count = i; 

    var firstTask = DoSomething(...); 

    var task = anotherTask.ContinueWith(
     (t) => ProcessResult(count, t.Result)); 
} 

Przykład:

for (int i = 0; i < 5; ++i) 
{ 
    var a = i; 
    var task = Task.Delay(0).ContinueWith((t) => a.Dump()); 
} 

wyprowadza To coś jak:

0 
2 
1 
4 
3 

Ale:

for (int i = 0; i < 5; ++i) 
{ 
    var task = Task.Delay(0).ContinueWith((t) => i.Dump()); 
} 

Wyjścia:

5 
5 
5 
5 
5 

Answer 2

Musisz utworzyć zmienną temp wewnątrz pętli; w bieżącym kodzie przechwytywana jest zmienna , która nie jest wartością, co oznacza, że ​​po ostatecznym wykonaniu zadań kontynuacji pętla została już zakończona, a i jest N-1.

for (int i = ...) 
{ 
    var temp = i; 
    var task = anotherTask.ContinueWith(t => ProcessResult(temp, t.Resume)); 
} 

Answer 3

Wartość lambda, która wykorzystuje zmienną zewnętrzną, przechwytuje zmienną, a nie zapisaną w niej wartość. Oznacza to, że w miarę postępu w pętli zmienia się również wartość, którą można odczytać z przechwyconej zmiennej.

Można to naprawić, używając zmiennej tymczasowej w pętli. Twój kod będzie dużo czystsze chociaż jeśli użyto async/await zamiast ContinueWith i lambda, np:

for (int i=0;i<N;i++) 
{ 
    //... 
    var result=await thatOtherAsyncMethod(...); 
    ProcessResult(i, result)); 
} 

Ogólnie, można uniknąć problemów przechwytywania kopiując zmiennej pętli do zmiennej zdefiniowanej wewnątrz zakresu pętli za.

Rozwiązuje to problem, ponieważ zmienna tymczasowa istnieje tylko w ciele pętli. Lambda jest również tworzone wewnątrz ciała pętli i oddaje lokalną, niezmienny zmiennej:

for (int i=0;i<N;i++) 
{ 
    var temp=i; 
    var myLambda = new Action(()=>MyMethod(temp)); 

    //This runs with the local copy, not i 
    myLambda(); 
} 

Jeszcze lepszym sposobem jest jednak do unikać przechwytuje i przekazać wartość pętli jako parametr państwowej do ContinueWith, np .:

for (int i = 0; i < N; ++i) 
{ 
    //... 
    var task = anotherTask.ContinueWith(
           (t,state) => ProcessResult((int)state, t.Result), 
           i); 
    //... 
}