在 C# 中引发忽略处理程序引发的异常的事件

Question

我对在 C# 中引发事件的一个最讨厌的事实是，事件处理程序中的异常会破坏我的代码，并且可能会阻止调用其他处理程序，如果破坏的处理程序碰巧首先被调用的话；在大多数情况下，我的代码不会在意监听其事件的其他人的代码是否损坏。

我创建了一个捕获异常的扩展方法：

public static void Raise(this EventHandler eh, object sender, EventArgs e)
{
  if (eh == null)
    return;
  try
  {
    eh(sender, e);
  }
  catch { }
}

尽管这确实意味着我的代码无论如何都会继续运行，但此方法不会阻止第一个事件处理程序抛出异常并阻止第二个和后续处理程序收到事件通知。我正在研究一种通过 GetInvocationList 进行迭代的方法，以将每个单独的事件处理程序包装在它自己的 try/catch 中，但这似乎效率低下，而且我不确定最好的方法，或者即使我应该这样做。

另外，我真的很不习惯在这里忽略异常（FxCop/Resharper 也不是）；实际上，在这种情况下，异常应该发生什么？

Answer 1

如果你做了这样的事情呢？

public static void Raise(this EventHandler eh, object sender, EventArgs e)
{
    if (eh == null)
        return;

    foreach(var handler in eh.GetInvocationList().Cast<EventHandler>())
    {
        try
        {
            handler(sender, e);
        }
        catch { }
    }
}

Answer 2

您应该只捕获那些您可以处理的异常。例如，您可能正在访问一个文件并且没有足够的权限。如果您知道这种情况偶尔会发生，并且只需要记录该案例，然后捕获该异常。

如果您的代码无法处理异常，请不要捕获它。

这是一种特殊情况，因此其他事件处理程序很有可能也无法“做他们的事情”，所以让它传播到可以安全处理的水平。

Answer 3

如果事件处理程序没有捕获异常，我看不出有什么好的方法可以在抛出事件的类中处理异常。您没有任何上下文可以知道处理程序在做什么。在这种情况下，让应用程序严重崩溃是最安全的，因为您不知道其他事件处理程序可能造成的可能会破坏您的应用程序稳定性的副作用。这里的教训是事件处理程序必须是健壮的（处理它们自己的异常），因为触发事件的类不应该捕获处理程序抛出的异常。

Answer 4

一个简单但重要的规则：

代码中的每个缺陷都应该是 尽早致命 可能。

通过这种方式，错误被发现并修复，软件变得越来越强大。别人说的是对的；永远不要捕捉到你没有预料到并且知道如何处理的异常。