Bẫy biến bên ngoài

Question

Bẫy biến ngoài là gì? Giải thích và ví dụ trong C# được đánh giá cao.

EDIT: Kết hợp diktat Jon Skeet :)

Eric Lippert on the Outer Variable Trap

Answer 1

Các "Trap Biến Outer" xảy ra khi một nhà phát triển hy vọng giá trị của một biến được chụp bởi một biểu thức lambda hoặc đại biểu vô danh, khi thực sự biến được chụp chính nó.

Ví dụ:

var actions = new List<Action>(); 
for (var i = 0; i < 10; i++) 
{ 
    actions.Add(() => Console.Write("{0} ", i)); 
} 
foreach (var action in actions) 
{ 
    action(); 
} 

có thể sản lượng # 1:

0 1 2 3 4 5 6 7 8 9 

có thể sản lượng # 2:

10 10 10 10 10 10 10 10 10 10 

Nếu bạn mong đợi đầu ra # 1, bạn đã rơi vào Trap biến ngoài. Bạn nhận được đầu ra # 2.

Fix:

Khai báo một "Biến Nội" để được chụp liên tục thay vì "Biến Outer" được chụp chỉ một lần.

var actions = new List<Action>(); 
for (var i = 0; i < 10; i++) 
{ 
    var j = i; 
    actions.Add(() => Console.Write("{0} ", j)); 
} 
foreach (var action in actions) 
{ 
    action(); 
} 

Để biết thêm chi tiết, xem thêm Eric Lippert's blog.

Answer 2

Something như

foreach (var s in strings) 
    var x = results.Where(r => (r.Text).Contains(s)); 

Sẽ không cung cấp cho các kết quả mà bạn đang mong đợi vì Có không được thực hiện cho mỗi lần lặp. Gán s cho một biến tạm thời bên trong vòng lặp sẽ sửa lỗi này.

Answer 3

@dtb là chính xác (+1 lớn), nhưng điều quan trọng cần lưu ý là điều này chỉ áp dụng nếu phạm vi của việc đóng cửa kéo dài ngoài vòng lặp. Ví dụ:

var objects = new [] 
    { 
     new { Name = "Bill", Id = 1 }, 
     new { Name = "Bob", Id = 5 }, 
     new { Name = "David", Id = 9 } 
    }; 

for (var i = 0; i < 10; i++) 
{ 
    var match = objects.SingleOrDefault(x => x.Id == i); 

    if (match != null) 
    { 
     Console.WriteLine("i: {0} match: {1}", i, match.Name); 
    } 
} 

này sẽ in:

i: 1 match: Bill 
i: 5 match: Bob 
i: 9 match: David

ReSharper sẽ cảnh báo về "Tiếp cận đóng cửa sửa đổi," có thể được bỏ qua một cách an toàn trong trường hợp này.

Answer 4

Bài viết này giải thích các khái niệm về việc đóng cửa là hữu ích:

http://en.wikipedia.org/wiki/Closure_(computer_science)

Ngoài ra, bài viết này là thực sự tốt từ một cụ thể hơn C# thực hiện:

http://blogs.msdn.com/b/abhinaba/archive/2005/08/08/448939.aspx

Dù sao, tl lr là phạm vi biến đó cũng quan trọng trong các biểu thức đại biểu hoặc lambda ẩn danh như nó ở bất kỳ nơi nào khác trong mã của bạn - hành vi này không rõ ràng.

Answer 5

Nó xứng đáng để lưu ý rằng cái bẫy này tồn tại cho foreach vòng quá nhưng has been changed từ C# 5.0, ví dụ: bên foreach vòng đóng cửa bây giờ gần hơn một bản sao tươi của biến vòng lặp mỗi lần. Vì vậy, các mã bên dưới:

var values = new List<int>() { 100, 110, 120 }; 
var funcs = new List<Func<int>>(); 
foreach (var v in values) 
    funcs.Add(() => v); 
foreach (var f in funcs) 
    Console.WriteLine(f()); 

Prints 120 120 120< C# 5.0, nhưng 100 110 120> = C# 5.0

Tuy nhiên for vòng vẫn cư xử theo cùng một cách.