Space Invaders ArrayList IndexOutOfBoundsException

时间:2013-10-28 01:08:33

标签: java swing 2d-games

Okey,我已经搜索了这个错误并发现原因可能是我对Arraylist的更新与我的repaint()方法有些冲突。我已经多次修改我的代码试图修复它,但要么我无法编译,要么我最终得到完全相同的错误。 我正在创建太空入侵者并且游戏运行正常,直到我快速拍摄(即点击向上按钮快速)并出现此错误: (注意:“索引:x,大小:x”中的x并不总是1,范围从1到6)

Exception in thread "AWT-EventQueue-0" java.lang.IndexOutOfBoundsException: Index: 1, Size: 1
at java.util.ArrayList.rangeCheck(ArrayList.java:604)
at java.util.ArrayList.get(ArrayList.java:382)
at Board.checkCollisions(Board.java:185)
at Board.actionPerformed(Board.java:172)
at javax.swing.Timer.fireActionPerformed(Timer.java:312)
at javax.swing.Timer$DoPostEvent.run(Timer.java:244)
at java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:251)
at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:733)
at java.awt.EventQueue.access$200(EventQueue.java:103)
at java.awt.EventQueue$3.run(EventQueue.java:694)
at java.awt.EventQueue$3.run(EventQueue.java:692)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:76)
at java.awt.EventQueue.dispatchEvent(EventQueue.java:703)
at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:242)
at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:161)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:150)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:146)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:138)
at java.awt.EventDispatchThread.run(EventDispatchThread.java:91)

以下是似乎导致问题的代码部分。

        public void actionPerformed(ActionEvent e){
    if(enemies.size() == 0){
        ingame = false;
    }

    ArrayList ss = gun.getShots();

    for(int i = 0; i < ss.size(); i++){
        Shot s = (Shot) ss.get(i);
        if(s.isVisible()){
            s.move();
        }
        else{
            ss.remove(i);
        }
    }

    for(int i = 0; i < enemies.size(); i++){
        Enemy en = (Enemy) enemies.get(i);
        if(en.isVisible()){
            en.move();
        }
        else{
            enemies.remove(i);
        }
    }

    for(int i = 0; i < bunkers.size(); i++){
        Bunker b = (Bunker) bunkers.get(i);
        if(b.isVisible() == false){
            bunkers.remove(i);
        }
    }

    gun.move();
    checkCollisions();
    repaint();
}

public void checkCollisions(){

    ArrayList ss = gun.getShots();

    for(int i = 0; i < ss.size(); i++){
        Shot s = (Shot) ss.get(i);
        Rectangle r1 = s.getBounds();
        //Collision for Enemies
        for(int j = 0; j < enemies.size(); j++){
            Enemy en = (Enemy) enemies.get(i);
            Rectangle r2 = en.getBounds();

            if(r1.intersects(r2)){
                s.setVisible(false);
                en.setVisible(false);
            }
        }
        //Collision for Bunker
        for(int j = 0; j < bunkers.size(); j++){
            Bunker b = (Bunker) bunkers.get(i);
            Rectangle r2 = b.getBounds();

            if(r1.intersects(r2)){
                s.setVisible(false);
                b.setVisible(false);
                score++;
            }
        }

    }
}

更具体地说,这些代码行:

checkCollisions();
Enemy en = (Enemy) enemies.get(i);

有时候

checkCollisions();
Bunker b = (Bunker) bunkers.get(i);

还有另一个问题。出于某种原因,当我射击外星人或当我在沙坑射击时,我总是必须射击ArrayList中的第一个因为它太消失了。如果我先拍摄其他人没先发生任何事情,拍摄就会飞过他们。这不是我的主要关注点,所以如果您不想浪费时间向我解释,我理解这一点。 提前谢谢。

2 个答案:

答案 0 :(得分:3)

  

以下是似乎导致问题的代码部分。

没有必要猜测,因为异常堆栈跟踪应该告诉你完全哪一行导致异常。如果你发布了类似的问题,你会想告诉我们导致问题的是哪一行,这里是at Board.checkCollisions(Board.java:185),或者是Board.java的第185行。

此代码看起来很可疑:

for(int i = 0; i < bunkers.size(); i++){
    Bunker b = (Bunker) bunkers.get(i);
    if(b.isVisible() == false){
        bunkers.remove(i);
    }
}

你正在通过它向前搜索时从列表中删除一个项目 - 这是你不应该做的事情,因为它经常会导致像你看到的AIOOBE。考虑通过集合迭代向后

答案 1 :(得分:3)

我猜这两个片段是问题所在:

    for(int j = 0; j < enemies.size(); j++){
        Enemy en = (Enemy) enemies.get(i);


    for(int j = 0; j < bunkers.size(); j++){
        Bunker b = (Bunker) bunkers.get(i);

您使用j进行迭代,但在getter中使用i