【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,那么应该在查库放入缓存前就进行排序。即将现有排序逻辑移至查询生成缓存处,使用缓存时不再需要进行排序,就会避免这个问题。

总结

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

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

猜你喜欢

转载自blog.csdn.net/qq_34577234/article/details/125239921