访问修改后的闭包，这是 ReSharper 错误吗？

Question

我有最新的 ReSharper 5.0 版本 (1655)，我在以下代码中遇到了“访问修改后的闭包”的建议：

var now = new DateTime(1970, 1, 1);
var dates = new List<DateTime>();
dates.Where(d => d > now);
...
now = new DateTime();

并且 lambda 表达式中的 now 带有警告下划线。

我很确定这是一个 ReSharper 错误，但真的是这样吗？

编辑：我应该检查得更好，代码后面有一个分配给now。

EDIT 2 下面 Jon Skeet 的 answer 几乎回答了这个问题，但以下内容呢：

var query = dates.Where(d => d > now).ToList();

这不应该通过立即执行查询来解决问题吗？

Answer 1

好的，现在您已经修改了问题，这完全有道理。您正在修改闭包中使用的变量 - 这可能会产生意想不到的结果：

var now = new DateTime(1970, 1, 1);
var dates = new List<DateTime>();
var query = dates.Where(d => d > now);
...
now = new DateTime(1990, 1, 1);
foreach (DateTime date in query)
{
    // This will only see dates after 1990, not after 1970
    // This would confuse many developers.
}

事实上，这不仅仅是查询何时开始的问题 - 您可以在迭代结果时修改它：

var now = new DateTime(1970, 1, 1);
var dates = new List<DateTime>();
var query = dates.Where(d => d > now);
...
foreach (DateTime date in query)
{
    now = date;
    Console.WriteLine(date);
}

这将给出一个严格递增的日期序列……再次，有点令人困惑。

R# 对此提出警告是绝对正确的，IMO。它有时很有用 - 但应谨慎使用。

Answer 2

ReSharper 警告您的是，now 的值被捕获在 lambda 表达式中，而不是您认为执行 lambda 时的值。

要解决您的问题，您需要在使用之前将now 的值赋给一个局部变量：

var now = new DateTime(1970, 1, 1);
var dates = new List<DateTime>{new DateTime(2001, 12, 12)};
DateTime localNow = now;
dates.Where(d => d > localNow);

now = new DateTime(2003, 12, 12);

如果您想了解更多信息，ReSharper 论坛上有一个post，其中包含几个链接以及进一步的解释。

Answer 3

不，这更清楚地认为您正在修改集合的内容，同时循环它，dates 很可能。您可以放心地忽略这一点。