C# Обработка событий с использованием подписчика издателя не работает

Question

У меня есть класс издателя и класс подписчика. Класс publisher имеет событие, и подписчик хочет выполнить один из своих методов, когда это событие будет поднято.

Вот мой издатель класс с событием:

public delegate EventHandler<MyEventArgs> MyEventHandler(object sender, MyEventArgs args); 
public class MyEventArgs : EventArgs 
{ 
    public string Content { set; get; } 
} 

public class Publisher 
{ 
    public event MyEventHandler MyCustomEvent; 

    public void TriggerEvent() 
    { 
     if(MyCustomEvent!=null) 
      MyCustomEvent(this, new MyEventArgs{ Content = "Geeee! This isn't working!" }); 
    } 
} 

А вот абонент:

class Subscriber 
{ 
    static void Main(string[] args) 
    { 
     Publisher publisher = new Publisher(); 

     //hook the event to a method 
     publisher.MyCustomEvent += (s, e) => delegate 
     { 
      PrintThis(e.Content); 
     }; 

     //Do something to trigger the event  
     publisher.TriggerEvent(); 
    } 
    static public void PrintThis(string content) 
    { 
     Console.Write(content); 
    } 
} 

Издатель не должен быть в курсе абонента. Я хочу, чтобы подписчик мог выполнять свою функцию PrintThis, когда издатель вызывает событие. Но это не работает. Что я здесь делаю неправильно?

Answer 1

Вы не определили обработчик событий.

Делают это так:

public delegate void MyEventHandler(object sender, MyEventArgs args); 

Обратите внимание, что тип возвращаемого значения void.

А потом сам обработчик события должен быть:

publisher.MyCustomEvent += (s, e) => 
    { 
     PrintThis(e.Content); 
    }; 

Кроме того, ваш метод TriggerEvent безопаснее, если они определены таким образом:

public void TriggerEvent() 
{ 
    var mce = MyCustomEvent; 
    if (mce!=null) 
    { 
     mce(this, new MyEventArgs{ Content = "Geeee! This isn't working!" }); 
    } 
} 

В многопоточной среде MyCustomEvent обработчик может переключаться между null проверить и вызвать.