我使用自定义注释为我的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配置,所有配置都在注释中完成。
答案 0 :(得分:4)
感谢您在我的评论中显示我要求的日志输出。现在我可以告诉你问题是什么:
Before ..........call(String com.mycomp.controller.InstallmentController.test())
Before ..........execution(String com.mycomp.controller.InstallmentController.test())
很明显你使用AspectJ(可能是LTW),而不是Spring AOP。我为什么这么说呢?因为Spring AOP只知道execution()
个连接点,而不是call()
个连接点。
由于上面给出的原因,您的切入点对于每个方法调用匹配两次:一次用于进行调用的连接点(调用者),一次用于实际执行被调用方法的连接点(被调用者)。这就是您在日志中获得两个输出行的原因。
所以你真正想做的是在你的切入点中指定你想要截取,调用或执行的确切内容。我建议你将&& execution(* *(..))
添加到切入点。然后你得到预期的结果,类似于Spring AOP即使没有添加也会做什么。
获得的经验:AspectJ比Spring AOP强大得多。您需要学习如何使用这样一个强大的工具,有时还会限制其功能。 :-)