背景
由于项目集成了Mybatis-Plus,所以更新语句就自然采用了LambdaUpdateWrapper来构造更新条件,使用 update(T entity, Wrapper<T> updateWrapper)执行更新语句。由于此处逻辑借用CAS思想,利用乐观锁版本号来滚动更新。但是代码部署后发现滚动了3次更新都失败了。
问题复现
问题代码逻辑如下:
LambdaUpdateWrapper<People> updateWrapper = Wrappers.lambdaUpdate();
updateWrapper
.eq(People::getId, people.getId())
.eq(People::getDisable, 1);
boolean update;
int i = 0;
do {
update = iPeopleService.update(new People()
.setMoney(100)
.setVersion(people.getVersion()),updateWrapper);
if (!update) {
log.info("更新人口钱数失败,滚动更新次数:{}", i);
people = iPeopleService.getById(people.getId());
i++;
}
} while (!update && i < 3);
当开启sql日志后,发现三次执行得update语句分别如下:
--第一次执行sql,此条失败的原因是version版本号不对
UPDATE people SET money=?, version=? WHERE id = ? AND disable = ? AND version = ?;
--第二次执行sql,更新版本号后,此条失败的原因是where条件不对,莫名多出了一个version
UPDATE people SET money=?, version=? WHERE id = ? AND disable = ? AND version = ? AND version = ?;
--第三次执行sql,居然又多出了一个version
UPDATE people SET money=?, version=? WHERE id = ? AND disable = ? AND version = ? AND version = ? AND version = ?;
大概问题是发现了点,就去查问题所在,最终在官网看到下面一段话:
而我们的代码不就是因为复用了wrapper了吗,知道问题后我们及时调整逻辑如下:
boolean update;
int i = 0;
do {
LambdaUpdateWrapper<People> updateWrapper = Wrappers.lambdaUpdate();
updateWrapper
.eq(People::getId, people.getId())
.eq(People::getDisable, 1);
update = iPeopleService.update(new People()
.setMoney(100)
.setVersion(people.getVersion()),updateWrapper);
if (!update) {
log.info("更新人口钱数失败,滚动更新次数:{}", i);
people = iPeopleService.getById(people.getId());
i++;
}
} while (!update && i < 3);
代码调整之后测试了几遍发现功能正常了。
总结
- update(T entity, Wrapper<T> updateWrapper)方法中,updateWrapper不能复用!!!
- api调用时一定要先详看官网,理清来龙去脉后再使用可以避免很多坑!!!