JPA持久化问题
问题描述
在项目开发中遇到这个样一个问题,新增数据时某个字段保存为1500,保存之前没有任何修改操作
保存之后因为业务关系又紧接着去查询,操作其他数据,最终结果却变成了3000。具体情况请看示例代码:
@Autowired
private UserRepository userRepository;
public User save(User model){
//此处省略其他细节
//假设:前端入参,user的returnMoney=3000,returnMoneyNew=1500
User after = userRepository.save(model);
resolveDetailService.updatePlanTotalAmount(model.getDebtId(), model.getAppType());
return after;
}
再看调用的其他业务方法updatePlanTotalAmount()
public void updatePlanTotalAmount(String debtId,String appType) {
//此处省略其他代码....
List<User> users = userRecordRepository.findByDebtIdAndAppTypeAndDelFlag(debtId, appType, "0");
List<User> userList = users.stream()
.filter(//一些过滤条件)
.map(user->{
if("2".equals(user.getChangeType())){
BigDecimal oldMoney = user.getReturnMoney()==null? BigDecimal.ZERO : new BigDecimal(user.getReturnMoney());
BigDecimal newMoney = user.getReturnMoneyNew()==null? BigDecimal.ZERO : new BigDecimal(user.getReturnMoneyNew());
if(oldMoney.compareTo(newMoney)>0){
user.setReturnMoneyNew(oldMoney.toString());
}
}
return user;
}).collect(Collectors.toList());
//此处省略其他代码....
}
可以看到,我在保存user后,又需要在其他业务中查询,后续业务就没有保存user的操作。
问题复现
那么问题就出现了,入参参数returnMoney=3000,returnMoneyNew=1500,
可结果确是returnMoney=3000,returnMoneyNew=3000
,为什么returnMoneyNew
的结果是3000而不是1500?难道是名称太相似?no no no…
调试
博主百思不得其解,于是就debug找问题。但是断点在走到保存方法时看到after结果中returnMoneyNew
的确是1500啊,但是结果还是3000。
既然保存之前都没有问题,那么向后看,问题一定在这里:updatePlanTotalAmount()
。
在这里发现只有一个对returnMoneyNew
字段操作的地方:
这里对returnMoneyNew
进行了set,而oldMoney
其实就是returnMoney
,也就是set为3000。
但是请注意,在后续代码中没有任何对user进行save或update的操作。继续往后看,返回save()
方法。
after中returnMoneyNew=3000
,这这这…。
分析
到此时,我突然想起来JPA框架中对象的四种状态:
瞬时状态:对象刚被创建时的状态,也成为临时状态。
持久状态:对象被EntityManager管理时成为持久状态,持久状态的数据最终写入数据库。
游离状态:事务提交后或对象有id且在数据库存在,该对象称为游离状态,脱离EntityManager管理。
删除状态:调用删除方法后,对象即为删除状态。
- 我的
userRepository
在调用save()
后,对象并不是说已经写入数据库,还是在内存中,此时是持久状态,也就是after。 - 然后在
updatePlanTotalAmount()
方法中进行了查询,查询结果中包含了这条数据,而且该数据对象的内存地址和after是相同的,证明还是after对象本身。 - 然后调用
setReturnMoneyNew()
设值为3000,其实就是改变了after中的returnMoneyNew=3000
。 - 事务提交时after内容被修改,最终写入数据库的值也就是3000了。
解决
知道了问题的根源就好办了,博主这里只是需要取值判断,把对象中某个值取出来,不操作after对象本身即可。