I gestori di eventi non sono thread safe?

Question

Così ho letto in giro che invece di chiamare un evento direttamente con

if (SomeEvent != null) 
    SomeEvent(this, null); 

dovrei fare

SomeEventHandler temp = SomeEvent; 
if (temp != null) 
    temp(this, null); 

Perché è così? In che modo la seconda versione diventa sicura? Qual è la migliore pratica?

Answer 1

Gli eventi sono davvero zucchero sintattico su un elenco di delegati. Quando invochi l'evento, questo sta davvero iterando su quell'elenco e invoca ciascun delegato con i parametri che hai passato.

Il problema con i thread è che potrebbero aggiungere o rimuovere elementi da questa raccolta sottoscrivendo/annullando l'iscrizione. Se lo fanno mentre stai iterando la raccolta ciò causerà problemi (penso che venga generata un'eccezione)

L'intento è quello di copiare l'elenco prima di iterarlo, in modo da essere protetto dalle modifiche alla lista.

Nota: Ora è possibile invocare l'ascoltatore anche dopo aver annullato l'iscrizione, pertanto è necessario accertarsi di gestirlo nel codice del listener.

Answer 2

Le migliori pratiche sono la seconda forma. Il motivo è che un altro thread potrebbe annullare o modificare SomeEvent tra il test "if" e il richiamo.

Answer 3

Here è una buona scrittura su eventi .NET e condizioni di gara con thread. Copre alcuni scenari comuni e contiene alcuni buoni riferimenti.

Spero che questo aiuti.

Answer 4

IMO, le altre risposte mancano un dettaglio chiave - che delegati (e quindi eventi) sono immutabili. L'importanza di ciò è che l'iscrizione o l'annullamento dell'iscrizione di un gestore di eventi non è semplicemente aggiungi/rimuovi a un elenco, ma sostituisce sostituisce l'elenco con uno nuovo con un elemento aggiuntivo (o uno in meno).

Dal riferimenti sono atomiche, questo significa che al punto che fai:

var handler = SomeEvent; 

ora avete una rigida un'istanza che non può cambiare, anche se nella prossima picosecondo un altro unsubscribes filo (causando la campo effettivo per diventare null).

Così testate per null e invocatelo, e tutto va bene. Si noti che c'è ancora lo scenario confuso dell'evento è sollevato su un oggetto che lo ritiene non sottoscritto un picosecondo fa!