To jest rozszerzenie pytania z Access to Modified Closure . Chcę tylko sprawdzić, czy poniższe elementy są wystarczająco bezpieczne do użytku produkcyjnego.
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); });
}
Wykonuję powyższe tylko raz dla każdego uruchomienia. Na razie wydaje się, że działa dobrze. Jak Jon wspomniał o sprzecznych z intuicją wynikach w niektórych przypadkach. Więc na co muszę tutaj uważać? Czy będzie dobrze, jeśli lista zostanie sprawdzona więcej niż raz?
Odpowiedzi:
Przed C # 5 musisz ponownie zadeklarować zmienną wewnątrz foreach - w przeciwnym razie jest ona współdzielona, a wszystkie twoje programy obsługi będą używać ostatniego ciągu:
Co ważne, zwróć uwagę, że od C # 5 to się zmieniło, a konkretnie w przypadku
foreach
, nie musisz już tego robić: kod w pytaniu działałby zgodnie z oczekiwaniami.Aby pokazać, że to nie działa bez tej zmiany, rozważ następujące kwestie:
Uruchom powyższe przed C # 5 i chociaż każdy przycisk wyświetla inną nazwę, kliknięcie przycisku powoduje wyświetlenie „Wilma” cztery razy.
Dzieje się tak, ponieważ specyfikacja języka (ECMA 334 v4, 15.8.4) (przed C # 5) definiuje:
Zauważ, że zmienna
v
(twojalist
) jest zadeklarowana poza pętlą. Zatem zgodnie z regułami przechwyconych zmiennych, wszystkie iteracje listy będą miały wspólny uchwyt przechwyconej zmiennej.Począwszy od C # 5, zostało to zmienione: zmienna iteracji (
v
) jest objęta zakresem wewnątrz pętli. Nie mam odniesienia do specyfikacji, ale w zasadzie wygląda to następująco:Re anulowanie subskrypcji; jeśli chcesz aktywnie anulować subskrypcję anonimowego programu obsługi, sztuczka polega na przechwyceniu samego modułu obsługi:
Podobnie, jeśli potrzebujesz jednorazowej obsługi zdarzeń (na przykład Load itp.):
To jest teraz samodzielna rezygnacja z subskrypcji ;-p
źródło
for
jest niezmieniony w wersji 5.0