求Java高手, 迭代器与for循环的问题

方法1 : 这是从一个集合中取出所有子弹并且绘出来

Iterator <Weapons> iter1 = this.fio.getH_gun().iterator(); //取出集合的元素
while(iter1.hasNext())
{
Weapons w = iter1.next();
g2.drawImage(w.getH_Image(), w.getX(), w.getY(), this);

if(!w.isLive) //如果子弹为死
{
this.fio.getH_gun().remove(w);
}

方法2 : 方法2和方法1的功能一样,只不过是for循环

for(int i=0; i<this.fio.getH_gun().size(); i++)
{
Weapons w = this.fio.getH_gun().get(i);
g2.drawImage(w.getH_Image(), w.getX(), w.getY(), this);

if(!w.isLive)
{
this.fio.getH_gun().remove(w);
}

}

}

现有一个问题,方法1会报异常,如下:

java.util.ConcurrentModificationException

求解!
方法2不会报异常

因为方法2是错误的!你一旦进入
if(!w.isLive)
{
this.fio.getH_gun().remove(w);
}
这块代码,就会移掉一个元素,后面的元素会向前移,所有后面的元素的下标会减1,导致接下来的那个元素无法遍历到
你该成
if(!w.isLive)
{
this.fio.getH_gun().remove(w);
i--;
}
或者循环的时候
for(int i=this.fio.getH_gun().size()-1;i>=0; i--)
这样往前循环也是对的

异常肯定会重现,具体原因自己找!
温馨提示:内容为网友见解,仅供参考
第1个回答  2012-06-15
在删除元素的时候,只有迭代器是安全的,可以使用 Iterator 本身的方法 remove() 来删除对象。
第2个回答  2012-06-12
迭代器在遍历的时候是不允许修改其长度的
第3个回答  2012-06-11
- -很久没用了 都忘了
第4个回答  2012-06-11
iter1 remove();
this.fio.getH_gun().remove(w);
第5个回答  2012-06-12
其实方法2也有,只不过方式不一样,因为你遍历的对象 长度变化了。
相似回答