捕获System.Exception总是一个坏习惯吗？

Question

请考虑以下代码,它会抛出三个不同的异常(即System.Configuration.ConfigurationErrorsException,System.FormatException和System.OverflowException):

int SomeInt = Convert.ToInt32(ConfigurationManager.AppSettings["SomeIntValue"]);

异常是不同的,因此在实践中我应该有三个不同的catch块来处理每个特定的异常.但是,在这种特殊情况下,所有异常都以相同的方式处理:将日志写入,例如,EventViewer,并显示一条消息,通知配置错误......在这个特定的原因中,使用它太糟糕了

try
{
    int SomeInt = ConfigurationManager.AppSettings["SomeIntValue"];
}
catch (Exception ThisException)
{
    /* Log and display error message. */
}

或者我应该使用这三个catch块并在每个块中重复代码？

Answer 1

有关此问题的讨论,请参阅C#异常处理漏洞.

简而言之,如果你捕获一般的异常,你应该检查它是否是预期的类型之一,如果没有重新抛出它.

更新(并且有点超出范围)

还有几次我认为做一个全部捕获是有效的.这是非常罕见的,但有时在web服务中,或者如果你在asp.net中的后台线程上执行某些操作,并且异常会重新启动整个应用程序,如果你依赖于它,人们可能会丢失会话.

Answer 2

我不认为这是不好的做法。如果您想要的功能是“每当此代码引发异常时，就采取这些操作”，那么我认为捕获 System.Exception 是非常合适的。

在我看来，您正在包装一个非常具体的框架函数而不是一大块自定义代码这一事实也有所帮助。