8

我正在实现一个抛出ApplicationExceptions 的会话 bean。这些异常具有链式堆栈跟踪,其中可能包含其类在客户端上不可用的异常。就像是:

@Override
public void doSomethingSpecial(MyObject o) throws MyException {
    try {
        legacySystem.handle(o);
    } catch (LegacyException e) {
        logger.warn(e.getMessage(), e);
        throw new MyException(e);
    }
}

在这里,客户端可能会遇到一个它没有类的异常。这可能导致:

Exception in thread "main" java.lang.reflect.UndeclaredThrowableException
    at sun.proxy.$Proxy0.doSomethingSpecial(Unknown Source)
    at com.myapp.client.Client.main(Client.java:56)
Caused by: java.lang.ClassNotFoundException: MyLegacyException

我不希望客户端知道可以在服务器端抛出的所有可能的异常,但是拥有堆栈跟踪永远不会坏。

你如何处理这些问题?Interceptor当异常被发送回客户端时,实现一个解耦堆栈跟踪的解决方案是否可行?但是Interceptor应该只处理通过 的调用RemoteInterface,因为在内部我对整个堆栈跟踪感兴趣。

4

3 回答 3

1

这取决于您的客户类型。如果客户是另一个正在开发另一个组件或子系统的团队,我同意你的观点:

拥有堆栈跟踪从来都不是坏事

但是,如果他们是不了解您的应用程序内部结构的客户,那么他们就没有理由知道您的异常类甚至查看您的堆栈跟踪。如果有一个协议可以强制您捕获所有异常并将它们包装在具有error_code属性的高级异常类中,那就太好了。这样,您可以为应用程序中的每个 catch 语句设置一个特定的错误代码,并为您的客户提供这些代码的列表。

无论如何,从技术角度来看,如果您的客户无权访问您的内部Exception类,那么他们无法访问您的堆栈跟踪,而无需引用ClassNotFoundException. 如果您真的希望他们看到堆栈跟踪,一种解决方案可能是让Aspect位于 API 的最上层(将由客户端调用)并捕获所有异常,写入它们的堆栈跟踪在 a 中String并将其作为最终异常的属性发送,该异常将被调用者捕获。这样,调用者可以访问堆栈跟踪作为异常的格式化字符串属性。

编辑:

你甚至可以配置你的构建脚本,这样这个特性就不会成为你发布版本的一部分。因此,您可以仅在调试版本中提供此堆栈跟踪消息。

于 2013-09-20T08:33:06.660 回答
0

我想到了一些迂回的解决方案,但这只是未经检验的猜测。

您使用内部异常初始化外部异常。但是如果我们查看Throwable的 javadoc,我们可以看到方法 get 和setStackTrace(StackTraceElement[] stackTrace)

StackTraceElement用字符串初始化。因此,也许您可​​以从内部异常中获取堆栈跟踪并将其设置为您的外部异常(MyException)。

于 2013-09-20T08:36:17.390 回答
0

由于 RMI 已确定,Serialization您可以使用Serialization特性来有条件地替换异常。

import java.io.ObjectStreamException;
import java.io.Serializable;
import java.lang.reflect.InvocationTargetException;

public class CarryException extends RuntimeException implements Serializable
{
  final String exceptionClass;

  public CarryException(Exception cause)
  {
    super(cause.getMessage());
    exceptionClass=cause.getClass().getName();
    setStackTrace(cause.getStackTrace());
  }

  @Override
  public String getMessage()
  {
    // if we get here, reconstructing the original exception did not work
    return exceptionClass+": "+super.getMessage();
  }

  /** Invoked by Serialization to get the real instance */
  final Object readResolve() throws ObjectStreamException
  {
    try
    {
      Exception ex = Class.forName(exceptionClass).asSubclass(Exception.class)
        .getConstructor(String.class).newInstance(super.getMessage());
      ex.setStackTrace(getStackTrace());
      return ex;
    }
    catch(InstantiationException|IllegalAccessException|ClassNotFoundException
      | IllegalArgumentException|InvocationTargetException|NoSuchMethodException
      | SecurityException ex)
    {
      // can't reconstruct exception on client side
    }
    return this; // use myself as substitute
  }
}

现在您可以通过throw new CarryException(originalException);. 如果类可用,CarryException它将始终记录原始异常的堆栈跟踪和消息,并在客户端重新创建原始异常。否则,客户端会CarryException很明显地看到客户端必须知道一种异常类型。

异常类型必须有标准构造函数接收消息String以使重建工作。(所有其他事情都太复杂了)。但是大多数异常类型都有。

还有另一个Serialization问题:替换 via 仅在Serialization涉及时才有效,因此当在同一个 JVM 中运行时,您不能直接调用实现类上的方法。否则你会CarryException无条件地看到。所以你甚至必须在本地使用存根,例如

((MyRemoteInterface)RemoteObject.toStub(myImplementation)).doSomethingSpecial();

更新

如果MyException客户知道并且只有LegacyException不知道,则以下工作当然有效:

catch (LegacyException e) {
    logger.warn(e.getMessage(), e);
    MyException me=new MyException(e.toString());
    me.setStackTrace(e.getStackTrace());
    throw me;
}
于 2013-09-20T08:41:44.263 回答