自动创建空的C＃事件处理程序

Question

无法在C＃中触发没有附加处理程序的事件。 因此，在每次调用之前，有必要检查事件是否为null。

if ( MyEvent != null ) {
  MyEvent( param1, param2 );
}

我想保持我的代码尽可能干净，并摆脱这些空检查。 我认为这不会对性能产生太大影响，至少在我看来不会如此。

MyEvent( param1, param2 );

现在，我通过为每个事件手动添加一个空的内联处理程序来解决此问题。 这很容易出错，因为我需要记住要这样做。

void Initialize() {
  MyEvent += new MyEvent( (p1,p2) => { } );
}

有没有一种方法可以使用反射和一些CLR魔术自动为给定类的所有事件生成空处理程序？

Answer 1

从那以后，我在另一篇文章中看到了这一点，并从此无耻地偷走了它，并在我的许多代码中使用了它：

public delegate void MyClickHandler(object sender, string myValue);
public event MyClickHandler Click = delegate {}; // add empty delegate!

//Let you do this:
public void DoSomething() {
    Click(this, "foo");
}

//Instead of this:
public void DoSomething() {
    if (Click != null) // Unnecessary!
        Click(this, "foo");
}

*如果有人知道此技术的起源，请在评论中发布。 我确实相信来源会得到应有的重视。

（ 编辑：我从这篇文章中了解到了C＃的隐藏功能？ ）

Answer 2

表示法：

if ( MyEvent != null ) {
  MyEvent( param1, param2 );
}

不是线程安全的。 您应该这样进行：

EventHandler handler = this.MyEvent;
if ( null != handler ) { handler( param1, param2 ); }

我知道，这很麻烦，因此您可以使用辅助方法：

static void RaiseEvent( EventHandler handler, object sender, EventArgs e ) {
    if ( null != handler ) { handler( sender, e ); }
}

然后致电：

RaiseEvent( MyEvent, param1, param2 );

如果使用的是C＃3.0，则可以将helper方法声明为扩展方法：

static void Raise( this EventHandler handler, object sender, EventArgs e ) {
    if ( null != handler ) { handler( sender, e ); }
}

然后致电：

MyEvent.Raise( param1, param2 );

您还可以为其他事件处理程序创建下一个扩展/帮助程序方法。 例如：

static void Raise<TEventArgs>( this EventHandler<TEventArgs> handler,
    object sender, TEventArgs e ) where TEventArgs : EventArgs
{
    if ( null != handler ) { handler( sender, e ); }
}

Answer 3

在C＃6.0中，由于有条件的null运算符?. ，因此无需进行任何长度的空检查?.

该文档解释说，调用MyEvent?.Invoke(...)将事件复制到一个临时变量，执行null检查，如果不为null，则在该临时副本上调用Invoke 。 从某种意义上说，这不一定是线程安全的，因为有人可以在将副本复制到临时变量之后添加一个新事件，该事件不会被调用。 确实可以确保您不会在null上调用Invoke 。

简而言之：

public delegate void MyClickHandler(object sender, string myValue);
public event MyClickHandler Click;

public void DoSomething() {
    Click?.Invoke(this, "foo");
}

Answer 4

您可以这样写：

MyEvent += delegate { };

我不确定您要做什么是正确的。

Answer 5

您不需要针对不同事件处理程序的几种扩展方法，只需一种：

public static class EventHandlerExtensions {
  public static void Raise<T>(this EventHandler<T> handler, object sender, T args) where T : EventArgs {
    if (handler != null) handler(sender, args);
  }
}

Answer 6

这是一个坏主意，因为现在使用该事件的代码期望该事件的对象默认已通过操作编码。 如果您的代码永远不会被其他任何人使用，那么我想您可以摆脱它。

Answer 7

不幸的是，C＃事件声明包括许多众所周知的安全问题和效率低下的问题。 我在委托上设计了许多扩展方法，以安全地调用它们，并以线程安全的方式注册/注销委托 。

您的旧的引发事件的代码：

if (someDelegate != null) someDelegate(x, y, z);

您的新代码：

someDelegate.Raise(x, y, z);

您的旧活动注册代码：

event Action fooEvent;
...
lock (someDummyObject) fooEvent += newHandler;

您的新代码：

Action fooEvent;
...
Events.Add(ref fooEvent, newHandler);

无需锁定，也无需使用插入编译器的伪对象来锁定事件。

Answer 8

您可以使用PostSharp在构建时添加此魔术。 这是最好的方法。