在for循环中分配代表的问题[重复]

Question

我有一个支持插件 (MEF) 的应用程序。 插件是导入服务的 WPF 用户控件。

用户可以从应用程序的主菜单中选择想要的插件。

为此，我使用以下循环：

foreach(IToolPlugin Plugin in ToolPlugins)
{
    Plugin.Init();
    MenuItem PluginMenuItem = Plugin.MenuItem; //New MenuItem but with Header set.
    PluginMenuItem.Click += new RoutedEventHandler(delegate(object o, RoutedEventArgs e) { DoSomething(Plugin.Control);});
    PluginsMenu.Items.add(PluginMenuItem);
}

这对于单个项目非常有效。但是只要我有超过 1 个插件，所有菜单项都会执行最后一个循环的委托。或者至少使用最后一个循环的 Plugin.Control。

我该如何解决这个问题？
感谢您的帮助。

Answer 1

在循环的每次迭代中，您必须先“捕获”迭代值的值，然后才能在闭包中使用它。否则，每个委托中的 Plugin 将指向 Plugin 的最后一个值，而不是它在创建匿名函数时所持有的值。

您可以在此处阅读 Eric Lippert 的更深入解释：

Closing over the loop variable considered harmful - Fabulous Adventures in Coding

简而言之，编写 foreach 循环的正确方法是：

foreach(IToolPlugin Plugin in ToolPlugins)
{
    Plugin.Init();
    MenuItem PluginMenuItem = Plugin.MenuItem;

    IToolPlugin capturedPlugin = Plugin;

    PluginMenuItem.Click += 
        new RoutedEventHandler(delegate(object o, RoutedEventArgs e) {
            DoSomething(capturedPlugin.Control);
        });

    PluginsMenu.Items.add(PluginMenuItem);
}