.NET 捕获一般异常

Question

.NET 编程指南声明我们不应捕获一般异常。由于一般的异常类型捕获，我假设以下代码不是很好：

    private object CreateObject(string classname)
    {
        object obj = null;
        if (!string.IsNullOrEmpty(classname))
        {
           try
           {
              System.Type oType = System.Type.GetTypeFromProgID(customClass);
              obj = System.Activator.CreateInstance(oType);
           }
           catch (Exception ex)
           {
               Log.Error("Unable to create instance for COM Object with class name " + classname + "\n" + ex.Message);
           }
        }
        return obj;
    }

在下面的代码中，我捕获了特定的异常，但不是全部，然后我重新抛出异常，以防与非泛型异常不同。然而，函数“CreateInstance”可以抛出许多异常（ArgumentNullException、ArgumentException、NotSupportedException、TargetInvocationException、MethodAccessException、MemberAccessException、InvalidComObjectException、MissingMethodException、COMException、TypeLoadException）。

是否可以捕获所有其他单个异常？还是有更好的办法？

    private object CreateObject(string classname)
    {
        object obj = null;
        if (!string.IsNullOrEmpty(classname))
        {
           try
           {
              System.Type oType = System.Type.GetTypeFromProgID(customClass);
              obj = System.Activator.CreateInstance(oType);
           }
           catch (NotSupportedException ex)
           {
              Log.Error("...." + ex.Message);
           }
           catch (TargetInvocationException ex)
           {
              Log.Error("...." + ex.Message);
           }
           catch (COMException ex)
           {
              Log.Error("...." + ex.Message);
           }
           catch (TypeLoadException ex)
           {
              Log.Error("...." + ex.Message);
           }
           catch (InvalidComObjectException ex)
           {
              Log.Error("...." + ex.Message);
           }
           catch (Exception ex)
           {
               Log.Error("Unable to create instance for COM Object with class name " + classname + "\n" + ex.Message);
               throw;
           }
        }
        return obj;
    }

Answer 1

作为一般规则，您不应捕获异常，除非：

1. 您有一个可以处理和解决的特定异常。但是，在这种情况下，您应该始终检查您是否应该首先考虑并避免异常。

2. 您处于应用程序的顶层（例如 UI），不希望将默认行为呈现给用户。例如，您可能想要一个带有“请将您的日志发送给我们”样式消息的错误对话框。

3. 您在以某种方式处理异常后重新抛出异常，例如，如果您回滚数据库事务。

在这个例子中，你为什么要捕捉所有这些不同的类型？在我看来，您的代码可以是：

try
{
    System.Type oType = System.Type.GetTypeFromProgID(customClass);
    return System.Activator.CreateInstance(oType);
}
catch (Exception ex)
{
    Log.Error("...." + ex.Message);

    //the generic catch is always fine if you then do this:
    throw;
}

所以你的问题是规则 (3) 的一个例子 - 你想记录一个异常，然后继续并抛出它。

所有不同的类型都在那里，因此在某些情况下你知道你可以处理（即案例 1）。例如，假设您知道有一个围绕 COMException 工作的非托管调用 - 那么您的代码变为：

try
{
    System.Type oType = System.Type.GetTypeFromProgID(customClass);
    return System.Activator.CreateInstance(oType);
}
catch (COMException cex)
{   //deal with special case:
    return LoadUnmanaged();
}
catch (Exception ex)
{
    Log.Error("...." + ex.Message);

    //the generic catch is always fine if you then do this:
    throw;
}

Answer 2

在框架的 .NET 2+ 中捕获一般异常是完全可以接受的。

-- 编辑

您不这样做的唯一原因是，您是否可以通过不同的例外来做不同的事情。如果您打算以同样的方式处理它们，只需抓住一般的（或您所追求的特定的，然后让其他任何东西上升）。

Answer 3

我认为捕获所有异常以拦截问题并向用户显示友好消息而不是一些可怕的堆栈输出是可以的。

只要您不只是吞下异常，而是记录它们并在后台对它们做出适当的反应。

Answer 4

如果你想对不同类型的异常做一些特殊的事情，那么在一个单独的 catch 块中捕获它。否则单独记录是没有意义的。

Answer 5

我们不应该使用一般异常并没有严格的规则，但是指导方针说，每当我们可以选择处理特定类型的异常时，我们不应该使用一般异常..

如果我们确定所有异常都将以相同的方式处理，则使用泛型异常，否则针对每个特定异常，而泛型应在某些未知异常的最后出现..

有时在您的应用程序中发生任何由于特定异常处理而未在代码中处理的异常，那么您的应用程序可能会崩溃。..

所以更好的方法是处理所有特定的异常，然后给通用异常一个角落，这样应用程序就可以保持稳定而不会崩溃..

并且可以在某处报告或记录那些不需要的通用异常，以便将来对应用程序进行版本改进..

Answer 6

在处理错误时习惯性地捕捉基址Exception 被认为是一种不好的做法，因为这表明您可能对实际处理的内容缺乏了解。当你看到一段代码 catch Exception 时，它读到的是，“如果这里有任何问题，请执行此操作”，其中任何内容都可以从 NullReferenceException 到 OutOfMemoryException。

以同样的方式处理所有错误的危险在于，这意味着您不关心错误的严重程度或您可能如何解决错误。 99% 的情况下，当我看到代码 catch(Exception ex) 时，紧随其后的是吞下异常的代码，无法说明语句失败的实际原因。呃。

您的错误记录示例显示了使用基础Exception 的正确方法，在您真正希望以相同方式处理所有异常的情况下，通常在应用程序的顶层以防止应用程序终止于丑陋的烂摊子。

Answer 7

我同意基思的回答。这两个代码示例的真正问题是它们可以返回 null。您有一个名为 CreateObject 的方法，它返回一个对象类型的实例。该方法由于某种原因失败，例如 COMException。然后，两个代码示例都将返回 null。调用代码现在负责检查结果与 null，否则将抛出 NullReferenceException。

引用Framework Design Guidelines 2nd ed.:

如果会员不能成功 它的设计目的是什么，它应该 被视为执行失败， 并且应该抛出异常。

它并没有像它的名字所暗示的那样做。扔！

Keith 的代码很好；记录异常并重新抛出就可以了。

这是一种私有方法，因此可以说设计指南不适用。不过，我仍然会在此处应用它们 - 当您可以使用异常处理时，为什么要使用“if (result == null)”来乱扔代码？