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"; }
-
在开发过程还是要注意这些细节,养成良好习惯!!