包装已检验 Java 异常的标准方法

2022-09-03 02:55:37

我有一个相当详细的问题,关于包装检查异常的正确方法,以及番石榴的做法。(为长度道歉,但我想把我的思维过程放下来)


标准界面如下所示:Runnable

public interface Runnable
{
   public void run();
}

其中不能引发已检查的异常。run()

因此,如果我想有一个用于包装引发已检查异常的任务的哪个,并且我打算让调用来处理这些异常的东西,而不是本身,我必须将异常包装在未经检查的异常中。RunnableRunnable.run()Runnable.run()

所以有一段时间我一直在使用:

Runnable r = new Runnable {
   @Override public void run()
   {
       try {
          doNastyStuff();
       }
       catch (NastyException e)
       {
          throw new RuntimeException(e);
       }
   }      
};

然后我可以在上层处理。除了那时我想,我真正想要的是单独处理一个包装的异常,因为我知道它的语义是包装一个检查的异常,所以我写了这个帮助器类:RuntimeException

/**
 * Wrapped exception: the purpose of this is just to wrap another exception,
 * and indicate that it is a wrapped exception
 */
public class WrappedException extends RuntimeException
{
    /**
     * @param t any throwable
     */
    public WrappedException(Throwable t)
    {
        super(t);
    }
}

然后我可以这样做:

/* place that produces the exception */
...
catch (NastyException e)
{
   throw new WrappedException(e);
}

...
/* upper level code that calls Runnable.run() */
try
{
   ...
   SomeOtherNastyCode();
   r.run();
   ...
}
catch (SomeOtherNastyException e)
{
   logError(e);
}
catch (WrappedException e)
{
   logError(e.getCause());
}

它似乎工作得很好。

但是现在我在想,好吧,如果我想在库以及使用库的应用程序中使用它,现在它们都依赖于,所以它应该真正存在于一个基础库中,我可以将其包含在任何地方。WrappedException

这让我想到,也许番石榴在某个地方有一个标准类,因为我现在默认将番石榴作为依赖项。所以我可以做WrappedException

throw new WrappedException(e);

throw Exceptions.wrap(e);

Exceptions.rethrow(e);

我刚刚在番石榴中环顾四周,发现Storables具有看起来很相似的Throwables.propagate(),但它只是将检查的异常包装在,而不是一个特殊的子类中。RuntimeExceptionRuntimeException

哪种方法更好?与 ?我的顶级代码想知道增加信息价值的最顶层异常。WrappedExceptionRuntimeException

如果我有一个 包装 a 包装 a ,包装不会增加信息价值,我不在乎它,所以我会记录的错误将是 .RuntimeExceptionNastyExceptionNullPointerExceptionRuntimeExceptionNastyException

如果我有一个 包装 a ,通常会增加信息价值。IllegalArgumentExceptionNastyExceptionIllegalArgumentException

因此,在我执行错误日志记录的顶级代码中,我必须执行如下操作:

catch (RuntimeException re)
{
   logError(getTheOutermostUsefulException(re));
}

/** 
 * heuristics to tease out whether an exception
 * is wrapped just for the heck of it, or whether
 * it has informational value
 */
Throwable getTheOutermostUsefulException(RuntimeException re)
{        
   // subclasses of RuntimeException should be used as is
   if (re.getClass() != RuntimeException)
      return re;
   // if a runtime exception has a message, it's probably useful
   else if (re.getMessage() != null)
      return re;
   // if a runtime exception has no cause, it's certainly
   // going to be more useful than null
   else if (re.getCause() == null)
      return re;
   else
      return re.getCause();
}

这种理念对我来说是正确的,但实现感觉很糟糕。有没有更好的方法来处理包装的异常?


相关问题:


答案 1

Spring是我所知道的唯一一个具有相对相似的东西的图书馆。它们有嵌套的例外:NestedRuntimeExceptionNestedCheckedException。这些异常具有有用的方法,例如 或 。他们的方法返回原因的消息(如果包装异常已有消息,则会追加该消息)。getMostSpecificCause()contains(Class exType)getMessage()

它用于Spring的数据访问异常层次结构。这个想法是每个数据库供应商在其 JDBC 驱动程序中公开不同的异常。Spring捕获了这些,并将它们转换为更通用的DataAccessExceptions。这样做的另一个好处是,已检查的异常会自动转换为运行时异常。

话虽如此,它不是非常复杂的代码,我相信你可以在你的代码库中做类似的事情。无需为此添加弹簧依赖项。

如果我是你,我不会试图过早地“解开”异常,除非你真的可以在当时和那里处理它们。我会让他们冒泡(无论是否包装),使用全局 ExceptionHandler 查看他们的因果链,找到第一个“有意义”的异常,并提取智能错误消息。当无法做到这一点时,我只会打印“技术错误”,并在错误消息的详细信息或某种日志中添加整个堆栈跟踪,以用于错误报告目的。然后,您将修复该错误和/或引发更有意义的异常。

Guava的Throwables.getCausalChain()也可能对简化异常处理感兴趣:

Iterables.filter(Throwables.getCausalChain(e), IOException.class));

编辑:

我对你的问题想得更多一点,我认为你不应该真正担心用特定的“WrapperException”类型包装你的异常。您应该使用最有意义的方法包装它们:简单(番石榴可能在那里有用),带有附加错误消息,或者在适当的时候使用更有意义的异常类型。RuntimeExceptionThrowables.propagate()RuntimeException

Java的因果链机制无论如何都会让你找到根本原因。您不必担心代码中所有位置的异常包装。编写一个全局异常处理程序来管理它,并在其他任何地方使用标准异常冒泡。


答案 2

通常,最好避免将已检查的异常与未选中的异常包装在一起(请参阅此处此处)。一些绕过它的方法:

  1. 使用“可调用”而不是“可运行”
  2. 使用波希米亚建议的框架Executors
  3. 子类(或您正在使用的任何接口/类),并添加一些方法在事后运行期间检查异常,可能是方法或类似方法。Runnablepublic Exception getRunException()

如果你必须包装一个已检查的异常,我认为最好的方法是像你已经这样做一样,定义一个子类。但如果可能的话,我会尽量避免它。RuntimeException