是否有任何理由在提升之前将事件分配给局部变量？

Question

我经常看到如下代码,并想知道是否有任何理由为事件使用局部变量,而不仅仅是使用事件本身.在那儿？

var handler = OnQueryComplete;
if (handler != null)
    handler(this, new RepositoryEventArgs<T>(results));

Answer 1

是的,绝对 - 它使无效检查安全.

如果你有:

// Bad code, do not use
if (OnQueryComplete != null)
{
    OnQueryComplete(this, ...);
}

然后最后一个订阅者可以在支票和调用之间取消订阅,从而导致NullReferenceException.

这里有很多选择:

• 决定你不关心线程安全性,甚至是它抛出异常的程度.(在许多情况下,这可能是合理的,特别是如果这是您自己的代码库.)
• 使用扩展方法使这很简单,无论如何无效检查都会消失.

• 在C#6中,使用条件null运算符:

  OnQueryComplete?.Invoke(this, ...);
  

• 使用一个永不删除的空处理程序设置事件:

  public event FooEventHander OnQueryComplete = delegate {};
  

您可能还希望使用Interlocked以确保您获得变量的最新值,以避免内存模型问题.有关此问题的更多讨论,请参阅我的博客文章(包括评论).