"La sottrazione dei delegati ha un risultato imprevedibile" in ReSharper / C #?

Quando si utilizzano myDelegate -= eventHandlerproblemi con ReSharper (versione 6):

La sottrazione di delegati ha un risultato imprevedibile

Il razionale dietro questo è spiegato da JetBrains qui . La spiegazione ha senso e, dopo averla letta, dubito di tutti i miei usi -sui delegati.

Come dunque ,

• posso scrivere un evento non automatico senza rendere ReSharper scontroso?
• oppure esiste un modo migliore e / o "corretto" per implementarlo?
• oppure posso semplicemente ignorare ReSharper?

Ecco il codice semplificato:

public delegate void MyHandler (object sender);

MyHandler _myEvent;

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

Non aver paura! La prima parte dell'avviso di ReSharper si applica solo alla rimozione di elenchi di delegati. Nel codice rimuovi sempre un singolo delegato. La seconda parte parla dell'ordinamento dei delegati dopo la rimozione di un delegato duplicato. Un evento non garantisce un ordine di esecuzione per i suoi iscritti, quindi non influisce nemmeno su di te.

Poiché i meccanismi di cui sopra possono 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 del delegato multicast può avere trucchi, non sta condannando completamente quella caratteristica del linguaggio. Fortunatamente questi trucchi sono in casi marginali ed è improbabile che li incontri se stai solo strumentando eventi semplici. Non c'è modo migliore per implementare i tuoi add/ removegestori, devi solo prenderne atto.

Suggerirei di ridurre il livello di avviso di ReSharper per quel messaggio a "Suggerimento" in modo da non essere desensibilizzato ai loro avvisi, che di solito sono utili.

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

MyHandler _myEvent;

Dovrebbe invece essere dichiarato anche come evento. Questo risolverà il problema senza rischiare la tua soluzione e avrai comunque il vantaggio dell'utilizzo degli eventi.

event MyHandler _myEvent;

L'utilizzo della somma o della sottrazione del delegato è pericoloso perché è possibile perdere eventi quando si assegna semplicemente il delegato (come da dichiarazione, lo sviluppatore non dedurrà direttamente che si tratta di un delegato multicast come quando viene dichiarato come evento). Giusto per esemplificare, se la proprietà menzionata in questa domanda non è stata contrassegnata come un evento, il codice sottostante farà sì che i primi due incarichi siano PERSI, perché qualcuno semplicemente assegnato al delegato (che è anche valido!).

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

Durante l'assegnazione di Method3, ho perso completamente i due abbonamenti iniziali. L'utilizzo degli eventi eviterà questo problema e allo stesso tempo rimuoverà l'avviso ReSharper.

impostalo su = null invece di usare - =