Problema con l'assegnazione di delegati nel ciclo for-

Question

Possiedo un'applicazione in grado di plug-in (MEF). I plugin sono UserControls WPF che importano i servizi.

L'utente può selezionare il plugin desiderato dal menu principale dell'applicazione.

Per fare questo, io uso il seguente ciclo:

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); 
} 

che funziona molto bene per un singolo elemento. Ma non appena ho più di 1 plugin, tutti i menu eseguono il delegato dell'ultimo ciclo. O almeno con il Plugin.Control dell'ultimo ciclo.

Come posso risolvere questo problema?
Grazie per qualsiasi aiuto.

Answer 1

Su ogni iterazione del ciclo, è necessario "acquisire" il valore del valore iterato prima di utilizzarlo in una chiusura. In caso contrario, il plug-in di ciascun delegato punterà all'ultimo valore del plugin anziché al valore che ha mantenuto quando è stata creata la funzione anonima.

Si può leggere una più approfondita spiegazione di Eric Lippert qui:

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

In breve, il modo corretto di scrivere il ciclo 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); 
}