异常出现前提：公司的Android项目中有一个功能是获取通讯录中某个部门下的所有成员，因为该部门下的可能会有子部门，但是子部门又可能还有子部门，因此该问题就可转化成一个遍历一棵树所有节点的问题。之前的做法是写了一个方法，用递归的方式求得一个部门下所有成员，代码如下：
/**
*
* @param groupId
* @return
*/
private List<Map<String, String>> getGroupContact(String deptId) {
List<Map<String, String>> selectContactList = ObjectFactory
.newArrayList();
List<Map<String, String>> childGroups = ContactsDB
.getInstance(mContext).selectChildGroup(deptId);
// 将部门下的联系人加入选中列表
for (Map<String, String> contactMap : childGroups) {
String childGroupId = MapUtil.getString(contactMap, Tag.DEPTID);
selectContactList.addAll(getGroupContact(childGroupId));
}
List<Map<String, String>> childContacts = ObjectFactory.newLinkedList();
if (((CreateGroupChatActivity) mContext).isFromWebView) {
childContacts = ContactsDB.getInstance(mContext)
.selectGroupContactById(deptId);
} else {
childContacts = ContactsDB.getInstance(mContext)
.selectGroupContactNotNullById(deptId);
}
selectContactList.addAll(childContacts);
return selectContactList;
}selectGroupContactById和selectGroupContactNotNullById这两个方法都是通过deptId字段获得所有联系人信息，这个递归方法没有问题，可以获得结果，但是之后却发现其效率无比低下，于是只好想着把递归改为循环，于是出现了下面的方法：
public List<String> selectAllChildDeptId(String parendId) {
List<String> childAllDeptIdList = ObjectFactory.newLinkedList();
// 首先获得直系子部门的deptId
childAllDeptIdList = mDataBase.selectChildDeptId(parendId);
for (String deptId:childAllDeptIdList) {
childAllDeptIdList.addAll(mDataBase.selectChildDeptId(deptId));
}
return childAllDeptIdList;
}自然而然地当运行到这里时就出现了ConcurrentModificationException异常，原因是在java中对集合进行迭代，遍历的过程中，如果修改该集合的数据：添加或者删除等操作时，就会出现这个异常，那么为什么会出现这个异常呢，先看下面一段代码：
//ConcurrentModificationException异常测试方法
public static void iteratorTest(){
ArrayList<Integer> list = new ArrayList<Integer>();
list.add(2);
Iterator<Integer> iterator = list.iterator();
while(iterator.hasNext()){
Integer integer = iterator.next();
if(integer==2)
list.remove(integer);
}
}运行后异常为：
从异常信息可以发现，异常出现在checkForComodification()方法中。
我们不忙看checkForComodification()方法的具体实现，我们先根据程序的代码一步一步看ArrayList源码的实现：
首先看ArrayList的iterator()方法的具体实现，查看源码发现在ArrayList的源码中的terator()这个方法如下：
从这段代码可以看出返回的是一个指向Itr类型对象的引用，我们接着看Itr的具体实现：
/**
* An optimized version of AbstractList.Itr
*/
private class Itr implements Iterator<E> {
int cursor; // index of next element to return
int lastRet = -1; // index of last element returned; -1 if no such
int expectedModCount = modCount;
public boolean hasNext() {
return cursor != size;
}
@SuppressWarnings("unchecked")
public E next() {
checkForComodification();
int i = cursor;
if (i >= size)
throw new NoSuchElementException();
Object[] elementData = ArrayList.this.elementData;
if (i >= elementData.length)
throw new ConcurrentModificationException();
cursor = i + 1;
return (E) elementData[lastRet = i];
}
public void remove() {
if (lastRet < 0)
throw new IllegalStateException();
checkForComodification();
try {
ArrayList.this.remove(lastRet);
cursor = lastRet;
lastRet = -1;
expectedModCount = modCount;
} catch (IndexOutOfBoundsException ex) {
throw new ConcurrentModificationException();
}
}
final void checkForComodification() {
if (modCount != expectedModCount)
throw new ConcurrentModificationException();
}
}首先我们看一下它的几个成员变量：
cursor：表示下一个要访问的元素的索引，从next()方法的具体实现就可看出
lastRet：表示上一个访问的元素的索引
expectedModCount：表示对ArrayList修改次数的期望值，它的初始值为modCount。
modCount是AbstractList类（ArrayList的父类）中的一个成员变量
该值表示对List的修改次数，查看ArrayList的add()和remove()方法就可以发现，每次调用add()方法或者remove()方法就会对modCount进行加1操作。如下图：
好了，到这里我们再看看上面的程序：
当调用list.iterator()返回一个Iterator之后，通过Iterator的hashNext()方法判断是否还有元素未被访问，我们看一下hasNext()方法，hashNext()方法的实现很简单：
public boolean hasNext() {
return cursor != size();
}然后通过Iterator的next()方法获取到下标为0的元素，我们看一下next()方法的具体实现：
@SuppressWarnings("unchecked")
public E next() {
checkForComodification();
int i = cursor;
if (i >= size)
throw new NoSuchElementException();
Object[] elementData = ArrayList.this.elementData;
if (i >= elementData.length)
throw new ConcurrentModificationException();
cursor = i + 1;
return (E) elementData[lastRet = i];
}这里是非常关键的地方：首先在next()方法中会调用checkForComodification()方法，然后判断是否越界，接着将cursor的值赋给lastRet，并对cursor的值进行加1操作。初始时，cursor为0，lastRet为-1，那么调用一次之后，cursor的值为1，lastRet的值为0。注意此时，modCount为0，expectedModCount也为0。
但是当程序循环运行到我们进行remove（）方法时却报错了，下面我们看下remove（）进行的操作：
/**
* Removes the first occurrence of the specified element from this list,
* if it is present. If the list does not contain the element, it is
* unchanged. More formally, removes the element with the lowest index
* <tt>i</tt> such that
* <tt>(o==null ? get(i)==null : o.equals(get(i)))</tt>
* (if such an element exists). Returns <tt>true</tt> if this list
* contained the specified element (or equivalently, if this list
* changed as a result of the call).
*
* @param o element to be removed from this list, if present
* @return <tt>true</tt> if this list contained the specified element
*/
public boolean remove(Object o) {
if (o == null) {
for (int index = 0; index < size; index++)
if (elementData[index] == null) {
fastRemove(index);
return true;
}
} else {
for (int index = 0; index < size; index++)
if (o.equals(elementData[index])) {
fastRemove(index);
return true;
}
}
return false;
}
/*
* Private remove method that skips bounds checking and does not
* return the value removed.
*/
private void fastRemove(int index) {
modCount++;
int numMoved = size - index - 1;
if (numMoved > 0)
System.arraycopy(elementData, index+1, elementData, index,
numMoved);
elementData[--size] = null; // Let gc do its work
}
通过remove方法删除元素最终是调用的fastRemove()方法，在fastRemove()方法中，首先对modCount进行加1操作（因为对集合修改了一次），然后接下来就是删除元素的操作，最后将size进行减1操作，并将引用置为null以方便垃圾收集器进行回收工作。
那么注意此时各个变量的值：对于iterator，其expectedModCount为0，cursor的值为1，lastRet的值为0。
对于list，其modCount为1，size为0。
接着看程序代码，执行完删除操作后，继续while循环，调用hasNext方法()判断，由于此时cursor为1，而size为0，那么返回true，所以继续执行while循环，然后继续调用iterator的next()方法：
注意，此时要注意next()方法中的第一句：checkForComodification()。
在checkForComodification方法中进行的操作是：
final void checkForComodification() {
if (modCount != expectedModCount)
throw new ConcurrentModificationException();
}如果modCount不等于expectedModCount，则抛出ConcurrentModificationException异常。
很显然，此时modCount为1，而expectedModCount为0，因此程序就抛出了ConcurrentModificationException异常。
到这里，想必大家应该明白为何上述代码会抛出ConcurrentModificationException异常了。
关键点就在于：调用list.remove()方法导致modCount和expectedModCount的值不一致。
注意，像使用for-each进行迭代实际上也会出现这种问题。
在单线程环境下的解决办法
往前看可以发现，在实现iterator接口的Itr类中也有remove（）方法，如下：
public void remove() {
if (lastRet == -1)
throw new IllegalStateException();
checkForComodification();
try {
AbstractList.this.remove(lastRet);
if (lastRet < cursor)
cursor--;
lastRet = -1;
expectedModCount = modCount;
} catch (IndexOutOfBoundsException e) {
throw new ConcurrentModificationException();
}
}expectedModCount = modCount;
因此，在迭代器中如果要删除元素的话，需要调用Itr类的remove方法。
将上述代码改为下面这样就不会报错了：
public class Test {
public static void main(String[] args) {
ArrayList<Integer> list = new ArrayList<Integer>();
list.add(2);
Iterator<Integer> iterator = list.iterator();
while(iterator.hasNext()){
Integer integer = iterator.next();
if(integer==2)
iterator.remove(); //注意这个地方
}
}
}在多线程环境下的解决方法
上面的解决办法在单线程环境下适用，但是在多线程下适用吗？看下面一个例子：public class Test {
static ArrayList<Integer> list = new ArrayList<Integer>();
public static void main(String[] args) {
list.add(1);
list.add(2);
list.add(3);
list.add(4);
list.add(5);
Thread thread1 = new Thread(){
public void run() {
Iterator<Integer> iterator = list.iterator();
while(iterator.hasNext()){
Integer integer = iterator.next();
System.out.println(integer);
try {
Thread.sleep(100);
} catch (InterruptedException e) {
e.printStackTrace();
}
}
};
};
Thread thread2 = new Thread(){
public void run() {
Iterator<Integer> iterator = list.iterator();
while(iterator.hasNext()){
Integer integer = iterator.next();
if(integer==2)
iterator.remove();
}
};
};
thread1.start();
thread2.start();
}
}运行结果：
有可能有朋友说ArrayList是非线程安全的容器，换成Vector就没问题了，实际上换成Vector还是会出现这种错误。
原因在于，虽然Vector的方法采用了synchronized进行了同步，但是由于Vector是继承的AbstarctList，因此通过Iterator来访问容器的话，事实上是不需要获取锁就可以访问。那么显然，由于使用iterator对容器进行访问不需要获取锁，在多线程中就会造成当一个线程删除了元素，由于modCount是AbstarctList的成员变量，因此可能会导致在其他线程中modCount和expectedModCount值不等。
就比如上面的代码中，很显然iterator是线程私有的，
初始时，线程1和线程2中的modCount、expectedModCount都为0，
当线程2通过iterator.remove()删除元素时，会修改modCount值为1，并且会修改线程2中的expectedModCount的值为1，
而此时线程1中的expectedModCount值为0，虽然modCount不是volatile变量，不保证线程1一定看得到线程2修改后的modCount的值，但是也有可能看得到线程2对modCount的修改，这样就有可能导致线程1中比较expectedModCount和modCount不等，而抛出异常。
因此一般有2种解决办法：
1）在使用iterator迭代的时候使用synchronized或者Lock进行同步；
2）使用并发容器CopyOnWriteArrayList代替ArrayList和Vector。
最后还有一点，大家可能会发现，iterator没有add方法，只有remove，那该怎么办呢，这时候就要用到ListIterator了，如下：
/**
* 通过上级部门ID,查找所有子部门信息(只提取子部门的deptId)
*
* @param parendId
*/
public List<String> selectAllChildDeptId(String parendId) {
List<String> childAllDeptIdList = ObjectFactory.newLinkedList();
// 首先获得直系子部门的deptId
childAllDeptIdList = mDataBase.selectChildDeptId(parendId);
ListIterator<String> iterator = childAllDeptIdList.listIterator();
while (iterator.hasNext()) {
String deptId = iterator.next();
List<String> childDeptList = mDataBase.selectChildDeptId(deptId);
for (String childDeptId : childDeptList) {
iterator.add(childDeptId);
}
}
return childAllDeptIdList;
}
2163
被折叠的 条评论
为什么被折叠?
到【灌水乐园】发言
