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。这可以确保我的关闭不会引用将会更改的项目。

在使用本地化之前，事件会触发当前值，而不是我预期的捕获值。