今天早上来上班就看到一个之前做的项目出现了一个报错。
1、首先是看日志,检查了一下发现是出现了java.util.ConcurrentModificationException: null异常。
看到具体的处理方法就去看了下代码,传参fids为[0],因为之前没有遇到过这个异常,就去看了一下它发生的场景:
java.util.ConcurrentModificationException
异常通常在使用迭代器或并发修改集合时抛出。这个异常表示在集合被遍历的同时,有其他线程对集合进行了结构性修改(增加、删除、修改元素等)。当发生这种并发修改时,会导致迭代器的状态不一致,从而抛出ConcurrentModificationException
异常。
if (!CollectionUtils.isEmpty(fids)) {
// 移除非法fid
Iterator<Long> iterator = fids.iterator();
while (iterator.hasNext()) {
Long fid = iterator.next();
if (fid == null || fid < 1) {
fids.remove();
}
}
}
结合我的代码我猜测是我迭代器遍历里面用集合去remove元素,因为进来的是0,就导致迭代器的结构状态与结合本身不一致问题。然后我就自己写了个demo验证了一下,具体如下:
@Test
void modifyExceptionTest(){
List<Integer> list = new ArrayList<>();
list.add(1);
list.add(2);
try {
Iterator<Integer> iterator = list.iterator();
while (iterator.hasNext()){
if (iterator.next() == 2){
list.remove();
}
}
} catch (Exception e) {
throw new RuntimeException(e);
}
}
结果确实出现了java.util.ConcurrentModificationException
异常,然后改用迭代器本身对象去remove,iterator.remove( )就安全,没有出现异常。
也就是修改成:
// 用迭代器移除,不能调用集合对象移除,以防出现两者结构不一致出现异常
iterator.remove();
这里调用迭代器对象去删除也是可以的,因为迭代器内部维护一个指针指向集合的元素,所以删除迭代器的当前元素时也会删除集合中的元素,因此调用iterator.remove( )就可以,并且安全。