Spring注释AOP调用了两次

时间:2018-02-08 10:26:00

标签: annotations aop spring-aop

我使用自定义注释为我的spring启动控制器注释了一些用于记录目的的函数。但是,我发现对于嵌套方法,before advice被执行了两次。在这里寻找一些想法。请参阅下面的代码段。

控制器

@RequestMapping(value = "apply")
    @OperationMILog
    public ApplyHttpResponse apply(@RequestHeader final String custId, @RequestAttribute final String cardNo,
        @RequestBody final InstallmentApplyHttpRequest installApplyReq, @PathVariable final String source) {

        //test
        this.test();  //**line 387**

...
}


 ....
     @OperationMILog
        private String test() {
            return this.test1(); //**line 593**
        }

@OperationMILog
private String test1() {
    return "test1";
}

注释

@Target({java.lang.annotation.ElementType.METHOD, java.lang.annotation.ElementType.ANNOTATION_TYPE})
@Retention(RetentionPolicy.RUNTIME)
@Inherited
@Documented
public @interface OperationMILog {

}

方面

@Aspect
public class GenericLoggingAspect {

      public static GenericLoggingAspect genericLoggingAspect;

        @PostConstruct
        public void init() {    
            genericLoggingAspect = this;
        }


    @Before("@annotation(com.mycomp.log.OperationMILog)")
    public void doBefore(JoinPoint joinPoint) {
        System.out.println("Before .........." + joinPoint.getSignature().getName());
    }

}

在控制器中触发应用功能时,将打印以下日志。

Before ..........apply
Before ..........test
Before ..........test
Before ..........test1
Before ..........test1

在doBefore函数处设置断点并在调试模式下查看堆栈跟踪。 当打印出第一个“Before ............ test”时,堆栈跟踪看起来很好。

GenericLoggingAspect.doBefore(JoinPoint) line: 87   
InstallmentController.apply(String, String, InstallmentApplyHttpRequest, String) line: 387  
InstallmentController$$FastClassBySpringCGLIB$$55eeb128.invoke(int, Object, Object[]) line: not available   

当第二个“Before .......... test”显示时,堆栈跟踪如下所示

GenericLoggingAspect.doBefore(JoinPoint) line: 87   
InstallmentController.test() line: 593  
InstallmentController.apply(String, String, InstallmentApplyHttpRequest, String) line: 387  
InstallmentController$$FastClassBySpringCGLIB$$55eeb128.invoke(int, Object, Object[]) line: not available   

我不知道为什么第593行会触发doBefore。同样的情况适用于test1的打印。

我的项目没有任何XML配置,所有配置都在注释中完成。

1 个答案:

答案 0 :(得分:4)

感谢您在我的评论中显示我要求的日志输出。现在我可以告诉你问题是什么:

Before ..........call(String com.mycomp.controller.InstallmentController.test())
Before ..........execution(String com.mycomp.controller.InstallmentController.test())
  1. 很明显你使用AspectJ(可能是LTW),而不是Spring AOP。我为什么这么说呢?因为Spring AOP只知道execution()个连接点,而不是call()个连接点。

  2. 由于上面给出的原因,您的切入点对于每个方法调用匹配两次:一次用于进行调用的连接点(调用者),一次用于实际执行被调用方法的连接点(被调用者)。这就是您在日志中获得两个输出行的原因。

  3. 所以你真正想做的是在你的切入点中指定你想要截取,调用或执行的确切内容。我建议你将&& execution(* *(..))添加到切入点。然后你得到预期的结果,类似于Spring AOP即使没有添加也会做什么。

  4. 获得的经验:AspectJ比Spring AOP强大得多。您需要学习如何使用这样一个强大的工具,有时还会限制其功能。 :-)