リフレクションによるイベントハンドラの追加と削除c＃

Question

おはよう！ 私の目的は、オブジェクトの購読や購読をイベントから（やりとりする）ことができるクラスを実装することです。ここに私のクラスのコードです。

public static class EventSubscriber 
{ 
    public static void AddEventHandler(EventInfo eventInfo, object item, Action action) 
    { 
     var parameters = GetParameters(eventInfo); 
     var handler = GetHandler(eventInfo, action, parameters); 
     eventInfo.AddEventHandler(item, handler); 
    } 

    public static void RemoveEventHandler(EventInfo eventInfo, 
         object item, Action action) 
    { 
     var parameters = GetParameters(eventInfo); 
     var handler = GetHandler(eventInfo, action, parameters); 
     eventInfo.RemoveEventHandler(item, handler); 
    } 

    private static ParameterExpression[] GetParameters(EventInfo eventInfo) 
    { 
     return eventInfo.EventHandlerType 
      .GetMethod("Invoke") 
      .GetParameters() 
      .Select(parameter => Expression.Parameter(parameter.ParameterType)) 
      .ToArray(); 
    } 

    private static Delegate GetHandler(EventInfo eventInfo, 
       Action action, ParameterExpression[] parameters) 
    { 
     return Expression.Lambda(
      eventInfo.EventHandlerType, 
      Expression.Call(Expression.Constant(action), 
         "Invoke", Type.EmptyTypes), parameters) 
      .Compile(); 
    } 
} 

ご覧のとおり、オブジェクトを実際に購読したり購読を解除したりする2つのパブリックメソッドがあります。そして、ここで私は、サンプルの

class Program 
{ 
    static void Main() 
    { 
     Test test = new Test(); 
     test.SubscribeTimer(); 
     while (true) 
     { 
      if(test.a == 10) 
      { 
       break; 
      } 
     } 
     test.UnsubscribeTimer(); 
     while (true) 
     { 

     } 
    } 
} 

class Test 
{ 
    System.Timers.Timer timer; 
    public int a = 0; 

    public Test() 
    { 
     timer = new System.Timers.Timer(1000); 
     timer.Start(); 
    } 

    public void SubscribeTimer() 
    { 
     var eventInfo = typeof(System.Timers.Timer).GetEvent("Elapsed"); 
     EventSubscriber.AddEventHandler(eventInfo, timer, TimerElapsed); 
     EventSubscriber.RemoveEventHandler(eventInfo, timer, TimerNotElapsed); 
    } 

    public void UnsubscribeTimer() 
    { 
     var eventInfo = typeof(System.Timers.Timer).GetEvent("Elapsed"); 
     EventSubscriber.AddEventHandler(eventInfo, timer, TimerNotElapsed); 
     EventSubscriber.RemoveEventHandler(eventInfo, timer, TimerElapsed); 
    } 

    public void TimerElapsed() 
    { 
     Console.WriteLine("timer elapsed"); 
     a++; 
    } 

    public void TimerNotElapsed() 
    { 
     Console.WriteLine("timer not elapsed"); 
     a++; 
    } 
} 

予想される動作は、初めに、我々はメッセージ10番目秒後に我々は唯一の「しないタイマを参照する必要があり、毎秒「のタイマーが経過する」を参照してくださいということですそれをテストする方法サンプルです私たちはやっていますが、まだ "タイマー経過"を見ています。つまり、AddEventHandlerメソッドは機能しますが、RemoveEventHandlerメソッドは機能しません。

あなたが私を助けるならば、私はとても幸せです。前もって感謝します。

Answer 1

は、私はあなたが新しいハンドラを毎回作成しているからだと思う：（前のハンドラと一致していないので、呼び出しリストから削除することはできません）

public static void RemoveEventHandler(EventInfo eventInfo, 
        object item, Action action) 
{ 
    var parameters = GetParameters(eventInfo); 
    var handler = GetHandler(eventInfo, action, parameters); // <-- 
    eventInfo.RemoveEventHandler(item, handler); 
} 

はなぜあなたをしていますActionをラップしますか？パラメータを失う？パラメータのためにeventInfo.RemoveEventHandler(item, action);を追加/削除することはできません。新しく生成されたハンドラを削除する場合は、ハンドラを削除するときにそのハンドラを返す必要があります。削除するイベントハンドラのために

public static Delegate AddEventHandler(EventInfo eventInfo, object item, Action action) 
{ 
    var parameters = GetParameters(eventInfo); 
    var handler = GetHandler(eventInfo, action, parameters); 
    eventInfo.AddEventHandler(item, handler); 
    return handler; 
} 

public static void RemoveEventHandler(EventInfo eventInfo, 
        object item, Delegate handler) 
{ 
    eventInfo.RemoveEventHandler(item, handler); 
} 

---

var handler = EventSubscriber.AddEventHandler(eventInfo, timer, TimerElapsed); 

EventSubscriber.RemoveEventHandler(eventInfo, timer, handler);