方法同时使用@Transactional和synchronized导致的坑

 

@Transactional(rollbackFor = Exception.class)
public synchronized void pickOrder(BaseOrderDTO orderDTO) {

    // 校验订单
    check(orderDTO);

    // 修改订单数据代码
    orderMapper.modify(orderDTO);

}

---

两个线程同时调用者个方法会导致订单被修改两次。

正常流程应该在第二个线程进来后，检验订单失败。

原因在于事务的范围比synchronized范围大，当锁释放了才会提交事务，在这期间就有可能别的线程读取到任务去执行。

解决方案：将之前事物和同步锁一起作用在方法的代码拆开，修改订单数据重新封装一个方法，并且加上事物。

---

public synchronized void pickOrder(BaseOrderDTO orderDTO) {

    // 校验订单
    check(orderDTO);
    
    // 修改订单数据
    modify(orderDTO);

}

@Transactional(rollbackFor = Exception.class)
private void modify(BaseOrderDTO orderDTO){
    // 修改订单数据
    orderMapper.modify(orderDTO);
}