是否有任何理由在引发事件之前将事件分配给局部变量？答案

【问题标题】：Is there any reason to assign an event to a local variable before raising it?是否有任何理由在引发事件之前将事件分配给局部变量？
【发布时间】：2015-02-22 16:35:51
【问题描述】：

我经常看到如下代码，想知道是否有任何理由为事件使用局部变量，而不仅仅是使用事件本身。有吗？

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

【问题讨论】：

Raise event thread safely - best practice 的可能重复项

标签： c# .net events

【解决方案1】：

是的，绝对 - 它使无效检查安全。

如果你有：

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

那么最后一个订阅者可能会在检查和调用之间取消订阅，从而导致NullReferenceException。

这里有很多个选项：

决定不关心线程安全，即使它会引发异常。（在许多情况下，这可能是合理的，特别是如果这是您自己的代码库。）
使用扩展方法使实现这一点变得简单，这样无效性检查就会消失。
在 C# 6 中，使用条件 null 运算符：
```
OnQueryComplete?.Invoke(this, ...);
```
使用一个永远不会删除的空处理程序设置事件：
```
public event FooEventHander OnQueryComplete = delegate {};
```

您可能还想使用Interlocked 来确保您获得了变量的最新值，以避免内存模型问题。有关此问题的更多讨论，请参阅 my blog post (including comments)。

【讨论】：

@RuneFS：嗯，它稍微改变了一下。基本上总是有围绕这个的竞争条件，并且没有完美的解决方案。
感谢乔恩如此精明的回答。我刚刚习惯了delegate { }，但之前使用过你的代码示例，很幸运地忽略了任何线程安全。
是的，是的，它们将永远是竞争条件，原始评论更多是因为据说“分配给本地”通常会消除竞争条件。它确实删除了一个关键的，因为应用程序可能会崩溃，但它会创建另一个在解释“分配给本地”方法时似乎“总是”被遗忘