捕获非特定异常（例如 System.Exception）是一种不好的做法吗？为什么？

我目前正在进行代码审查，下面的代码让我跳了起来。我发现这段代码存在多个问题。你是否同意我的观点？如果是这样，我该如何向我的同事解释这是错误的（顽固型......）？

• 捕获一般异常（Exception ex）
• 使用“if（ex 是某物）”而不是另一个 catch 块
• 我们吃SoapException、HttpException和WebException。但如果 Web 服务失败，就没有什么可做的了。

Code:

try
{
    // Call to a WebService
}
catch (Exception ex)
{
    if (ex is SoapException || ex is HttpException || ex is WebException)
    {
        // Log Error and eat it.
    }
    else
    {
        throw;
    }
}

咒语是：

• 你应该只在以下情况下捕获异常 你可以妥善处理它们

Thus:

• 你不应该抓住将军 例外情况。

就你而言，是的，你应该抓住这些异常并做一些有用的事情（可能不仅仅是吃掉它们——你可以throw记录后）。

您的编码器正在使用throw (not throw ex) 这是good.

这是捕获多个特定异常的方法：

try
{
    // Call to a WebService
}
catch (SoapException ex)
{
    // Log Error and eat it
}
catch (HttpException ex)
{
    // Log Error and eat it
}
catch (WebException ex)
{
    // Log Error and eat it
}

这与您的代码的作用几乎相同。您的开发人员可能这样做是为了避免重复“记录错误并吃掉它”块。