java - 在两个地方循环 LinkedLinkst 会出错

Question

所以我正在用 Java 制作一个小游戏，在游戏中您有存储在 LinkedList 中的实体。在 tick 方法中，我像这样遍历实体的 LinkedList:

@Override
public void tick(float delta) {
    for (Entity e : entities) {
        e.tick(delta);
    }
    player.tick(delta);
}

渲染方法也一样

@Override
public void render(Graphics g) {
    g.setColor(Color.DARK_GRAY);
    g.fillRect(0, 0, Game.WIDTH, Game.HEIGHT);
    for(Entity e : entities) {
        e.render(g);
    }
    player.render(g);
}

其中一个实体是一个名为 Block 的类。在 Block 类中有一个函数，如果一个 block 在附近则返回，在这个函数中我还遍历实体的 LinkedList 并从 Block 中的以下 tick 方法调用它:

    @Override
public void tick(float delta) {
    if (color == Color.WHITE) {
        if (getSurrounding(-32, -32) != null && getSurrounding(-32, -32).getColor() == Color.BLACK) {
            if (getSurrounding(-32, -32).getStrength() < strength) {
                getSurrounding(-32, -32).setStrength(-50);
            }
        }
    }

    if (!inGame.entities.isEmpty() && strength <= 0) {
        inGame.entities.remove(this); // <------ REMOVING AN ELEMENT
    }
}

public Block getSurrounding(int xOffset, int yOffset) {
    for (Entity e : inGame.entities) {
        if (e instanceof Block) {
            if (x + xOffset == e.getX() && y + yOffset == e.getY()) {
                return (Block) e;
            }
        }
    }
    return null;
}

(顺便在block的tick方法中调用使用该函数)

现在每次我运行游戏时，游戏都会启动并正常运行几毫秒，然后抛出此异常:

Exception in thread "Thread-2" java.util.ConcurrentModificationException
    at java.util.LinkedList$ListItr.checkForComodification(Unknown Source)
    at java.util.LinkedList$ListItr.next(Unknown Source)
    at _47b3n.game.engine.gamestate.gamestates.InGame.tick(InGame.java:36)
    at _47b3n.game.engine.Handler.tick(Handler.java:17)
    at _47b3n.game.engine.Game.tick(Game.java:70)
    at _47b3n.game.engine.Game.run(Game.java:52)
    at java.lang.Thread.run(Unknown Source)

(前两个代码示例在 InGame.java 文件中，第 36 行是 tick 方法中的 for 循环)现在我该如何停止这个错误？

Answer 1

ConcurrentModificationException通常涉及在循环中迭代列表时修改列表。

我的假设是您在迭代时删除了一个元素。

为了阻止这种情况发生，我建议在循环中(并且满足某些条件)将要删除的对象保存在变量中，然后在循环完成后将其从列表中删除。

编辑:根据更新后的问题，您正在删除 tick 方法中的一个元素。我不会在这里这样做，而是返回您想要删除的元素，并且仅当您不再迭代要从中删除的列表时才将其删除。

可能的解决方案:我将创建另一个方法来检查实体的强度并从 tick 方法调用它(这是为了保存 tick 方法必须返回要删除的对象)，如下所示:

@Override
public void tick(float delta) {
    Entity removeEntity = null;
    for (Entity e : entities) {
        e.tick(delta);
        if(e.checkStrength()){
           removeEntity = e;
           break;
        }
    }

    if(removeEntity != null){
       entities.remove(removeEntity);
    }

    player.tick(delta);
}

在实体类中:

public boolean checkStrength(){
    if (!inGame.entities.isEmpty() && strength <= 0) {
        return true;
    }
    return false;
}

请注意，这不是目前为止最好的解决方案，但应该可以帮助您解决现在遇到的问题。有空的时候一定要清理代码。