Czy metoda rozszerzenia jest bezpieczna?

Question

Czy ta metoda rozszerzenia jest bezpieczna?

public static class Extensions 
    { 
     public static void Raise<T>(this EventHandler<T> handler, 
     object sender, T args) where T : EventArgs 
     { 
     if (handler != null) handler(sender, args); 
     } 
    } 

Czy muszę to zmienić?

public static class Extensions 
    { 
     public static void Raise<T>(this EventHandler<T> handler, 
     object sender, T args) where T : EventArgs 
     { 
     var h = handler; 
     if (h!= null) h(sender, args); 
     } 
    } 

Answer 1

Znalazłeś interesującą pętlę, która potknęła wszystkich. Nie, to nie jest bezpieczne dla wątków.

Chociaż jest to wygląda podobnie jak EventHandler <> odwołanie jest kopiowane za pośrednictwem argumentu method, to nie jest to, co dzieje się w czasie wykonywania. Metody rozszerzeń mogą podlegać inkrementacji, podobnie jak w przypadku zwykłej metody instancji. W rzeczywistości jest bardzo prawdopodobne, że zostanie zaindeksowane, ponieważ jest tak małe. Nie ma kopii, sam musisz ją wykonać.

Answer 2

Żadna z wersji nie jest bezpieczna dla wątków, w zależności od tego, co rozumie się przez "Threadafe". Zastanów się nad drugą wersją:

var h = handler;   
    if (h!= null) 
     h(sender, args); 

"handler" to kopia pola, które ma w sobie niezmiennego delegata. Załóżmy, że pole zostało zmutowane do "null" w innym wątku po sprawdzeniu zerowym. Twój kod nie ulega awarii w tym przypadku, ponieważ wykonałeś kopię oryginalnej wartości niezerowej. Lecz tylko nie rozbicie się na nie powoduje, że program jest bezpieczny. Program, który nie ulega awarii, ale nadal generuje złe wyniki, nadal nie jest bezpieczny dla wątków.

Załóżmy, że gdy drugi wątek ustawił wartość pola zdarzenia na wartość null, zmutowano także stan, w którym poprzednia zawartość wymagała poprawnego działania. Teraz uruchomisz program obsługi zdarzeń, który zależy od stanu, który został zmutowany w innym wątku; używasz obsługi zdarzeń przestarzały.

Nie ma łatwego sposobu ochrony przed tym problemem; jeśli to jest sytuacja, w której się znajdujesz, będziesz musiał zaprojektować swoją logikę gwintowania bardzo ostrożnie, aby poradzić sobie z sytuacją.