Sử dụng phương pháp vô danh bên trong một nhiệm vụ bị trì hoãn trong vòng lặp

Question

Tôi đã sau -

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

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

Vấn đề là rằng giá trị của i thông qua vào ProcessResult dường như là giá trị khi nó bắt đầu, không phải là giá trị của lặp lại khi nó được tạo ra.

Cách tốt nhất để bảo vệ chống lại điều này là gì?

Answer 1

Bạn cần nắm bắt giá trị i thành biến riêng.

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

    var firstTask = DoSomething(...); 

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

Ví dụ:

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

này kết quả đầu ra một cái gì đó như:

0 
2 
1 
4 
3 

Nhưng điều này:

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

Đầu ra:

5 
5 
5 
5 
5 

Answer 2

Bạn cần tạo biến tạm thời trong vòng lặp; trong mã hiện tại của bạn, bạn đang chụp biến i, không phải là giá trị, có nghĩa là khi các tác vụ tiếp tục được thực hiện cuối cùng, vòng lặp đã kết thúc và i là N-1.

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

Answer 3

Một lambda sử dụng biến ngoài thực sự nắm bắt biến, chứ không phải giá trị được lưu trữ trong biến đó. Điều đó có nghĩa là khi tiền tố lặp lại, giá trị bạn sẽ đọc từ biến bị bắt cũng thay đổi.

Bạn có thể sửa lỗi này bằng cách sử dụng biến tạm thời bên trong vòng lặp. Mã của bạn sẽ có rất nhiều bụi tuy nhiên nếu bạn sử dụng async/await thay vì ContinueWith và lambdas, ví dụ:

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

Nói chung, bạn có thể tránh được vấn đề chụp bằng cách sao chép các biến vòng lặp thành một biến được định nghĩa bên trong phạm vi của vòng lặp.

Điều này khắc phục sự cố vì biến tạm thời chỉ tồn tại bên trong phần thân của vòng lặp. Lambda là cũng tạo ra bên trong cơ thể của vòng lặp và chụp một địa phương, biến không thay đổi:

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

Một thậm chí cách tốt hơn, mặc dù là để tránh chụp và vượt qua giá trị vòng lặp như các tham số trạng thái để ContinueWith, ví dụ:

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