java - 从 ArrayList 中删除对象会破坏程序

Question

我正在创建一个 sscce，但是代码太多，无法在此处发布，因此我将发布部分内容，希望您能够关注!

所以，我有一个名为 Ship 的类，在该类中有这样的内容:

block 1:

keyboard.keyPress("SPACE", new Runnable(){

    @Override
    public void run(){
        Laser laser = new Laser(
                new Sprite("/media/images/laser.png"),
                sscce.Ship.this.room);
        sscce.Ship.this.room.addGameObjectAt(
                laser,
                sscce.Ship.this.getX() + (sscce.Ship.this.getWidth() / 2) - 3,
                sscce.Ship.this.getY());
    }
});

当按下Space键时，它会创建一个Laser对象，然后我们将其添加到游戏中，该对象存储在ArrayList中> 然后在我的主游戏循环中绘制到屏幕，该循环遍历 ArrayList。

Laser是一个非常简单的类，它看起来像这样:

block 2:

package JGame.GameObject;

import JGame.Room.Room;

public class Laser extends GameObject{

    public Laser(Sprite sprite, Room room){
        super(sprite, room);
        move.moveToY(0 - this.getHeight(), 5, new Runnable(){
            @Override
            public void run(){
                Laser.this.destroy.destroyGameObject();
            }
        });
    }
}

因此， block 1 创建 block 2，然后将其添加到数组列表中。这一切都运行良好，花花公子!

block 3:

public void moveToY(int y, int speed, Runnable complete){
    this.obj.moveEndY = y;
    this.obj.moveAmountY = speed;
    this.complete = complete;
    this.obj.needsToMoveY = true;
}

这会向新创建的激光添加移动操作。然后我们垂直移动对象(由 block 2 定义并在主游戏循环中执行)。

下一部分是破坏游戏，我所说的破坏是指一切都停止了(事件监听器、移动等)。

在 block 2 中，我说“当 moveToY 到达目的地时，运行可运行代码”，可运行代码表示使用 destroyGameObject() 销毁激光。销毁看起来像这样，并在所有游戏对象构造函数(Laser、Ship 等)中初始化。

block 4:

package JGame.Actions;

import JGame.GameObject.GameObject;
import JGame.Room.Room;

public class DestroyAction extends Action{
    GameObject obj;
    Room room;
    public DestroyAction(Room room, GameObject obj){
        this.room = room;
        this.obj = obj;
    }

    public void destroyGameObject(){
        room.removeGameObject(this.obj);
    }
}

block 4 只是告诉房间“嘿，我们需要从列表中删除这个对象”

block 5:

public class Room extends JPanel implements Runnable{

    ArrayList<GameObject> gameObjects = new ArrayList<>();

    public void run(){
        try{
            while(true){
                // Do Event Listeners
                // Move objects along Y axis
                for(GameObject go : gameObjects){
                    if(go.needsToMoveY){
                        int objY = go.getY();
                        if(objY > go.moveEndY){
                            go.setY(objY - go.moveAmountY);
                        }else if(objY < go.moveEndY){
                            go.setY(objY + go.moveAmountY);
                        }else{
                            go.needsToMoveY = false;
                            // Executes Runnable parameter from moveToY()
                            go.move.actionComplete();
                        }
                    }
                }
                this.repaint();
                Thread.sleep(roomSpeed);
            }
        }catch(Exception e){
        }
    }

    public void removeGameObject(GameObject go){
        gameObjects.remove(go);
        System.gc();
    }
}

所以，最后在 block 5 中，这是主循环，许多代码已被删除，但这会测试它是否应该移动，如果完成，则运行通过 moveToY() 传入的 actionComplete()作为可运行的。好吧，这个 Runnable 表示当它到达它的位置时删除该对象。当它在 Runnable 1 时间内运行代码时，一切都会停止。 Action 停止；因此，如果发射 2 个以上的激光，它们应该向上移动，但它们会停在自己的位置并且不会移动。键盘按钮停止工作，因此移动Ship的箭头不再响应，因此您可以向上/下/左/右导航船，它只是位于可运行的位置的最后一个位置第一次触发。

所以...为什么一切都停止了？我只想删除 Runnable 中的对象。

Answer 1

您的代码组织存在严重问题:您正在直接修改 Room 中的 gameObjects 列表(在方法 removeGameObject 中)同时使用迭代器遍历列表(在方法 run 中)。这将导致迭代器提示(如抛出异常)。请注意，您正在捕获(并默默地忽略!)主 run 方法中的异常。永远不要丢弃这样的异常——当出现问题时它们会告诉你。

您需要组织代码，以便仅当没有迭代器遍历列表时才从列表中删除对象。您可以通过创建一种方法来标记要删除的对象，然后在游戏循环的单独步骤中删除它们来实现此目的。