【BUG】声明式事务失效导致日志记录失败

发生背景

公司里有一段代码，刚看到我是十分震惊的，我们知道在这种场景下事务会失效

1. 被@Transactional 注解修饰的事务方法权限修饰符只能是 public，否则事务会失效
2. 当前类中的方法不能调用事务方法，否则事务失效

然而公司内有一段向数据库中添加日志的代码，都存在导致事务失效的问题；带着事务是否会失效的疑问，自己亲自在本地实现了类似的代码。

问题复现

@Transactional
@Service
public class UserServiceImpl {@Autowiredprivate UserDao userDao;public void register(User user) {//...save(user);//...}// 模拟添加日志到数据库@Transactional(propagation = Propagation.REQUIRES_NEW)private void save(User user) {userDao.save(user);}
}

/*** <p>*  测试事务失效* @author zhl*/
public class TransactionTest {@Testpublic void testTransaction(){ApplicationContext ctx = new AnnotationConfigApplicationContext(MyBatisAutoConfiguration.class);User user = new User();user.setName("zhangsan");user.setPassword("123456");UserServiceImpl userServiceImpl = (UserServiceImpl) ctx.getBean("userServiceImpl");userServiceImpl.register(user);}
}

被 @Transactional 修饰的save方法的事务失效，传播属性失效，由于当前类被 @Transactional 修饰，register 方法变成事务方法。并且通过控制台输出的日志，只有 register 方法开启了事务，save 方法事务确实失效了。
说明：权限修饰符不是 public 或 当前类中直接调用事务方法 都会导致事务失效。

验证一：事务方法权限修饰符不是 public，导致事务失效

• 当前类实现 ApplicationContextAware ，从 Spring 容器中获取代理后的 UserServiceImpl.save 方法，让执行事务方法的途径正确（排除当前类直接调用事务方法导致的事务方法失效）

  1. 权限修饰符非 public，控制台只输出了一个 register 事务，原save方法事务失效；
     

  2. 权限修饰符为 public，控制台输出了两个事务，且在最外层通过除零运行时异常，发现只有最外层 register 方法事务异常回滚，内部 save 方法事务正常提交，说明 save 方法的传播属性正常生效。
     
     

验证二：本类方法直接调用本类的事务方法，导致事务失效

• 将 save 方法的权限修饰符改为 public（排除权限修饰符非 public 导致的事务失效）
  1. 控制台只输出一个事务，说明原 save方法事务失效
     
  2. 当前类实现 ApplicationContextAware，从 Spring 容器中获取代理后的 UserServiceImpl.save 方法；发现控制台输出两个事务，且原 save 方法事务正常挂起，证明本类直接调用本类的事务方法，导致原事务方法失效。
     
     

结论

经过验证证明 save 方法事务失效。由于类上的 @Transactional 注解，为所有方法开启了事务，包括register方法，传播属性和隔离属性等都是默认值，如果在 save 方法外，register 方法中遇到异常，整个方法都会回滚，save 事务失效传播属性失效，同样随着外部方法回滚而回滚，save 方法插入数据库失败。

由于直接调用了 save 方法，save 方法没有被 Spring 加工代理，导致事务失效；
由于权限修饰符不是 public，导致在动态代理时，无法访问到事务方法，无法为该方法进行加工添加事务。

解决方法

从 Spring 容器获取 Bean，调用事务方法，并且修改事务方法的权限修饰符为 public

@Transactional
@Service
public class UserServiceImpl implements ApplicationContextAware {private ApplicationContext applicationContext;@Autowiredprivate UserDao userDao;public void register(User user) {//...UserServiceImpl userServiceImpl = (UserServiceImpl) applicationContext.getBean("userServiceImpl");userServiceImpl.save(user);//...}//模拟日志记录@Transactional(propagation = Propagation.REQUIRES_NEW)public void save(User user) {userDao.save(user);}@Overridepublic void setApplicationContext(ApplicationContext applicationContext) throws BeansException {this.applicationContext = applicationContext;}
}

建议在方法上使用 @Transactional 注解