访问修改后的闭包(2)


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)在某些情况下提到的违反直觉的结果。那么,我需要在这里注意什么?如果列表多次运行,可以吗?


18
恭喜,您现在已成为Resharper文档的一部分。confluence.jetbrains.net/display/ReSharper/…–
Kongress

1
这很棘手,但是上面的解释对我来说很清楚: 这似乎是正确的,但实际上,每当单击任何按钮时,只会使用str变量的最后一个值。这样做的原因是foreach进入了while循环,但是迭代变量是在该循环之外定义的。这意味着,当您显示消息框时,str的值可能已经被迭代为字符串集合中的最后一个值。
DanielV

Answers:


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(是your 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


在这种情况下,临时变量将保留在内存中,直到应用程序关闭为止(以服务于委托),并且如果变量占用大量内存,则建议不要在非常大的循环中执行此操作。我对吗?
有故障

1
它将在内存中保留事件的时间(按钮)。有一种取消订阅仅一次委托的方法,我将其添加到帖子中。
马克·格雷韦尔

2
但是要证明您的观点:是的,捕获的变量确实可以扩大变量的范围。您需要注意不要捕捉到您不希望看到的东西...
马克·格拉维

1
您能否就C#5.0规范的更改来更新您的答案?只是为了使它成为有关C#中foreach循环的出色wiki文档。关于C#5.0编译器处理foreach循环bit.ly/WzBV3L的更改,已经有了一些很好的答案,但是它们不是类似于wiki的资源。
Ilya Ivanov

1
@Kos是的,for在5.0中不变
Marc Gravell
By using our site, you acknowledge that you have read and understand our Cookie Policy and Privacy Policy.
Licensed under cc by-sa 3.0 with attribution required.