修改集合中的元素引起的异常

今天在做项目时，遇到一个问题，先以解决，在这里吼一下，不要犯我的错误。
我有一个集合Collection<Organization>我要删除其中的一个元素，开始代码为
organizations = organizationDao.findAll();  //得到一个organization的集合
Organization org = organizationDao.findById(id);//要删除的那个对象
Iterator  iter = organizations.iterator();
while(iter.hashNext()){
       Organization orgs = (Organization) iter.next();
       if(org.getId() == orgs.getId()){
               organizations.remove(orgs);//错误就在这里
       }


}


运行时它抛出一个异常 java.util.ConcurrentModificationException

原因是因为：在Map或者Collection的时候，不要用它们的API直接修改集合的内容。对 Collection 或 Map 进行迭代操作过
程中尝试直接修改 Collection / Map 的内容时，即使是在单线程下运行，
java.util.ConcurrentModificationException 异常也将被抛出。


   Iterator 是工作在一个独立的线程中。 Iterator 被创建之后会建立一个指向原来对象的单链索引表，当原来的对象数量发生变化
时，这个索引表的内容不会同步改变，所以当索引指针往后移动的时候就找不到要迭代的对象，所以 Iterator 会马上抛出
java.util.ConcurrentModificationException 异常。


   所以 Iterator 在工作的时候是不允许被迭代的对象被改变的。但可以使用 Iterator 本身的方法 remove() 来删除对
象， Iterator.remove() 方法会在删除当前迭代对象的同时维护索引的一致性。


    后来代码改写成
organizations = organizationDao.findAll();  //得到一个organization的集合
Organization org = organizationDao.findById(id);//要删除的那个对象
Iterator  iter = organizations.iterator();
while(iter.hashNext()){
       Organization orgs = (Organization) iter.next();
       if(org.getId() == orgs.getId()){
               iter.remove();
       }



}

评论

7 楼 wen19851025 2008-03-20  

发现JAVAEYE的编辑器有些难用。。。得整个专门的帮助才行啊。。。传图片引用等等都不知道怎么用。。。汉。。。

6 楼 hbhjun 2008-01-03  

bianqioujin 写道

今天在做项目时，遇到一个问题，先以解决，在这里吼一下，不要犯我的错误。
我有一个集合Collection<Organization>我要删除其中的一个元素，开始代码为
organizations = organizationDao.findAll();  //得到一个organization的集合
Organization org = organizationDao.findById(id);//要删除的那个对象
Iterator  iter = organizations.iterator();
while(iter.hashNext()){
       Organization orgs = (Organization) iter.next();
       if(org.getId() == orgs.getId()){
               organizations.remove(orgs);//错误就在这里
       }


}


运行时它抛出一个异常 java.util.ConcurrentModificationException

原因是因为：在Map或者Collection的时候，不要用它们的API直接修改集合的内容。对 Collection 或 Map 进行迭代操作过
程中尝试直接修改 Collection / Map 的内容时，即使是在单线程下运行，
java.util.ConcurrentModificationException 异常也将被抛出。


   Iterator 是工作在一个独立的线程中。 Iterator 被创建之后会建立一个指向原来对象的单链索引表，当原来的对象数量发生变化
时，这个索引表的内容不会同步改变，所以当索引指针往后移动的时候就找不到要迭代的对象，所以 Iterator 会马上抛出
java.util.ConcurrentModificationException 异常。


   所以 Iterator 在工作的时候是不允许被迭代的对象被改变的。但可以使用 Iterator 本身的方法 remove() 来删除对
象， Iterator.remove() 方法会在删除当前迭代对象的同时维护索引的一致性。


    后来代码改写成
organizations = organizationDao.findAll();  //得到一个organization的集合
Organization org = organizationDao.findById(id);//要删除的那个对象
Iterator  iter = organizations.iterator();
while(iter.hashNext()){
       Organization orgs = (Organization) iter.next();
       if(org.getId() == orgs.getId()){
               iter.remove();
       }



}
我的代码：
for(Taskdispatch td:tdList)
{
if(td.getDptState()!=2)
tdList.remove(td);
}
犯同样错误

5 楼 Eastsun 2007-12-22  

myworkfirst 写道

  有本书，专门讲集合. 多看看，里面讲得很细致。

JAVA文档本来就讲的够仔细了.
JAVA Collection Fram的文档做的相当到位.

4 楼 liusong1111 2007-12-22  

不错，终于知道内部怎么干活的了。
我在写rails代码，也有类似问题，比如：

parent = Parent.find(1)
children = parent.children
for child in children
  if child.id == 3
     child.destroy
  end
end

这样可以把child id=3的记录从数据库删掉，但children的size没有发生变化。必须这样做：

parent = Parent.find(1)
parent.children.destroy_all("id=3")

children不但发送Sql语句，而且把它从自身集合删除。

3 楼 myworkfirst 2007-12-12  

  有本书，专门讲集合. 多看看，里面讲得很细致。

2 楼 抛出异常的爱 2007-12-12  

引用

所以 Iterator 在工作的时候是不允许被迭代的对象被改变的。但可以使用 Iterator 本身的方法 remove() 来删除对
象， Iterator.remove() 方法会在删除当前迭代对象的同时维护索引的一致性。

下次把你的观点标出来才方便看

1 楼 NicholasBugs 2007-12-12  

恩，经验值啊