Question

求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不会报异常

Answer 1

因为方法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--)
这样往前循环也是对的

异常肯定会重现，具体原因自己找！

Answer 2

在删除元素的时候，只有迭代器是安全的，可以使用 Iterator 本身的方法 remove() 来删除对象。

Answer 3

迭代器在遍历的时候是不允许修改其长度的

Answer 4

- -很久没用了 都忘了

Answer 5

iter1 remove();
this.fio.getH_gun().remove(w);

Answer 6

其实方法2也有，只不过方式不一样，因为你遍历的对象 长度变化了。