Tại sao resharper đề xuất "biến bọc trong mảng" để truy cập vào cảnh báo đóng cửa đã sửa đổi?

Question

Do sau (rất nhiều chỉnh sửa, pseudo-) mã:

int count = 0; 
thing.Stub(m => m.AddBlah()).WhenCalled(o => count++); 
thing.Stub(m => m.RemoveBlah()).WhenCalled(o => count--); 

DoStuff(thing); 

Assert.AreEqual(1, count); 

ReSharper cung cấp một cảnh báo trên đếm - "Tiếp cận đóng cửa sửa đổi". Tôi hiểu tại sao tôi nhận được cảnh báo này (biến đếm được sửa đổi trong hai lambdas khác nhau, và có khả năng có ngữ nghĩa không mong muốn), nhưng tôi không hiểu lời khuyên của ReSharper: "Wrap biến cục bộ trong mảng". Nếu tôi cho phép ReSharper thực hiện việc này, tôi nhận được:

int count[] = { 0 }; 
thing.Stub(m => m.AddBlah()).WhenCalled(o => count[0]++); 
thing.Stub(m => m.RemoveBlah()).WhenCalled(o => count[0]--); 

DoStuff(thing); 

Assert.AreEqual(1, count[0]); 

Và không cảnh báo.

Tại sao sử dụng mảng an toàn?

Answer 1

Điều này là do 2 loại khác nhau. Int là một kiểu giá trị và mảng là một kiểu tham chiếu. Điều này có nghĩa là int nằm trên stack và con trỏ của mảng nằm trên stack.

Khi bạn cập nhật một loại Giá trị, nó sẽ cập nhật phần bộ nhớ ngăn xếp đó. Các loại tham chiếu mặt khác lá đó mảnh ngăn xếp bộ nhớ một mình và sửa đổi những gì nó trỏ đến.

Trình chia sẻ lại không khiếu nại về mảng vì 2 phương pháp Lambda khác nhau đang tạo một khoảng đóng xung quanh bộ nhớ trỏ đến vị trí cập nhật giá trị. Cả Lambdas đều có cùng địa chỉ và không thay đổi địa chỉ gốc.

Answer 2

Tôi nhận thấy điều tương tự này trong chính bản thân mình và cũng tự hỏi tại sao nó không cảnh báo khi giá trị được bọc trong một mảng. Câu trả lời khác ở đây là không may sai, và dường như hiểu lầm cách đóng cửa được thực hiện, vì vậy tôi nghĩ rằng tôi sẽ cố gắng giải thích (những gì tôi nghĩ là) lý do đằng sau này refactoring. Như bạn đã thấy, kết quả là giống nhau cho dù được bọc mảng hay không, vì vậy việc tái cấu trúc không thực sự "sửa" bất cứ điều gì và các vấn đề tương tự có thể gặp phải khi truy cập một đóng cửa được sửa đổi thông thường tồn tại sau khi áp dụng sự thay đổi. Tuy nhiên, sau khi thay đổi vì bản thân countmảng không bị sửa đổi (chỉ nội dung của nó), cảnh báo "Truy cập vào đóng cửa đã sửa đổi" không còn phù hợp nữa.

Sự thay đổi không thực sự làm cho vấn đề trở nên rõ ràng hơn (ít nhất theo ý kiến ​​của tôi), vì vậy dường như gợi ý này về cơ bản nói cho ReSharper bỏ qua vấn đề, mà không cần phải sử dụng đến // ReSharper disable AccessToModifiedClosure cơ chế để ngăn chặn lỗi.