在@Around通知中处理有@RequestBody和无@RequestBody的请求

Question

我有这样基于方面的日志记录：

@Pointcut("@annotation(Loggable)")
public void loggableAnnotation() {}

@Around("loggableAnnotation()")
public Object simpleProcess(ProceedingJoinPoint joinPoint) throws Throwable {
  return this.processWithBody(joinPoint, null);
}

@Around("loggableAnnotation() && args(@org.springframework.web.bind.annotation.RequestBody body,..)")
public Object processWithBody(ProceedingJoinPoint joinPoint, Object body) throws Throwable {
  // do things
}

当我使用RequestBody执行请求时，它工作正常，会触发adviceprocessWithBody（）。但是，当我执行没有@RequestBody的请求时（仅@PathVariable和@RequestParam）不会触发simpleProcess（），而是在processWithBody（）中，我接收路径变量值作为body参数。

为什么会发生这种情况？我如何以不同的方式处理两种类型的请求（如果可能，在同一个通知中）？




Answer 1

你犯了三个基本的错误：

>

您的切入点语法是错误的，即使您将其转移到execution（），例如执行（**（@org.springframework.web.bind.annotation.RequestBody*，…） 将匹配参数的有一个@RequestBody注释，而不是参数本身
为了实现这一点，您需要将参数本身放入括号中，例如执行（**（@org.springframework.web.bind.annotation.RequestBody（*）），…） 。

您必须确保切入点是互斥的，否则多个建议将在同一个连接点上匹配。准确地说，您需要区分以下情况：

• 方法由Loggable注释，第一个方法参数由RequestBody注释
• 方法由Loggable注释，第一个方法参数不由RequestBody注释
• 由无任何参数的Loggable注释的方法

下面是纯Java AspectJ中的一个示例（没有Spring或Spring AOP），但Spring AOP中的方面语法应该相同：

注释驱动程序应用程序：

package de.scrum_master.app;

import static java.lang.annotation.ElementType.METHOD;
import static java.lang.annotation.RetentionPolicy.RUNTIME;

import java.lang.annotation.Retention;
import java.lang.annotation.Target;

@Retention(RUNTIME)
@Target(METHOD)
public @interface Loggable {}

package de.scrum_master.app;

import org.springframework.web.bind.annotation.RequestBody;

public class Application {
  public static void main(String[] args) {
    Application application = new Application();
    application.doNotLogMe("foo", 11);
    application.doNotLogMeEither();
    application.doNotLogMeEither("foo", 11);
    application.logMe("foo", 11);
    application.logMeToo("foo", 11);
    application.logMeToo();
  }

  public void doNotLogMe(@RequestBody String body, int number) {}
  public void doNotLogMeEither() {}
  public void doNotLogMeEither(String body, int number) {}
  @Loggable public void logMe(@RequestBody String body, int number) {}
  @Loggable public void logMeToo(String body, int number) {}
  @Loggable public void logMeToo() {}
}

方面：

如您所见，我正在使用区分上述三种情况的方法，并且还满足了您对一种常见辅助方法的需求，我称之为<代码>logIt（…） 。在那里，您可以放置所有想要使用的复杂日志记录，而不会在您的advice方法中有任何重复代码。

package de.scrum_master.aspect;

import org.aspectj.lang.ProceedingJoinPoint;
import org.aspectj.lang.annotation.Around;
import org.aspectj.lang.annotation.Aspect;
import org.aspectj.lang.annotation.Pointcut;

@Aspect
public class MyAspect {
  @Pointcut("@annotation(de.scrum_master.app.Loggable)")
  public void loggableAnnotation() {}

  @Around(
    "loggableAnnotation() && " + 
    "execution(* *())"
  )
  public Object simpleProcessWithoutParameters(ProceedingJoinPoint joinPoint) throws Throwable {
    return logIt(joinPoint, null);
  }

  @Around(
    "loggableAnnotation() && " +
    "execution(* *(!@org.springframework.web.bind.annotation.RequestBody (*), ..))"
  )
  public Object simpleProcessWithParameters(ProceedingJoinPoint joinPoint) throws Throwable {
    return logIt(joinPoint, null);
  }

  @Around(
    "loggableAnnotation() && " + 
    "execution(* *(@org.springframework.web.bind.annotation.RequestBody (*), ..)) && " +
    "args(body, ..)"
  )
  public Object processWithBody(ProceedingJoinPoint joinPoint, Object body) throws Throwable {
    return logIt(joinPoint, body);
  }

  private Object logIt(ProceedingJoinPoint joinPoint, Object body) throws Throwable {
    System.out.println(joinPoint + " -> " + body);
    return joinPoint.proceed();
  }
}

控制台日志：

execution(void de.scrum_master.app.Application.logMe(String, int)) -> foo
execution(void de.scrum_master.app.Application.logMeToo(String, int)) -> null
execution(void de.scrum_master.app.Application.logMeToo()) -> null

附言：执行（**（@MyAnn*））和执行（**（@MyAnn (*)))）之间的区别很微妙，因此很棘手。不幸的是，这里没有正确记录它应该在的地方。准确地说，后一种情况根本没有记录，只是可能在一些AeyJ发行说明中，当然也在单元测试中。但是普通用户不会看那里。