在 Spring Boot 进程中添加 AOP 将方法返回的对象更改为 null

Question

我想在我的 Spring Boot 应用程序中添加用于日志记录的 AOP。但它似乎以意想不到的方式改变了我的应用程序的行为。

例如，我的应用程序有一个方法doThis()，它创建了一个MyObject 实例：

MyObject myObect = doThis();  // a non-null myObject is returned from doThis()

效果很好，myObject 按预期填充了从doThis() 返回的实例。但我希望 doThis() 方法也通过 AOP 记录一些消息。

然后我添加以下方面类：

@Aspect
public class LoggingAspect {

    private static final Logger logger = LoggerFactory.getLogger(LoggingAspect.class);

    @Around("execution(* my.service.package.*.*(..))")
    public void log(ProceedingJoinPoint joinPoint) throws Throwable {
        logger.info("before");
        joinPoint.proceed();
        logger.info("after");
    }
}

而且我还添加了这个配置类：

@Configuration
@ComponentScan(basePackages = "my.service.package")
@EnableAspectJAutoProxy
public class AppConfig {
    @Bean
    public LoggingAspect aspect() {
        return new LoggingAspect();
    }
}

但是，现在当我运行我的应用程序时，确实会出现日志记录语句，正如预期的那样 - 但现在同样的 doThis() 方法显然返回了一个空对象：

MyObject myObect = doThis(); // myObject is now unexplainedly null

但事实并非如此！我的意思是当我在doThis() 的最后一行设置断点时，它即将返回的MyObject 实例非常明显not null。它已在doThis() 方法中创建和填充。那么它去哪儿了？为什么myObject 没有在doThis() 明确返回非空MyObject 实例时被填充？

似乎方面以某种方式使从doThis() 返回的对象无效。有没有人见过这个？有什么办法解决吗？

我相信我的 execution 语句中的第一个 * 应该表明被拦截的方法可以具有任何返回类型。但是，我截获的方法的返回值似乎以某种方式更改为 null。

我正在研究如何根据评论创建一个“最小的可重现示例”，我将在此处添加，但这似乎是一个相当标准的 AOP 用例，因此在此期间将其扔掉以防有人可能有一些见解。

Answer 1

您犯了一个简单的初学者 AOP 错误：您的 @Around 建议继续执行，但没有返回 proceed() 调用的结果。你的通知方法有一个 void 返回类型，你拦截的目标方法没有。所以建议隐式返回null。顺便说一句，对于像int 这样的原始类型，由于返回类型不兼容，这甚至不起作用并引发异常。我真的很惊讶 Spring AOP，奇怪的是，如果环绕建议返回 void，甚至会拦截非 void 方法，因为在这种情况下，AFAIR 原生 AspectJ 不会匹配非 void 方法。

那你能做什么？

• 如果您真的认为需要，请保留@Around 的建议。通常，只有在您想做的不仅仅是记录事情时才会出现这种情况，例如修改方法参数或返回值，处理异常或其他可能改变控制流的事情：

  @Around("execution(* my.service.package.*.*(..))")
  public Object log(ProceedingJoinPoint joinPoint) throws Throwable {
    logger.info("[BEFORE] " + joinPoint);
    try {
      return joinPoint.proceed();
    }
    finally {
      logger.info("[AFTER] " + joinPoint);
    }
  }
  

• 或者保持简单，只使用一对@Before 和@After 通知方法，如果不需要将数据从一个通知传输到另一个通知。这要简单得多，因为您不需要继续，使用try-finally 或返回任何内容：

  @Before("execution(* my.service.package.*.*(..))")
  public void logBefore(JoinPoint joinPoint) {
    logger.info("[BEFORE] " + joinPoint);
  }
  
  @After("execution(* my.service.package.*.*(..))")
  public void logAfter(JoinPoint joinPoint) {
    logger.info("[AFTER] " + joinPoint);
  }
  

  在这里，您还可以将重复的切入点表达式分解为它自己的@Pointcut，并简单地从两种建议方法中引用它。