【BUG共赏】求求你别乱修改缓存——并发场景缓存修改导致的java.util.ConcurrentModificationException

问题背景

最近在配合性能测试对系统一些接口进行批量性能测试 。
测试过程中发现一个接口并发总有部分号码报错，5W测试用例大概30个左右报错。

日志分析

查看上面业务日志，发现报错为调用一个规则决策方法时，抛出了java.util.ConcurrentModificationException异常 。
这是一个很常见的报错：

java.util.ConcurrentModificationException:
	当我们迭代一个ArrayList或者HashMap时，
	如果尝试对集合做一些修改操作（例如删除元素），
	可能会抛出java.util.ConcurrentModificationException的异常

分析代码

走查代码发现方法对缓存A数据中的XXXDto.XXXList 进行了排序，修改了XXXList 导致并发情况下的报错(缓存本质是静态对象，多线程共享）。

• 伪代码如下：

// 存缓存中取出
XXXDto = Cache.get("A");
XXXList = XXXDto.getXXXList()；
// 对XXXList 进行排序
sortFun(XXXList);
// 遍历XXXList 
for (dto : XXXList ) {
	deal(dto);
}

上述代码单线程下不会有问题，但是多线程下，举个例子，当a线程遍历时，b线程对其进行了sort，就会导致这个报错。

问题解决

原则上来讲，缓存应该只读，不能在使用时修改。
因此如果使用时需要一个有序的list，那么应该在查库放入缓存前就进行排序。即将现有排序逻辑移至查询生成缓存处，使用缓存时不再需要进行排序，就会避免这个问题。

总结

• 缓存本质就是静态对象，所以使用时一定要注意。如果想对其进行修改再处理，那么最好拷贝一份，避免引起并发问题。

都看到这里了 不妨点个赞吧！！！
如果有其他看法，欢迎在评论区交流！！！