Question

我无法理解两块代码之间的区别。考虑有一个程序

    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;
        }

    }

}

输出

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

提前致谢。

Answer 1

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

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

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

Answer 2

这两个代码块应该完全相同。

不，他们不应该 - 至少在C＃5中。在C＃3和4中，他们实际上。

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

在while循环中，您有一个变量，其中 all 迭代捕获。对该变量的更改将在捕获它的代理的 all 中看到。你可以在while循环后添加这一行来看到这一点：

number = new Number(999);

然后你的输出将是

现在在C＃3和4中，foreach规范基本上被设计破坏了 - 它将在所有迭代中捕获单个变量。然后在C＃5中对此进行了修复，以便每次迭代使用一个单独的变量，这基本上就是总是想要的那种代码。

Answer 3

在你的循环中：

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

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

Answer 4

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

C＃Closure没有按预期工作

4 个答案: