为什么我在调用 Func时得到错误的结果？

Question

我在 C# 中有以下代码片段：

var actions = new List<Func<int>>();

IEnumerable<int> values = new List<int> { 1, 2, 3 };

foreach (int value in values)
{
    actions.Add(() => value * value);
}

foreach (var action in actions)
{
    Console.WriteLine(action()); ;
}

Console.ReadLine();

它运行良好，但我没有得到我期望的结果。

实际结果

9,9,9

预期结果

1,4,9

为什么我没有得到预期的结果？

Answer 1

您需要捕获循环内的变量。现在，您的延迟执行操作正在使用第一个 foreach 循环中 value 的最后一个值。

var actions = new List<Func<int>>();
        IEnumerable<int> values = new List<int> { 1, 2, 3 };
        foreach (int value in values)
        {
            var v = value;
            actions.Add(() => v * v);
        }
        foreach (var action in actions)
        {
            Console.WriteLine(action()); ;
        }
        Console.ReadLine();

请注意 var v = value; 行。

Answer 2

您正在捕获 lambda 表达式中的循环变量，这意味着当最终调用委托时，它会看到循环变量的最终值。

简单修复：

foreach (int value in values)
{
    int copy = value;
    actions.Add(() => copy * copy);
}

通过这种方式，您可以在循环的每次迭代中获得一个新的 copy 变量，因此每个委托表达式将捕获不同的变量，并且它们不受循环变量（value）随着时间的推移而变化。

Eric Lippert 在 “关闭被认为有害的循环变量”（和 第二部分< /a>）。

基本上，这是 C# 中的一个“陷阱”，几乎每个人迟早都会陷入其中。