小编典典

安全地引发事件线程-最佳实践

c#

为了引发事件,我们使用如下方法OnEventName:

protected virtual void OnSomethingHappened(EventArgs e) 
{
    EventHandler handler = SomethingHappened;
    if (handler != null) 
    {
        handler(this, e);
    }
}

但是,这有什么区别?

protected virtual void OnSomethingHappened(EventArgs e) 
{
    if (SomethingHappened!= null) 
    {
        SomethingHappened(this, e);
    }
}

显然第一个是线程安全的,但是为什么以及如何执行?

不必启动新线程吗?


阅读 279

收藏
2020-05-19

共1个答案

小编典典

有一个微小的机会SomethingHappened成为null空校验之后,但在调用之前。但是,MulticastDelagates是不可变的,因此,如果您首先分配一个变量,对该变量进行null检查并通过它进行调用,那么在这种情况下是安全的(自我插入:我之前写过一篇有关此问题博客文章)。

但是硬币的背面。如果您使用temp变量方法,那么您的代码将受到NullReferenceExceptions的保护,但可能是该事件将在事件监听器
与事件分离后 调用事件监听器。这只是以最优雅的方式处理的事情。

为了解决这个问题,我有一个扩展方法,我有时会使用:

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

使用该方法,您可以像这样调用事件:

protected void OnSomeEvent(EventArgs e)
{
    SomeEvent.SafeInvoke(this, e);
}
2020-05-19