До 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;
Аналогично, если Вы хотите разовый обработчик событий (такой как Загрузка и т.д.):
EventHandler bar = null; // necessary for "definite assignment"
bar = delegate {
// ... code
obj.SomeEvent -= bar;
};
obj.SomeEvent += bar;
Это теперь самоотказывается от подписки;-p