Resharper是否不必要地警告我關於訪問修改的關閉？

Question

...所以我有這樣的代碼：

 foreach (var entry in list) 
     { 
      var marginOneEntry = otherList.FirstOrDefault(p => p.Margin == entry.Margin);     
      if (marginOneEntry == null) continue; 
      // Do stuff with marginOneEntry 
     } 

和ReSharper的5.1警告我，「進入修改封」在lambda表達式中比較語句使用entry。 FirstOrDefault-call是不是取消了這個問題？我的代碼不好，或者這個警告只是Resharper的一個缺點？

注意：我已經閱讀了關於在此處訪問修改的閉包的其他主題，但我想要爲這個特定案例找到答案，並且澄清Resharper是否對此主題過於敏感。

Answer 1

是的，這是ReSharper的一個問題，因爲它沒有區分與懶惰評估結果從方法不使用在懶惰計算給定lambda值的方法。 FirstOrDefault不會做懶惰的評估（忘記與'lazy'相對的術語），因此可以安全地在您的示例中使用它，但通常無法從方法的簽名中理解。

Answer 2

我覺得你在代碼中犯了一個錯誤之上。您的.Where(p => entry.Margin == 1)先決條件不涉及被枚舉的元素。你的意思是p.Margin == 1？

此外，而不是寫作x.Where(y).FirstOrDefault()你可以簡單地寫x.FirstOrDefault(y)。