记一次 Spring Boot 中 @Transactional事务中使用内置锁限制总数失效的BUG

Posted 2021-12-02 番东

前言：业务需要，最多只能绑定5个，目前不考虑集群，单体服务下的内容

第一版代码：

@Transactional(rollbackFor = {CommonException.class}, propagation = Propagation.REQUIRED)
    public Object bindCommonUsedPatient(CommonUsedPatientRequest commonUsedPatientRequest, User user) throws CommonException, ParseException {

        synchronized (this){
            //查询总绑定个数  超过则抛出异常
            //未超过 则向数据库插入一条数据 占位 使用了事务
        }

        //执行其它逻辑
    }

失效原因：同步代码块在事务中，一个事务插入了一条数据之后，在执行其它逻辑，并未完成事务时，另一个线程在另一个事务中无法看见之前插入的数据，导致统计数量失效

第二版代码：

@Transactional(rollbackFor = {CommonException.class}, propagation = Propagation.REQUIRED)
    public Object bindCommonUsedPatient(CommonUsedPatientRequest commonUsedPatientRequest, User user) throws CommonException, ParseException {

        checkCommonUsedPatientAndInsert();
        //执行其它逻辑
    }

@Transactional(propagation = Propagation.REQUIRES_NEW, rollbackFor = Exception.class)
   public synchronized CommonUsedPatient checkCommonUsedPatientAndInsert(CommonUsedPatientRequest commonUsedPatientRequest, User user) throws CommonException {
          //查询总绑定个数  超过则抛出异常
          //未超过 则向数据库插入一条数据 占位 使用了事务
    }

改进方法：将插入数据库的操作封装为方法，并指定事务的传播行为为 REQUIRES_NEW

失效原因：加注解都是生成代理类，事务的功能都在代理类内部，实例方法A调用实例方法B是直接调用，不走代理类，所以事务未生效

第三版代码：

将 checkCommonUsedPatientAndInsert 方法写到另外的 类中，测试OK