现象:数组越界
List<Integer> dateList = commonMapper.queryDateBefore(count); dateList.remove(0); return dateList.get(dateList.size() - 1);
报的是数组越界,错误下标是-1;然而我们从日志看commonMapper.queryDateBefore 返回的结果是8条;8条去掉1条,怎么会变成0了呢?
分析:缓存与返回同对象
一开始百思不得其解,翻遍日志,可以看到所有调用commonMapper.queryDateBefore的返回都是8条,而这里的List dateList是个局部变量,也不会有多线程来干扰
甚至看了remove()的源码,当然结果一无所获
直到后面打了断点,发现这个地方会执行多次,突然想到会不会是myBatis缓存问题,我们看到的日志是8没错,但是同一个线程多次执行此Sql,我印象中后续是会直接返回缓存,而不是再查数据库的,那myBatis自然不会再打印查询日志了,所以造成了所有查询的结果都是8条。
这其实有点让人意外,因为我一直认为myBatis的缓存是一种备份形式,没想到其缓存和返回值就是同一个对象,对查出来的数据进行修改后,再次查询其原值居然是错的
为了验证这种猜测,找到了缓存部分的源码:org.apache.ibatis.executor.BaseExecutor
果然,在查询list的时候,会把该list存储在缓存中再返回,如果我们对返回值进行操作,那实际也就是在操作缓存内容,导致本线程第二次查询时,得到错误的数据
结果:拷贝后修改
在查询出结果后,不直接对原list进行操作,而是对list采用一次部分深拷贝,然后再操作新集合
List<Integer> dateList = commonMapper.queryDateBefore(count); ArrayList<Object> newDateList = new ArrayList<>(dateList); newDateList.remove(0); return newDateList.get(newDateList.size() - 1);