这是对Access to Modified Closure的扩展。我只想验证以下内容是否真的足够安全用于生产。
List<string> lists = new List<string>();
//Code to retrieve lists from DB
foreach (string list in lists)
{
Button btn = new Button();
btn.Click += new EventHandler(delegate { MessageBox.Show(list); });
}
每次启动我只会执行一次以上操作。目前看来,它工作正常。正如乔恩(Jon)在某些情况下提到的违反直觉的结果。那么,我需要在这里注意什么?如果列表多次运行,可以吗?
最佳答案
在C#5之前,您需要在foreach中重新声明一个变量-否则它将被共享,并且所有处理程序都将使用最后一个字符串:
foreach (string list in lists)
{
string tmp = list;
Button btn = new Button();
btn.Click += new EventHandler(delegate { MessageBox.Show(tmp); });
}
值得注意的是,从C#5开始,这种情况已经改变,特别是在
foreach
的情况下,您不再需要这样做:问题中的代码将按预期工作。若要显示此更改不起作用,请考虑以下事项:
string[] names = { "Fred", "Barney", "Betty", "Wilma" };
using (Form form = new Form())
{
foreach (string name in names)
{
Button btn = new Button();
btn.Text = name;
btn.Click += delegate
{
MessageBox.Show(form, name);
};
btn.Dock = DockStyle.Top;
form.Controls.Add(btn);
}
Application.Run(form);
}
在C#5 之前运行上述,尽管每个按钮显示不同的名称,但是单击按钮将显示“Wilma”四次。
这是因为语言规范(ECMA 334 v4,15.8.4)(在C#5之前)定义了:
请注意,变量
v
(这是您的list
)在循环外部声明。因此,根据捕获变量的规则,列表的所有迭代将共享捕获变量的持有者。从C#5开始,这已更改:迭代变量(
v
)的作用域位于循环内的范围内。我没有规范引用,但是它基本上变成了:{
E e = ((C)(x)).GetEnumerator();
try {
while (e.MoveNext()) {
V v = (V)(T)e.Current;
embedded-statement
}
}
finally {
… // Dispose e
}
}
重新订阅;如果您主动希望退订匿名处理程序,则诀窍是捕获处理程序本身:
EventHandler foo = delegate {...code...};
obj.SomeEvent += foo;
...
obj.SomeEvent -= foo;
同样,如果您想要一个一次性事件处理程序(例如Load等):
EventHandler bar = null; // necessary for "definite assignment"
bar = delegate {
// ... code
obj.SomeEvent -= bar;
};
obj.SomeEvent += bar;
现在这是自我取消订阅;-p
关于c# - 访问修改后的闭包(2),我们在Stack Overflow上找到一个类似的问题:https://stackoverflow.com/questions/304258/