扩展方法线程安全吗？

Question

这个扩展方法线程安全吗？

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

或者我需要将其更改为这个？

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

Answer 1

你发现了一个有趣的漏洞，它把每个人都绊倒了。不，它不是线程安全的。

虽然它看起来像EventHandler<>引用是通过方法参数复制的，这不是运行时发生的情况。扩展方法需要内联，就像常规实例方法一样。事实上，由于它太小，它极可能会被内联。没有副本，必须自己制作一份。

Answer 2

这两个版本都不是线程安全的，具体取决于“线程安全”的含义。考虑您的第二个版本：

   var h = handler;          
   if (h!= null) 
       h(sender, args);

“处理程序”是某个字段的副本，其中包含不可变的委托。假设在空检查后该字段在另一个线程上突变为“空”。在这种情况下，您的代码不会崩溃，因为您已经复制了原始非空值。但仅仅不崩溃并不能使程序线程安全。一个没有崩溃但仍然产生错误结果的程序仍然不是线程安全的。

假设当另一个线程将事件字段设置为空时，它也改变了先前内容正确运行所需的某些状态。现在，您将运行一个事件处理程序，该处理程序取决于刚刚在另一个线程上发生变化的状态；您正在运行一个陈旧事件处理程序。

没有简单的方法可以防止这个问题；如果这就是您所处的情况，那么您将必须非常仔细地设计线程逻辑才能处理这种情况。