Question

我正在创建一个sscce，但是有太多的代码要在这里发布，所以我会发布部分，希望你可以关注！

所以，我有这个名为Ship的类，在类中有这样的：

第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是一个非常简单的类，它看起来像这样：

第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();
            }
        });
    }
}

So Chunk 1创建Chunk 2然后将其添加到数组列表中。这一切都很好，花花公子！

第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;
}

这会为新创建的Laser添加移动操作。然后我们垂直移动对象（由块2定义并在主游戏循环中执行）。

下一部分是打破游戏，休息时我的意思是每件事都停止了（事件发生者，动作等）。

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

第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);
    }
}

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

第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();
    }
}

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

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

Answer 1

您的代码组织存在严重问题：您正在gameObjects直接修改Room列表（在方法removeGameObject中），同时通过列表进行修改迭代器（在方法run中）。这将导致迭代器抱怨（如抛出异常）。请注意，您正在捕捉并默默地忽略主run方法中的！-exceptions。永远不要丢掉这样的例外 - 他们会在出现问题时告诉你。

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

从ArrayList中删除对象中断程序

1 个答案: