最近自己在写一个aop切面,主要是用来记录日志和打印一些信息的,在测试过程中,发现,业务方法在加了事务之后,如果有其他异常抛出,事务并没有回滚,于是,就Google了一把,发现有大佬遇到过这个问题,大部分博客给到的答案是:因为切面的优先级低于事务的优先级,导致事务advisor生成的代理对象被,切面生成的代理对象给覆盖了,需要在切面上加上@Order(1)注解,这样可以保证,切面的优先级较高
我自己忽然间想到之前,遇到过类似的问题,并且也记录到了线上问题模块,按照实际操作之后,确实:在切面上加了@Order(1)注解之后,事务就生效了
但是自己又想了下,之前学习spring源码的时候,并不记得会生成两个代理对象。。。所以这里就很好奇,所以,自己又撸了一遍aop的源码,最后发现原因了,我的结论是这样的:
- 因为我自己写的切面,在@Aroud环绕通知的方法,我手动加了try catch 捕获了throwable 然后打印了一行日志
- 在没有加@Order(1)的时候,确实是事务先执行,然后再执行aop的切面,但是在执行切面的时候,捕获到了数组越界的异常,然后打印了日志,没有继续上抛异常
- 继而导致事务在处理的时候,并没有捕获到异常,直接就提交事务了, 如果用一段伪代码来描述的话,那应该是这样的
try{
try{
执行目标方法
}catch(th){
log.error(“执行报错”)
}
}catch (Throwable ex) {
// target invocation exception
completeTransactionAfterThrowing(txInfo, ex);
throw ex;
}
finally {
cleanupTransactionInfo(txInfo);
}
那么问题,就来了,按照我的这个代码,只有在切面中,记录了error日志之后,然后继续抛出异常就可以了,我自己试了下,不在切面