3

我不太清楚两个代码块之间的区别。考虑有一个程序

    class Program
{
    static void Main(string[] args)
    {

        List<Number> numbers = new List<Number>
                                   {
                                       new Number(1),
                                       new Number(2),
                                       new Number(3)
                                   };


        List<Action> actions = new List<Action>();
        foreach (Number numb in numbers)
        {
            actions.Add(() => WriteNumber(numb));
        }

        Number number = null;
        IEnumerator<Number> enumerator = numbers.GetEnumerator();
        while (enumerator.MoveNext())
        {
            number = enumerator.Current;
            actions.Add(() => WriteNumber(number));
        }

        foreach (Action action in actions)
        {
            action();
        }

        Console.ReadKey();


    }

    public static void WriteNumber(Number num)
    {
        Console.WriteLine(num.Value);
    }

    public class Number
    {
        public int Value;

        public Number(int i)
        {
            this.Value = i;
        }

    }

}

输出是

1
2
3
3
3
3

这两个代码块应该相同地工作。但是您可以看到闭包不适用于第一个循环。我错过了什么？

提前致谢。

4

4 回答 4

3

number您在 while 循环之外声明变量。对于每个数字，您将其引用存储在number变量中 - 每次覆盖最后一个值。

您应该只在 while 循环内移动声明，这样每个数字都有一个新变量。

    IEnumerator<Number> enumerator = numbers.GetEnumerator();
    while (enumerator.MoveNext())
    {
        Number number = enumerator.Current;
        actions.Add(() => WriteNumber(number));
    }
于 2013-10-03T14:26:00.297 回答
3

这两个代码块应该相同地工作。

不，他们不应该——至少在 C# 5 中。事实上，在 C# 3 和 4 中他们会。

但是在foreach循环中，在 C# 5 中，每次循环迭代都有一个变量。您的 lambda 表达式会捕获该变量。循环的后续迭代创建不影响先前捕获的变量的不同变量。

在while循环中，您有一个所有迭代都会捕获的变量。对该变量的更改将在所有捕获它的委托中看到。您可以通过在while循环后添加这一行来看到这一点：

number = new Number(999);

那么你的输出将是

1
2 
3
999
999
999

现在在 C# 3 和 4 中，foreach规范基本上被设计破坏了 - 它会在所有迭代中捕获单个变量。然后在 C# 5 中修复了这个问题，每次迭代都使用一个单独的变量，这基本上是你一直想要的那种代码。

于 2013-10-03T14:26:05.120 回答
1

在你的循环中：

    Number number = null;
    IEnumerator<Number> enumerator = numbers.GetEnumerator();
    while (enumerator.MoveNext())
    {
        number = enumerator.Current;
        actions.Add(() => WriteNumber(number));
    }

number 在循环范围之外声明。因此，当它设置为下一个当前迭代器时，您对 number 的所有操作引用也会更新为最新的。因此，当您运行每个操作时，它们都将使用最后一个数字。

于 2013-10-03T14:28:02.670 回答
0

感谢您的所有回答。但我想我被误解了。我希望关闭工作。这就是为什么我将循环变量设置在范围之外。问题是：为什么它在第一种情况下不起作用？我忘了提到我使用的是 C# 3.5（不是 C# 5.0）。因此，soop 变量应该定义在范围之外，并且两个代码块应该相同地工作。

于 2013-10-04T09:43:34.580 回答