Löschen Sie alle Event Subscriptions (Clone verlinkt)

Question

Ich habe gerade Clone von ICloneable implementiert und festgestellt, dass auch die Event-Subskriptionen von meiner Quellinstanz gefolgt sind. Gibt es eine gute Möglichkeit, all diese zu klären?

Derzeit verwende ich ein paar dieser Schleifen für jedes Ereignis, das ich alles löschen muss.

foreach (var eventhandler in OnIdChanged.GetInvocationList()) 
{ 
    OnIdChanged -= (ItemEventHandler) eventhandler; 
} 

foreach (var eventhandler in OnNameChanged.GetInvocationList()) 
{ 
    ... 

Dies funktioniert gut, aber überfüllt den Code ein wenig. Meistens besorgt um Event-Dangling.

Answer 1

Ich glaube, Sie könnten einfach OnIdChanged = null in Ihrem geklonten Objekt setzen.

Nachdem Sie den Klon erstellt haben, rufen Sie einfach die ClearEvents-Methode für den Klon auf.

public class ClonedObject 
{ 
    public event EventHandler OnIdChanged; 
    public event EventHandler OnNameChanged; 

    public void ClearEvents() 
    { 
     OnIdChanged = null; 
     OnNameChanged = null; 
    } 
} 

Answer 2

Vermutlich, wenn Sie wirklich ein Objekt klonen wollen, man wollte halten diese Ereignisabonnements.

Wenn Sie Objekte klonen, die nicht für Ereignisse abonniert sein sollten, sollten Sie Ihren Code umgestalten. Lassen Sie Ihren Controller oder ähnliche Objekte die Ereignisse mit einem Verweis auf ein dediziertes Datenobjekt abonnieren, und lassen Sie Ihre Datenobjekte diese Daten speichern, ohne auf die Ereignisse Bezug zu nehmen. Klonen Sie die Datenobjekte und fügen Sie sie bei Bedarf in geeignete Controller-Objekte ein.

Letztendlich schlage ich vor, dass Sie das Problem umgehen, indem Sie keine Ereignisse abonnieren, die Sie nicht abonnieren müssen. Betrachte den Problemraum aus einem anderen Blickwinkel.