Problem z innym "kontekstem wykonania" anonimowej metody w pętli

Question

Mam problem z anonimową metodą w pętli.

Poniższy kod jest tylko do zilustrowania mój problem:

private void Form1_Load(object sender, EventArgs e) 
{ 
    List<string> bassists = new List<string>(){ 
     "Jaco Pastorius", 
     "Marcus Miller", 
     "Flea", 
     "Vicor Wooten" 
    }; 

    foreach (string item in bassists) 
    { 
     this.button1.Click += (s, ea) => Output(s, ea, item); 
    } 
} 

private void Output(object s, EventArgs e, string item) 
{ 
    this.listBox1.Items.Add(item); 
} 

A kiedy kliknij w przycisk, wyjście jest:

Victor Wooten
Victor Wooten
Victor Wooten
Victor Wooten

zamiast:

Jaco Pastorius
Marcus Miller
Flea
Vicor Wooten

Głównym punktem mojego problemu jest wykonanie zróżnicowanych kontekst. Wiem, że mój przykład jest głupi.

Answer 1

To jest problem z przechwytywaną zmienną. Napraw to zmieniając

foreach (string item in bassists) 
{ 
    this.button1.Click += (s, ea) => Output(s, ea, item); 
} 

do

foreach (string item in bassists) 
{ 
    string currentItem = item; 
    this.button1.Click += (s, ea) => Output(s, ea, currentItem); 
} 

Oto wyjaśnienie kwestii: Closing over loop variable considered harmful. Umieszczając zmienną lokalną currentItem w zasięgu pętli i zamykając ją, przechwytujemy teraz tę zmienną zamiast zmiennej pętli.

Answer 2

Twój problem polega na tym, że tworzysz nowe pętle w pętli, które są niepotrzebne i niebezpieczne.

Również tworzysz anonimową metodę, która ma zakodowaną wartość w pętli. To jest gorsze.

Answer 3

W każdym przypadku odpowiedź Jasona jest prawidłowa. Jest to problem przechwytywania zmiennych. Będzie to w większości przypadków występować w dwóch sytuacjach Metody gwintowania i Anonymous