"La sottrazione del delegato ha un risultato imprevedibile" in ReSharper/C#?

Question

Quando si utilizza myDelegate -= eventHandler ReSharper (versione 6) problemi:

Delegato sottrazione ha risultato imprevedibile

Il razionale dietro questo è explained by JetBrains here. La spiegazione ha senso e, dopo averlo letto, dubito di tutti i miei usi di - sui delegati.

Come dunque,

• posso scrivere un evento non-auto senza fare ReSharper scontroso?
• o, c'è un modo migliore e/o "corretto" per implementare questo?
• oppure, posso semplicemente ignorare ReSharper? Codice

Qui è semplificato:

public delegate void MyHandler (object sender); 

MyHandler _myEvent; 

public event MyHandler MyEvent 
{ 
    add 
    { 
     _myEvent += value; 
     DoSomethingElse(); 
    } 
    remove 
    { 
     _myEvent -= value; // <-- ReSharper warning here 
    } 
} 

Answer 1

Non abbiate paura! La prima parte dell'ammonizione di ReSharper si applica solo alla rimozione di elenchi di delegati. Nel tuo codice, rimuovi sempre un singolo delegato. La seconda parte parla dell'ordine di delegati dopo la rimozione di un delegato duplicato. Un evento non garantisce un ordine di esecuzione per i suoi abbonati, quindi non ha alcun effetto su di te.

Poiché la meccanica di cui sopra può portare a risultati imprevedibili, ReSharper emette un avviso ogni volta che incontra un operatore di sottrazione delegato.

ReSharper sta emettendo questo avviso perché la sottrazione di delegati multicast può avere dei trucchi, non sta condannando completamente quella caratteristica del linguaggio. Fortunatamente quei trucchi sono nei casi marginali ed è improbabile che li incontriate se state semplicemente strumentando eventi semplici. Non esiste un modo migliore per implementare i tuoi gestori add/remove, devi solo notarlo.

Suggerire di eseguire il downgrade del livello di avviso di ReSharper per quel messaggio a "Suggerimento" in modo da non essere desensibilizzati ai propri avvisi, che di solito sono utili.

Answer 2

set a = null invece di utilizzare - =

Answer 3

Non dovrebbe usare direttamente i delegati per sommare o sottrarre. Invece il tuo campo

MyHandler _myEvent; 

Dovrebbe essere dichiarato come evento. Ciò risolverà il problema senza rischiare la soluzione e continuerà a beneficiare dell'utilizzo degli eventi.

event MyHandler _myEvent; 

L'utilizzo di somma delegato o sottrarre è pericoloso perché si può perdere gli eventi quando semplicemente assegnando il delegato (come da dichiarazione, lo sviluppatore non inferire direttamente questo è un delegato multicast come quando si è dichiarato come un evento) . Giusto per esemplificare, se la proprietà menzionata in questa domanda non è stata contrassegnata come un evento, il codice seguente farà sì che i primi due incarichi siano LOST, perché qualcuno semplicemente ha assegnato al delegato (che è anche valido!).

myObject.MyEvent += Method1; 
myObject.MyEvent += Method2; 
myObject.MyEvent = Method3; 

Quando si assegnano method3, ho completamente perso le due sottoscrizioni iniziali. L'utilizzo degli eventi eviterà questo problema e allo stesso tempo rimuoverà l'avviso di ReSharper.