Добавить событие в метод класса?

Question

ОК, я пробовал, но я просто не понимаю.

У меня есть два класса logger и class1.

У меня есть метод под названием logger.Write(string) и метод под названием class1.Execute().

Теперь в моем приложении я хочу, чтобы logger.Write(class1.ToString()) работал, когда вызывается class1.Execute().

Я предполагаю, что вам нужно добавить делегатов и событий, но я просто не могу об этом разузнать, царапая его пару часов.

Одно дело, что регистратор и класс находятся в разных пространствах имен, и я не хочу обновлять код класса для любого из них, если это возможно.

Answer 1

Ну, вы, конечно же, не можете это сделать, не меняя код в любом классе (предполагая, что вы также не хотите менять везде, что вызывает class1.Execute) - по крайней мере, не без какой-либо глубокой маскировки кода/инструментария. Тем не менее, вы можете довольно легко добавить событие в Class1:

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 
    } 
} 

delegate{} бит просто чтобы убедиться, что всегда есть по крайней мере, не оп зарегистрированный обработчик события - это не означает, что вам не нужно, чтобы проверить недействительность.

Вы бы затем подключить его в письменной форме:

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

(Предполагается, что вы используете C# 3, так что вы имеете лямбда-выражения доступны для вас - дайте мне знать, если это не так).

Если Class1 реализует интерфейс (скажем IFoo), вы можете захотеть написать реализацию интерфейса, который оборачивает другой реализации, и только регистрирует перед каждым вызовом:

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(); 
    } 
} 

Затем просто используйте эту оболочку вокруг «реальной» реализации везде, где вы в настоящее время просто используете реализацию.

Answer 2

Можете ли вы передать параметр объекта для регистратора, а затем просто вызвать ToString? Будет вызываться правильный метод ToString. Если вы не хотите ничего менять в журнале или классе1, вы можете написать метод расширения и вызвать это вместо вызова класса1.Execute. Этот метод вызовет вызов регистратора, а затем вызов класса1.Execute.

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

И тогда вы просто вызываете obj.ExecuteAndLog();

Answer 3

Вам нужно объявить EventHandler для Class1

public event EventHandler OnExecute; 

и в вашем выполнения метода:

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

И тогда, когда вы используете Class1 в другом месте, то где вы положили ваше мероприятие;

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

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

Мы можем сделать заказ EventHandlers, если вы хотите получить больше информации, но для баребонов событий без параметров это должно работать.