101

这是从Access 到 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 在某些情况下提到的违反直觉的结果。那么我在这里需要注意什么？如果列表不止一次运行可以吗？

4

1 回答 1

159

在 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 之前）定义：

foreach (V v in x) embedded-statement然后扩展为：

{
    E e = ((C)(x)).GetEnumerator();
    try {
        V v;
         while (e.MoveNext()) {
            v = (V)(T)e.Current;
             embedded-statement
        }
    }
    finally {
        … // Dispose e
    }
}

请注意，变量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

于 2008-11-20T05:13:58.063 回答