Hinzufügen eines Ereignisses zu einer Klassenmethode?

Question

OK, ich habe es versucht, aber ich verstehe es einfach nicht.

Ich habe zwei Klassen logger und class1.

Ich habe eine Methode namens logger.Write(string) und Methode namens class1.Execute().

Jetzt in meiner Anwendung möchte ich logger.Write(class1.ToString()) ausgeführt haben, wenn class1.Execute() aufgerufen wird.

Ich nehme an, dass Sie Delegierte und Ereignisse hinzufügen müssen, aber ich kann es einfach nicht verstehen, ich habe es ein paar Stunden gekratzt.

Eine Sache ist, dass der Logger und die Klasse in verschiedenen Namespaces sind und ich möchte den Klassencode für beide nicht aktualisieren, wenn möglich.

Answer 1

Nun, Sie können es sicherlich nicht tun, ohne Code in beiden Klassen zu ändern (vorausgesetzt, Sie wollen auch nicht überall, dass class1.Execute ruft) - zumindest nicht ohne einige tiefe Code-Weaving/Instrumentierung Magie. Sie können jedoch ziemlich leicht ein Ereignis in Class1 hinzufügen:

public class Class1 
{ 
    // TODO: Think of a better name :) 
    public event EventHandler ExecuteCalled = delegate {}; 

    public void Execute() 
    { 
     ExecuteCalled(this, EventArgs.Empty); 
     // Do your normal stuff 
    } 
} 

Das delegate{} Bit nur um sicher zu machen, ist, dass es immer mindestens ein no-op-Ereignishandler registriert - es bedeutet, dass Sie nicht zu überprüfen, müssen für Nichtigkeit.

Sie würden dann durch Schreiben Haken:

Class1 class1 = new Class1(); 
Logger logger = new Logger(); 
class1.ExecuteCalled += (sender, args) => logger.Write(sender.ToString()); 

(Dies wird vorausgesetzt, Sie C# 3, so haben Sie Lambda-Ausdrücke für Sie verwenden - lassen Sie mich wissen, wenn das nicht der Fall ist.)

Wenn Class1 eine Schnittstelle implementiert (zB IFoo), könnten Sie eine Implementierung der Schnittstelle schreiben möchten, die eine andere Implementierung wickelt und protokolliert kurz vor jedem Aufruf:

public sealed class LoggingFoo : IFoo 
{ 
    private readonly IFoo original; 
    private readonly IFoo logger; 

    public LoggingFoo(IFoo original, Logger logger) 
    { 
     // TODO: Check arguments for nullity 
     this.original = original; 
     this.logger = logger; 
    } 

    // Implement IFoo 
    public void Execute() 
    { 
     logger.Write("Calling Execute on {0}", original); 
     original.Execute(); 
    } 
} 

Dann verwenden Sie einfach diesen Wrapper um eine "echte" Implementierung, wo immer Sie gerade die Implementierung verwenden.

Answer 2

Können Sie einen Objektparameter für den Logger übergeben und dann einfach den ToString aufrufen? Die richtige ToString-Methode wird aufgerufen. Wenn Sie in Logger oder Klasse1 nichts ändern möchten, können Sie eine Erweiterungsmethode schreiben und diese aufrufen, anstatt class1.Execute aufzurufen. Diese Methode würde den Aufruf zum Logger und dann den Aufruf von class1.Execute machen.

public static ExecuteAndLog(this class1 obj) 
{ 
    logger.Write(obj.ToString()); 
    obj.Execute(); 
} 

Und dann würden Sie rufen Sie einfach obj.ExecuteAndLog();

Answer 3

Sie benötigen ein EventHandler für Class1

public event EventHandler OnExecute; 

und in der Execute-Methode zu deklarieren:

public void Execute() 
{ 
    //... 
    if (OnExecute != null) 
     OnExecute(this, null); 
} 

Und dann, wenn Sie class1 an anderer Stelle verwenden, das ist, wo Sie Ihre Veranstaltung setzen;

private Class1 class1 = new Class1(); 
class1.OnExecute += SomeMethodName; 

public void SomeMethodName(sender obj, EventArgs e) 
{ 
    logger.Write(class1.ToString()); 
} 

Wir können Gewohnheit EventHandlers für machen, wenn Sie mehr Informationen wünschen, aber für Barebones parameterlos Ereignisse sollte diese Arbeit.