异常处理

Question

catch (Exception ex)
{
    DBGlobals.Error("OHMjson.Graph.saveLastGraphName - Error: " + ex.getMessage());
    msg = "Unable to save data";
    status = false;
}

如果我遇到一个错误，这段代码会抛出一个状态为false。

相反，我应该抛出新的异常。这条路对吗？如何以更好的方式处理异常。

Answer 1

首先，不要抓到Exception。捕获特定的子类。

其次，是的，这可能是一个底层的IOException，就像这样，应该包装在一个特定于应用程序的异常中。你可以这样做，例如：

final class DBGlobalsException extends Exception
{
   Field somethingDBGlobalsSpecific;

   //where this is shorthand for chained constructors that look like Exception
   DBGlobalsException(String message, Throwable cause)
   {
     super(message,cause);
   }

   // elaborate your exception with whatever error state you want to propagate outwards
   void setField(Field ...) {}
}

然后你就可以

catch (IOException ex) {
   DBGlobals.Error("OHMjson.Graph.saveLastGraphName - Error: " + ex.getMessage());
   msg = "Unable to save data";
   status = false;
   DBGlobalsException dbe = new DBGlobalsException(msg,ex);
   dbe.setField(status /* or whatever */);
   throw dbe;
}

另请参阅：Throwing Exception、Throwing new and old exceptions、Rolling your own。这只是一小部分，有一堆关于异常处理、链接等的优秀问答。在WikiBooks上概述了一些有用的最佳实践。

也请阅读Oded's answer并仔细思考……您的异常应该封装任何状态(错误代码、标志等)。是外部代码从条件中恢复所必需的--或者它应该是一个不可恢复的条件(请参阅RuntimeException)。

最后，介绍规范有效的Java参考：Item 43: Throw Exceptions Appropriate to the Abstraction。

Answer 2

更喜欢抛出异常而不是返回错误代码/状态。

返回错误代码意味着调用者应该始终记得检查它。抛出异常允许调用代码决定要做什么(通常情况下，决定越高，做出的决定越好)。