ReSharper的警告 - 訪問修改關閉

Question

我有以下代碼：

string acctStatus = account.AccountStatus.ToString(); 
if (!SettableStatuses().Any(status => status == acctStatus)) 
    acctStatus = ACCOUNTSTATUS.Pending.ToString(); 

注意account.AccountStatus是類型ACCOUNTSTATUS的枚舉。在第二行中，ReSharper向我發出acctStatus的「訪問修改的關閉」警告。當我做推薦的操作，複製到本地變量，它修改代碼如下：

string acctStatus = realAccount.AccountStatus.ToString(); 
string s = acctStatus; 
if (!SettableStatuses().Any(status => status == s)) 
    acctStatus = ACCOUNTSTATUS.Pending.ToString(); 

這是爲什麼好還是最好什麼我原本？

編輯

委員會還建議在陣列裹局部變量，主要生產：

string[] acctStatus = {realAccount.AccountStatus.ToString()}; 
if (!SettableStatuses().Any(status => status == acctStatus[0])) 
    acctStatus[0] = ACCOUNTSTATUS.Pending.ToString(); 

這似乎是徹頭徹尾的古怪的我。

Answer 1

警告的原因是在循環內部，您可能正在訪問正在更改的變量。然而，在這個非循環的上下文中，「修復」並不是真的爲你做任何事情。

想象一下，你有一個FOR循環，if是在它內部，並且字符串聲明在它之外。在這種情況下，錯誤會正確地識別抓取不穩定參考的問題。

你不想要的一個例子：

string acctStatus 

foreach(...) 
{ 
    acctStatus = account.AccountStatus[...].ToString(); 
    if (!SettableStatuses().Any(status => status == acctStatus)) 
     acctStatus = ACCOUNTSTATUS.Pending.ToString(); 
} 

的問題是，在關閉將搶acctStatus的引用，但每次循環迭代將會改變該值。在該情況下，它會更好：

foreach(...) 
{ 
    string acctStatus = account.AccountStatus[...].ToString(); 
    if (!SettableStatuses().Any(status => status == acctStatus)) 
     acctStatus = ACCOUNTSTATUS.Pending.ToString(); 
} 

由於變量的情況下是循環，一個新的實例將會每次被創建，因爲我們已經搬進了當地的環境裏面的變量（for循環）。

該建議聽起來像是Resharper解析該代碼時的一個錯誤。但是，在很多情況下，這是一個有效的關注點（例如，第一個示例，儘管它在封閉中被捕獲，但引用正在改變）。

我的經驗法則是，如果有疑問做一個本地。

這是一個現實世界的例子，我被咬傷：

 menu.MenuItems.Clear(); 
     HistoryItem[] crumbs = policyTree.Crumbs.GetCrumbs(nodeType); 

     for (int i = crumbs.Length - 1; i > -1; i--) //Run through items backwards. 
     { 
      HistoryItem crumb = crumbs[i]; 
      NodeType type = nodeType; //Local to capture type. 
      MenuItem menuItem = new MenuItem(crumb.MenuText); 
      menuItem.Click += (s, e) => NavigateToRecord(crumb.ItemGuid, type); 
      menu.MenuItems.Add(menuItem); 
     } 

注意，我抓住的NodeType類型的地方，注意節點類型，並HistoryItem crumb.ItemGuid，不屑[I] .ItemGuid。這可以確保我的關閉不會引用將會更改的項目。

在使用本地化之前，事件會觸發當前值，而不是我預期的捕獲值。