synchronized遇上spring @Transactional注解失效问题

synchronized遇上spring @Transactional注解失效问题

场景

多线程环境对数据进行更新，防止并发窜改数据，加了 synchronized 关键字，结果失效，还是出现并发修改数据的情况

@GetMapping("/synTrx")
public String synTrx(){
    CountDownLatch latch = new CountDownLatch(1);
    for (int i = 0; i < 100; i++) {
        new Thread(()->{
            try {
                latch.await();
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
            orderModelService.synTrx();
        }).start();
    }
    latch.countDown();
    return "success";
}

orderService:

@Service
@Slf4j
public class OrderModelServiceImpl extends ServiceImpl<OrderModelMapper, OrderModel>
        implements OrderModelService {

    @Override
    @Transactional
    public synchronized void synTrx() {
        OrderModel orderModel = baseMapper.selectById(16902);
        orderModel.setAmount(orderModel.getAmount() + 1);
        baseMapper.updateById(orderModel);
        OrderModel t = baseMapper.selectById(16902);
        log.info("{} update amount: {}", Thread.currentThread().getName(), t.getAmount());
    }
}

初始化 amount值为0， 如果锁住，则amount终值为 100，看结果:

Thread-16 update amount: 1
Thread-84 update amount: 0
Thread-59 update amount: 2
Thread-53 update amount: 1
Thread-97 update amount: 3
Thread-94 update amount: 2
Thread-58 update amount: 4
Thread-2 update amount: 3
Thread-36 update amount: 5
Thread-69 update amount: 4
Thread-48 update amount: 6
Thread-90 update amount: 7
...
Thread-89 update amount: 51

查询 sql ,见 amount 终值是51， 很不幸，没有锁住

那如果去掉 @Transactional 注解呢，运行结果:

Thread-1 update amount: 1
Thread-98 update amount: 2
Thread-97 update amount: 3
Thread-95 update amount: 4
Thread-96 update amount: 5
...
Thread-11 update amount: 100

不错，成功锁住，那为什么加了transactional注解就不行了呢

原因

spring的事务是基于AOP, 也就是 在执行 synTrx 方法时是这样的:

try{
    startTransaction()
    public synchronized void synTrx() {}
    commit()
}catch(e){
    rollback()
}

这样就清晰了

• 假设 amount=0
• 假设线程t1已经执行完 synTrx()方法，t1释放了synchronized 的锁，但是还在spring 事务中，并没有提交事务
• 此时线程t2将要开始执行 synTrx()方法， 因为 t1释放了 synchronized 的锁, 所以t2可以进入 synTrx()方法内执行，但是t1事务没有提交，修改的数据对其他线程不可见，所以t2查询数据库中的数据还是0，于是就会出现修改的数据是相同的

解决方案

• 可以不使用事务【但对于需要事务的地方不现实】

• 去掉 synTrx()方法上 synchronized关键字，放在调用 synTrx()方法 外面：

  • @GetMapping("/synTrx")
    public String synTrx(){
        CountDownLatch latch = new CountDownLatch(1);
        for (int i = 0; i < 100; i++) {
            new Thread(()->{
                try {
                    latch.await();
                } catch (InterruptedException e) {
                    e.printStackTrace();
                }
                synchronized (this){
                    orderModelService.synTrx();
                }
            }).start();
        }
        latch.countDown();
        return "success";
    }
    

在开发过程还是要注意这些细节，养成良好习惯！！