访问的foreach变量封警告

Question

我得到以下警告：

访问的foreach在封闭的可变。使用不同版本的编译器编译时可能会有不同的行为。

这就是它看起来像在我的编辑器：

我知道如何解决这个问题的警告，但我想知道为什么我会得到这样的警告？

这是关于“CLR”版本吗？它与“IL”有关吗？

Answer 1

此警告有两个部分。首先是...在封闭

...这是不是无效本身

访问的foreach变量，但它是反直觉的第一眼。做对也很困难。 （以至于我在下面链接的文章将其描述为“有害”）。

请注意，您摘录的代码基本上是C＃编译器（在C＃5之前）的扩展形式，生成foreach ：

我[不]明白为什么[以下是]无效：

string s; while (enumerator.MoveNext()) { s = enumerator.Current; ... 

嗯，这是有效的语法。如果你在循环中所做的只是使用的值的s那么一切都很好。但超过s将导致违反直觉的行为。看看下面的代码：

var countingActions = new List<Action>(); 

var numbers = from n in Enumerable.Range(1, 5) 
       select n.ToString(CultureInfo.InvariantCulture); 

using (var enumerator = numbers.GetEnumerator()) 
{ 
    string s; 

    while (enumerator.MoveNext()) 
    { 
     s = enumerator.Current; 

     Console.WriteLine("Creating an action where s == {0}", s); 
     Action action =() => Console.WriteLine("s == {0}", s); 

     countingActions.Add(action); 
    } 
} 

如果你运行这段代码，你会得到以下控制台输出：

Creating an action where s == 1 
Creating an action where s == 2 
Creating an action where s == 3 
Creating an action where s == 4 
Creating an action where s == 5 

这是你所期望的。

要看到的东西你可能没有想到，运行下面的代码后立即上面的代码：

foreach (var action in countingActions) 
    action(); 

您将获得以下控制台输出：

s == 5 
s == 5 
s == 5 
s == 5 
s == 5 

为什么？因为我们创建了五个函数，它们都完全相同：打印s（我们已经关闭）的值。实际上，它们是相同的功能（“打印s”，“打印s”，“打印s”...）。

在我们开始使用它们的时候，它们完全按照我们的要求：打印s的值。如果您查看s的最后一个已知值，您会看到它是5。所以我们得到s == 5五次打印到控制台。

这正是我们所要求的，但可能不是我们想要的。

警告的第二部分...当使用不同版本的编译器编译

可能有不同的行为。

......就是这样。 Starting with C# 5, the compiler generates different code that "prevents" this from happening via foreach。

因此，下面的代码将产生在不同版本的编译器不同的结果：

foreach (var n in numbers) 
{ 
    Action action =() => Console.WriteLine("n == {0}", n); 
    countingActions.Add(action); 
} 

因此，还会产生将R＃警告:)

我的第一个代码段，上方，将在所有版本的编译器中都表现出相同的行为，因为我没有使用foreach（相反，我已经将它扩展到C＃5之前的编译器的方式）。

这是用于CLR版本吗？

我不太确定你在问什么。

Eric Lippert的文章称这种变化发生在“C＃5”中。所以大概你必须使用C＃5或更高版本的编译器将.NET 4.5或更高版本的目标设为以获得新的行为，并且之前的所有内容都会得到旧的行为。

但要清楚，它是编译器的功能，而不是.NET Framework版本。

与IL有关吗？

不同的代码会产生不同的IL，因此在这个意义上说IL会产生后果。

foreach是比您在评论中发布的代码更常见的结构。这个问题通常是通过使用foreach而不是通过手动枚举产生的。这就是为什么在C＃5中对foreach的更改有助于防止此问题，但不完全。

Answer 2

第一个答案很好，所以我想我只是添加一件事。

因为在您的示例代码中，reflectModel被分配了一个IEnumerable，而这只会在枚举时进行评估，并且枚举本身可能发生在循环之外，如果您将reflectModel分配给了某些东西范围更广。

如果更改

...Where(x => x.Name == property.Value)

到

...Where(x => x.Name == property.Value).ToList()

然后reflectedModel将被分配foreach循环内的肯定列表，这样你就不会收到警告，因为枚举肯定会发生在循环内，而不是在循环之外。

Answer 3

块范围的变量应该解决警告。

foreach (var entry in entries) 
{ 
    var en = entry; 
    var result = DoSomeAction(o => o.Action(en)); 
}