我知道这个问题以前可能已经被问过很多遍了,但是我有一个问题,我找不到特别针对我的问题的解决方案。
问题如下:
我有两个火箭和弹丸的数组列表,并检查了它们之间的碰撞。当它们碰撞时,它们都将被移除。问题是,当击中arraylist中唯一的火箭时,就会发生异常,而我不知道如何阻止它的发生。
java.lang.IndexOutOfBoundsException: Index: 3, Size: 3
at java.util.ArrayList.rangeCheck(Unknown Source)
at java.util.ArrayList.get(Unknown Source)
at adam.miszczak.defendthebase.level.Level.collision(Level.java:158)
at adam.miszczak.defendthebase.level.Level.tick(Level.java:144)
at adam.miszczak.defendthebase.Game.tick(Game.java:124)
at adam.miszczak.defendthebase.Game.run(Game.java:104)
at java.lang.Thread.run(Unknown Source)
碰撞方法+数组列表:
public static ArrayList<Projectile> projectiles = new ArrayList<Projectile>();
private static ArrayList<Rocket> rockets = new ArrayList<Rocket>();
private void collision(){
for(int i = 0; i < rockets.size(); i++){
for(int j = 0; j < projectiles.size(); j++){
try{
if(rockets.get(i).bounds().intersects(projectiles.get(j).bounds())){
projectiles.remove(j);
rockets.get(i).die();
stats.addScore(rockets.get(i));
Particle p = new Particle(20, 20, 20, 50);
particles.add(p);
}
}catch(IndexOutOfBoundsException e){
e.printStackTrace();
}
}
}
}
删除和添加方法:
private void remove(ArrayList<Projectile> projectiles, ArrayList<Rocket> rockets){
for(int i = 0; i < rockets.size(); i++){
if(rockets.get(i).getVisible()){ rockets.remove(i); rocketsOnScreen--; }
}
for(int i = 0; i < projectiles.size(); i++){
if(projectiles.get(i).isRemoved()){ projectiles.remove(i);}
}
for(int i = 0; i < planes.size(); i++){
if(planes.get(i).isRemoved()){ planes.remove(i); planesOnLevel--;}
}
}
private void spawnRocket(int rocketType, int x, ArrayList<Rocket> rockets) {
switch (rocketType) {
case ROCKET_NORMAL:
rockets.add(new NormalRocket(x, -10, 80, 0, 2));
rocketsSpawned++;
rocketsOnScreen++;
break;
case ROCKET_FIRE:
if(difficulty > 1 && random.nextInt(100) > fireRocketSpawn){
rockets.add(new FireRocket(x, -10, 70, 0, 2));
rocketsSpawned++;
rocketsOnScreen++;
}else{
return;
}
break;
case ROCKET_ZIPPER:
if(difficulty > 2 && random.nextInt(100) > zipperRocketSpawn){
rockets.add(new ZipperRocket(x, -10, 40, 0, 4));
rocketsSpawned++;
rocketsOnScreen++;
}else{
return;
}
break;
case ROCKET_TANK:
if(difficulty > 3 && random.nextInt(100) > tankRocketSpawn){
rockets.add(new TankRocket(x, -10, 130, 0, 1));
rocketsSpawned++;
rocketsOnScreen++;
}else{
return;
}
break;
}
}
注意:rocket.die()基本上只是删除火箭,就像remove方法一样。
最佳答案
在除List
以外的任何其他内容上遍历Iterator
时,无法删除。想象一下后果。您正在遍历元素并删除元素n
。所有其他元素向下移动一个以填补空白。您使用元素n+1
,现在是开始循环时的元素n+2
。而且List
小一个元素,因此您的边界检查现在是错误的。您可以通过这种方式获得各种难以跟踪的错误。
在Java中,如果您使用正确的语法-即enhanced foreach loops,那么即使您在迭代时甚至尝试从列表中添加/删除列表,也会得到正确的错误-您将获得ConcurrentModificationException
。
要修正您的方法,请执行以下操作:1)删除所有索引。您正在滥用它们,它们是不必要的(现在很多时候,您在每次迭代中是否从每个List
中获得相同的元素?)。 2)对内部循环使用实际的Iterator
,以便您可以调用Iterator.remove
从List
中删除最后返回的元素。因为这是通过Iterator
完成的,所以不会导致错误:
private void collision() {
for (final Rocket rocket : rockets) {
final Iterator<Projectile> iter = projectiles.iterator();
while (iter.hasNext()) {
final Projectile projectile = iter.next();
if (rocket.bounds().intersects(projectile.bounds())) {
rocket.die();
stats.addScore(rocket);
Particle p = new Particle(20, 20, 20, 50);
particles.add(p);
iter.remove();
}
}
}
}