1、问题
在最新的一次需求上线后,后台报了很多ConcurrentModificationException异常,其实是很低级的错误,但还是记录下来,避免以后再犯这样的低级错误,报该异常的代码如下:
public void removeAllModule() {
Set<Integer> keySet = moduleMap.keySet();
// 使用这种方式进行遍历keySet,其实质上是使用keySet的迭代器进行遍历
// 所以这里会通过keySet.iterator()获取迭代器进行用于遍历操作
// 异常是从这里抛出
for(Integer key : keySet){
AdModule adModule = getModule(key);
if (adModule != null) {
adModule.destroy();
}
// 使用Set的remove()方法删除元素
keySet.remove(key);
}
}
这里我们来看一下iterator的说明
/**
*
* If a collection has been changed since the iterator was created,
* methods {@code next} and {@code hasNext()} may throw a {@code ConcurrentModificationException}.
*
* 如果获取一个集合的迭代器后,该集合的内容发生的变化(即添加了新的元素或删除元素)
* 那执行迭代器的next()或hasNext()将会抛出异常ConcurrentModificationException
*
* 。。。。
*/
public interface Iterator<E> {
}
通过iterator的说明可以定位到是由于获取到keySet集合的迭代器后调用了keySet.remove()方法移除了集合中的元素,当调用hasNext()方法后就会抛出异常。
所以在遍历集合时我们需要多加小心,如果需要在遍历的过程中删除元素,我们可用iterator迭代器进行删除,即:iterator.remove( )