数据库中存在一张表,用于存储每个调查问卷的填报编号,填报编号连续且唯一
问卷填报后,在最终提交时, 调用service的接口,获取填报编号
@Override
@Transactional
public synchronized String getQuestionAnswerNumber(String code) {
Integer no = genNumberMapper.getQuestionAnswerNoByCode(code);
if(no == null){
genNumberMapper.insertQuestionAnswerNoByCode(code);
no = 0;
}
GenNumber genNumber = new GenNumber();
genNumber.setStub(code);
genNumber.setNo(no + 1);
genNumberMapper.setQuestionAnswerNoByCode(genNumber);
return String.format("%06d", (no + 1));
}
代码编写流程主要是,通过code关键字,获取填号,然后将编号加1存入到数据库,这样下一个人获取编号时,能够按照顺序递增。为了避免多线程情况下,填报编号错乱,使用synchronized关键字,为方法加锁。
然而在生产环境中,并发量大的情况下,导出数据时,填报编号还是重复了。。。。
我们已经在方法生加了锁,为何编号还会出现重复?
我们在controller中开了多个线程,每个线程中循环100次,调用这个方法。结果还真出现了编号重复的情况。
@RequestMapping("/testSave")
@ResponseBody
@Transactional
public void testSave() {
new Thread(new Runnable(){
@Override
public void run() {
for(int i=0; i<100; i++){
genNumberService.getQuestionAnswerNumber("a");
}
}}).start();
new Thread(new Runnable(){
@Override
public void run() {
for(int i=0; i<100; i++){
genNumberService.getQuestionAnswerNumber("a");
}
}}).start();
}
经过分析,spring 在使用@Transactional关键字开启事务时,在方法执行完毕前,事务没有提交,数据会缓存在mybatis的一级缓存中,当方法执行完后,数据再进行提交。可我们为方法加了锁,为何还是会产生重复的数据呢,经过推测与测试,可能是获取填报编号的方法已经执行了,但事物并没有提交,这个时候另一个线程已经已经进来,导致获取的数据不准确。
验证方式
1.去掉@Transactional关键字,不开启事务。 -- 没有出现重复的
2.在controller层的方法上加锁,service的方法上去掉锁。 -- 没有出现重复的
经过以上方式,我们可以得出结论:spring在事务提交之前,另一个线程已经已经进来,导致获取的数据不准确。
项目采取的解决方案:在controller层的方法上加锁,service的方法上去掉锁。
补充:mybatis缓存机制
-----------------------------------------------------------------------------
感谢 “衣服架子”的评论
看到评论后,重新对问题进行了思考,应该是synchronized和@Transaction一起使用导致的。spring中对事务的处理,是采用动态代理的方式,事务里的方法是包含synchronized方法,导致有多个线程进入到事务中。
参考:https://www.cnblogs.com/Java3y/p/10392645.html