动作委托使用在foreach循环之外声明的最后一个变量值

我有这段代码:

int i = 0;
foreach(var tile in lib.dic.Values)
{
    var ii = i;
    var t = tile;
    Button b = new Button( () = > { MainStatic.tile = t; } );
    Checkbox c = new Checkbox( () = > { lib.arr[ii].b = !lib.arr[ii].b; } );
    i++;
}

虽然上面的代码正常工作,但下面这段代码:

int i = 0;
foreach(var tile in lib.dic.Values)
{
    Button b = new Button( () = > { MainStatic.tile = tile; } );
    Checkbox c = new Checkbox( () = > { lib.arr[i].b = !lib.arr[i].b; } );
    i++;
}

...将始终使用itile变量的最后值执行代表。 为什么会发生这种情况,为什么我必须制作这些变量的本地副本,尤其是非引用类型的int i


已知的“问题”,请查看Eric的博客Closures,获取变量。

微软决定进行一次重大更改,并将其修复到C#5中。


您不能像这样使用循环变量,因为在执行委托时,循环变量可能会处于其最终(循环结束)状态,因为它使用执行删除时变量的值而不是创建的值。

您需要创建变量的本地副本才能使其工作:

int i = 0;
foreach(var tile in lib.dic.Values)
{
    var tileForClosure = tile;
    var iForClosure = i;
    Button b = new Button( () = > { MainStatic.tile = tileForClosure ; } );
    Checkbox c = new Checkbox( () = > { lib.arr[iForClosure].b = !lib.arr[iForClosure].b; } );
    i++;
}

通过在每个循环上创建本地副本,值不会更改,因此您的委托将使用您期望的值。


这是预期的:当你做一个lambda时,编译器会创建一个闭包。 它将捕获临时变量的值,但它不会捕获在创建lambda后更改的循环变量和其他变量的值。

问题的核心是代表创建和执行时间不同。 委托对象是在循环运行时创建的,但在循环完成后调用它。 在调用委托时,循环变量具有在循环完成时达到的值,从而产生您所看到的效果(该值不会更改,并且您会看到循环中的最后一个值)。

忘记创建一个用于闭包的临时变量是一种常见的错误,流行的代码分析器(例如ReSharper)会警告您。

链接地址: http://www.djcxy.com/p/52781.html

上一篇: Action delegate uses the last values of variables declared outside foreach loop

下一篇: Unity3D C# Script : Class Variable Array Data Issue