直接在迭代时删除会报错!!
for (Enemy enemy : enemies) {
drawTank(enemy.getX(), enemy.getY(), g, enemy.getDirect(), 0 );
for ( Shot shot:enemy.getShots()) {
System.out.println(shot);
if(shot!=null && shot.isLive()) {
g.fill3DRect(shot.getX(), shot.getY(), 2, 2, false);
}else if(shot!=null && !shot.isLive()){
enemy.getShots().remove(shot);//不可以在这里删除某子弹 因为这样会导致iterator迭代器迭代出问题!!!!
}
}
}
正确写法、简单写法:建立一个集合,记录需要删除的元素,之后统一删除。
for (Enemy enemy : enemies) {
drawTank(enemy.getX(), enemy.getY(), g, enemy.getDirect(), 0 );
Vector<Shot> shotsTemps = new Vector<>();//用来记录刚刚得到的所有失去作用的子弹对象
for ( Shot shot:enemy.getShots()) {
System.out.println(shot);
if(shot!=null && shot.isLive()) {
g.fill3DRect(shot.getX(), shot.getY(), 2, 2, false);
}else if(shot!=null && !shot.isLive()){
shotsTemps.add(shot);
// enemy.getShots().remove(shot);//不可以在这里删除某子弹 因为这样会导致iterator迭代器迭代出问题!!!!
}
}
enemy.getShots().removeAll(shotsTemps);//删除刚刚得到的所有失去作用的子弹对象
}
也可以采用fori 写法 遍历删除
总结:
解决方法:(遍历一个集合时如何避免ConcurrentModificationException)
API文档上也有说的! 在迭代时只可以用迭代器进行删除!
单线程情况:
(1)使用Iterator提供的remove方法,用于删除当前元素。
(2)建立一个集合,记录需要删除的元素,之后统一删除。
(3)不使用Iterator进行遍历,需要之一的是自己保证索引正常。
(4)使用并发集合类来避免ConcurrentModificationException,比如使用CopyOnArrayList,而不是ArrayList。
多线程情况:
(1)使用并发集合类,如使用ConcurrentHashMap或者CopyOnWriteArrayList。