C＃在foreach中重用变量是否有原因？

在C＃中使用lambda表达式或匿名方法时，我们必须警惕对修改后的闭包陷阱的访问。例如：

foreach (var s in strings)
{
   query = query.Where(i => i.Prop == s); // access to modified closure
   ...
}

由于修改了闭包，因此上述代码将导致Where查询中的所有子句都基于的最终值s。

正如解释在这里，这是因为该s变量在声明foreach环以上的编译器编译如下：

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

而不是像这样：

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

如此处所指出的，在循环外声明变量没有任何性能优势，在正常情况下，我能想到的唯一原因是如果您打算在循环范围外使用变量：

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

但是，foreach循环中定义的变量不能在循环外使用：

foreach(string s in strings)
{
}
var finalString = s; // won't work: you're outside the scope.

因此，编译器以某种方式声明该变量，使其极易出现通常难以查找和调试的错误，而不会产生明显的收益。

是否可以通过foreach这种方式对循环执行某些操作，如果它们是使用内部作用域变量进行编译则无法做到的，或者这只是在匿名方法和lambda表达式可用或通用之前做出的任意选择，并且没有从那以后就没有修改过？

编译器以一种很容易出错的方式声明该变量，该错误通常很难查找和调试，同时不会产生明显的好处。

您的批评是完全有道理的。

我在这里详细讨论这个问题：

关闭循环变量被认为是有害的

使用foreach循环，是否可以通过内部作用域变量进行编译而无法做到？还是这只是在匿名方法和lambda表达式可用或通用之前做出的任意选择，并且自那时以来没有进行过修改？

后者。实际上，C＃1.0规范没有说明循环变量是在循环体内还是在循环体内，因为它没有明显的区别。在C＃2.0中引入闭包语义时，选择将循环变量放入循环之外，与“ for”循环一致。

我认为可以说所有人都对该决定表示遗憾。这是C＃中最糟糕的“陷阱”之一，我们将进行重大更改以对其进行修复。在C＃5中，foreach循环变量在逻辑上将位于循环体内，因此闭包每次都会获得新的副本。

该for循环将不会改变，并且改变不会是“向后移植”到C＃的早期版本。因此，在使用此惯用语时，您应继续小心。

你所问的是彻底埃里克利珀在他的博客盖关闭了循环变量认为是有害的和它的续集。

对我而言，最令人信服的论据是，每次迭代中都有新变量与for(;;)样式循环不一致。您是否希望int i在的每次迭代中都有一个新的for (int i = 0; i < 10; i++)？

此行为最常见的问题是对迭代变量进行闭包，并且有一个简单的解决方法：

foreach (var s in strings)
{
    var s_for_closure = s;
    query = query.Where(i => i.Prop == s_for_closure); // access to modified closure

我的博客文章有关此问题：C＃中的foreach变量关闭。

被这个问题咬住之后，我有一个习惯，就是将局部定义的变量包含在我用来传递给任何闭包的最内层作用域中。在您的示例中：

foreach (var s in strings)
    query = query.Where(i => i.Prop == s); // access to modified closure

我做：

foreach (var s in strings)
{
    string search = s;
    query = query.Where(i => i.Prop == search); // New definition ensures unique per iteration.
}        

一旦有了这种习惯，在实际打算绑定到外部范围的极少数情况下，就可以避免这种习惯。老实说，我认为我从未这样做过。

在C＃5.0中，此问题已修复，您可以关闭循环变量并获得所需的结果。

语言规范说：

8.8.4 foreach语句

（...）

形式的foreach语句

foreach (V v in x) embedded-statement

然后扩展为：

{
  E e = ((C)(x)).GetEnumerator();
  try {
      while (e.MoveNext()) {
          V v = (V)(T)e.Current;
          embedded-statement
      }
  }
  finally {
      … // Dispose e
  }
}

（...）

vwhile循环内部的位置对于嵌入式语句中发生的任何匿名函数如何捕获它很重要。例如：

int[] values = { 7, 9, 13 };
Action f = null;
foreach (var value in values)
{
    if (f == null) f = () => Console.WriteLine("First value: " + value);
}
f();

如果v在while循环之外声明，它将在所有迭代之间共享，并且for循环之后的值将是最终值13，这是调用的f结果。相反，由于每个迭代都有其自己的变量v，因此f在第一次迭代中捕获的变量将继续保留value 7，即将要打印的值。（注意：v在while循环外部声明的C＃的早期版本。）