c# - 在每个类的每个方法中尝试/捕获？

Question

当我们将一堆语句包装在 try/catch 中并且其中一个发出异常时，在 catch 中我们无法知道是哪个语句导致了异常(ex.stacktrace 显示了我们当前的方法(doit )、它的调用者、它的调用者的调用者等，但既不是 do1 也不是 do2):

function doit() {
   try {
     do1();
     do2();
     [...]
   }
   catch (Exception ex) {
     // what failed?
   }
}

通常我会包装所有语句并重新抛出，有点像:

private void do1() {
  try {
     // do whatever
  } catch(Exception e) {
     // write to my error log
     throw new Exception("do1: " + e.Message, e.InnerException);
  }
}

这会在我的日志中留下痕迹，并使链可用于上游。当然，问题是我必须用这种代码包装我编写的每个方法。

有些事情告诉我，我对此很愚蠢。什么是正确的方法？

Answer 1

好吧，这很难做到正确，因为异常处理是一个非常敏感的话题，过去人们为如何正确地做到这一点而进行过宗教 war 。

首先:既不使用空 catch(try { ... } catch { ... })，也不使用 catch(Exception ex)。异常派生类的唯一目的是为您提供有关发生的异常类型的丰富信息，以便您可以在异常处理程序中做一些有意义的事情(如果线程崩溃，请重新启动它，如果数据库连接非永久性失败，请重试，然后失败，等等)。

人们倾向于在他们的代码的最外层使用一个包罗万象的处理程序来记录未捕获的异常，这很好，但在任何情况下你都应该提示用户或重新抛出异常(使用 throw ，而不是 throw ex - 对此也有大量讨论)。

基本上，您根本不会以编程方式关心异常发生的位置。你要么可以处理它，要么你不能。如果你不能处理它，那么你就不会捕获它。

附录:这种“如果你能做点什么，就去做，否则你不敢碰那个异常”理念的最重要原因是默默地捕获异常(无论是否记录)会导致真正难以发现的错误。仅仅将它们推送到日志文件中可能还不够，因为在实时系统中，您可能无法获得完整注释的堆栈跟踪(包括行号和所有内容)。

附录 2:以需要整数输入的文本框为例。如果用户提供了一个你无法有意义地处理输入的字符串，可能会抛出一个转换异常，你会捕获该特定异常并将文本框重置为其旧值，并可能会通知用户错误输入。或者，您的程序可能会因异常而死(糟糕的设计，您可以从该异常中恢复)或静默地继续显示错误输入，但仍然使用旧值(糟糕的设计，程序具有误导性)。