我可以使用 Spring AOP 拦截方法中的空指针异常吗？

Question

我们有一堆 pdf 报告类，在大量导入存档数据后，当数据位丢失时，它们偶尔会抛出空指针异常，例如 thing.getOtherThing().getText(); 将不会OtherThing 并且报告会翻倒。

正如我所见，我们有几个选项

1. 修复数据 - 不太清楚每种情况下的数据应该是什么，在某些情况下是人为判断的问题
2. 将 try/catch 到处放置并重新抛出一个有用的异常以向用户冒泡- 大量工作和丑陋的代码，但对用户有用
3. 在各处放置空检查 - 不，谢谢
4. 在 DAO 代码上工作，在查询中进行合并/nvl 并在缺失字段（或银行）中生成 N/A 的报告 - 好的但没有那么有用

或者...可以我们在这里用 AOP 做什么？是否可以拦截某些类/方法中抛出的空指针？当这些信息被拦截时，可以获得什么级别的信息？请记住，方法中的多个位置可能会抛出空指针:(

谢谢。

Answer 1

呃..这可能很痛苦，但我真诚地建议您仔细检查您的代码，并在任何合理的地方检查 null ，为自己构建一个实用程序类，这样您就可以执行类似

MyUtilityHelper.checkNotNull (toCheck, beautifulMessage)

并让该方法抛出一些您可以获得更多信息的内容。

AOP在这里确实不适用恕我直言，你不想用AOP做应该用顺序逻辑完成的事情，只是因为代码更适合顺序处理......当你想添加水平层时AOP适合功能，例如安全性、日志记录......

Answer 2

AOP 可能会帮助您找到解决方案，但您需要在抛出异常之前采取行动，因为异常会导致您丢失太多执行上下文。首先，当您编写 thing.getOtherThing.getText() 时， getOtherThing 是一个具有奇怪名称的字段或您不小心省略了括号的方法调用？如果是后者，可以使用Spring AOP来拦截方法（provided thing实际上是一个bean）。如果没有，您将需要完整的 AspectJ，因为 Spring AOP 不会进行字段访问拦截（因为它的实现方式）。

这个想法是，您使用 @Around 来定义切入点并提供如下建议：

static final OtherThing THE_DEFAULT_VALUE = ...;

@Around("execution(* getOtherThing())")
public Object supplyDefault(ProceedingJoinPoint pjp) throws Throwable {
    Object result = pjp.proceed();
    if (result == null)
        result = THE_DEFAULT_VALUE;
    return result;
}

或者，使用 AspectJ （未经测试；我实际上没有使用 AspectJ 进行任何操作）：

aspect ADefaultOtherThing {
    static final OtherThing THE_DEFAULT_VALUE = ...;

    OtherThing around(): get(OtherThing getOtherThing) {
        OtherThing result = proceed();
        if (result == null)
            result = THE_DEFAULT_VALUE;
        return result;
    }
}

尽管如此，我还是会质疑是否这是AOP的一个特别好的用法。将这样的业务逻辑直接放在代码中肯定会更好吗？保证字段永远不会null 似乎是一个很好的不变量。