转自:https://zhuanlan.zhihu.com/p/426115410
1 抛出检查异常
方法若是抛出检查异常，比如 fileNotFound 这种，事务是不会回滚的，原因也很简单，因为 @Transactional 注解默认的 rollbackFor 是运行时异常。这也就是为什么阿里的开发规范中要求一定要指定 rollbackFor 的原因。
@Transactional(rollbackFor = Exception.class)
2 错误的使用 try catch
业务方法中使用了 try catch 捕获了异常，然后异常顺利出现，结果进入你的 catch 块中，你却没有抛出，结果事务没有正确被回滚。这里出现的原理是 Spring 使用了代理来实现事务管理，调用顺序是开启事务，执行目标方法，提交或回滚事务，虽然你的目标方法出现了异常，架不住你自己处理了，在代理类看来，目标方法没有抛出异常。所以事务也就正常提交。
正确做法有两种，一种是在 catch 中继续抛出异常，第二中是告诉 Spring 我的当前事务需要 rollback.
3 错误添加切面
AOP 切面顺序导致事务不能正确回滚，原因是事务切面优先级最低，但如果自定义的切面优先级和它一样，则还是自定义切面在内层，这时若自定义切面没有正确抛出异常，在 catch 中吃掉了异常，此时就会出现和第二种情况类似的情况，代理类得不到异常信息，也就不会回滚。
解决方案就是在切面里面也抛出异常，或者是将自定义的切面的优先级设置为更小。但是建议使用第一种在切面中抛出异常，不过话说回来，为什么我们要为一个已经有事务管理的方法添加切面呢…… 事务方法一般在 Service 层，我们可以为 Controller 层添加切面。
4 @Transactional 事务默认只能加在 public 的方法上
非 public 的方法会导致事务失效。Spring 为方法创建代理，添加事务通知，前提条件是该方法是 public 的，这点需要注意的就是要么设置方法为 public，要么设置可以为非 public 的方法添加事务通知。
5 调用本类方法导致传播行为失效
同一个 Service 的两个方法之间调用，就会出现这个问题，原因还是在代理对象这里，我们期待的调用是一个代理类的调用，但是我们若是直接在方法中内部调用，不好意思，被调用的方法的事务失效，没有被 AOP 增强。
改进方案就是自己调用自己，自己注入自己。
Transactional 并没有保证原子性行为
这个问题非常常见，我们总会以为加了事务管理，尤其是加了 Propagation.REQUIRES_NEW 之后我们的事务就会在方法执行之后提交事务，或是加了 synchronized 关键字之后，这就是一个原子操作了，这是不对的哈。为什么呢？还要从代理类说起，代理类开启事务，执行目标方法，提交事务。而不管是 REQUIRES_NEW 还是 synchronized 关键字都只是作用于目标方法，即使目标方法执行成功，可是事务还是没有提交呢。
对于 insert，delete，update，select --- for update，语句来说，都是原子性的。但是 select 不是。
解决方案就是扩大 synchronized 的范围，为整个代理方法加锁，而不是把锁加在目标方法上。也可以通过 SQL 来控制，保证操作的原子性，使用 select --- for update