如何安全地触发事件

Question

当事件没有订阅者时，如何确保在触发事件时不会引发异常。

 // Delegate declaration
 public delegate void _delDisplayChange(object sender,string option);

 // Event declaration
 public event _delDisplayChange  DisplayChange;

 //throwing the event
 DisplayChange(this, "DISTRIBUTION");

Answer 1

这是推荐的方法：

protected void RaiseDisplayChanged(string message)
{
    var handlers = DisplayChange;
    if(handlers != null)
        handlers(this, message);
}

在检查之前复制事件处理程序枚举有两件事：

1. 如果 DisplayChange 处理程序在检查和触发之间变为 null，您不会死
2. 如果侦听器在枚举时修改 DisplayChange 列表，您不会遇到奇怪的事情。

另外，您没有使用标准事件协议。您的代表应该是：

public delegate void DisplayChangeDelegate(object sender, OptionsEventArgs args);

其中 OptionsEventArgs 派生自 EventArgs。更进一步，在 .Net 3.5 中，您永远不应该像这样定义委托。相反，您应该只定义您的事件：

public event EventHandler<OptionsEventArgs> DisplayChanged;

我想通过定义这个类更进一步：

public class EventArgs<T> : EventArgs
{
    public T Payload { get; private set }
    public EventArgs(T payload)
    {
        Payload = payload;
    }
}

那么，你就不需要定义 OptionsEventArgs：

public event EventHandler<EventArgs<string>> DisplayChanged;

只是一些需要考虑的事情......

Answer 2

改变这个：

// Event declaration      
public event _delDisplayChange  DisplayChange;

到这里：

// Event declaration      
public event _delDisplayChange  DisplayChange = delegate{};

这将确保您的活动始终拥有至少一名订阅者。

Answer 3

正如布赖恩所说：许多消息来源建议在检查事件是否为空之前制作一个副本：

_delDisplayChange displayChangeCopy = DisplayChange;
if (displayChangeCopy != null)
    displayChangeCopy(this, "DISTRIBUTION");

这有助于使您的代码更加线程安全，因为 displayChangeCopy 的值在 null 检查和调用之间不会改变。