java - 更新 Sprite 图像时出现 ConcurrentModificationException

标签 java multithreading concurrency 2d-games concurrentmodification

运行我的游戏时,我不断收到 ConcurrentModificationException,该游戏利用多线程创建新的 Sprite 并移动它们。主要问题似乎发生在“火球”的产生和/或移动中。

通过注释掉 createNewFireballs 方法,我已经能够成功运行我的程序,没有出现任何异常。但是,每当我使用 createNewFireballs 方法时,每当我调用更新火球 Sprite 图像的函数时,通常都会出现错误(并且对于任何其他类型的 Sprite 都不会发生)。我想知道是否有人可以帮助我找到问题的根源并可能解决问题。

public synchronized void createNewFireball() {
    new Thread(new Runnable() {
        public void run() {
            while (gameNotPaused && hero.isNotDead()) {
                fireball = new Fireball(dragon.getX(), dragon.getY());
                fireballs.add(fireball);
            try
                {
                Thread.sleep(100); //Waits for .1 second
                }
            catch(InterruptedException e)
                {
                e.printStackTrace();
                Thread.currentThread().interrupt();
                }
            }
        }
    }).start();
}

    //The problem commonly occurs in the update method, 
    //specifically the line  "FireballIter.next().updateImage(g);"
public synchronized void update(Graphics g) {
    Iterator<Sprite> FireballIter = fireballs.iterator();

    Iterator<Sprite> arrowIter = arrows.iterator();
    while (arrowIter.hasNext()) {
        arrowIter.next().updateImage(g);
    }

    Iterator<Sprite> iterator = sprites.iterator(); 
    while (iterator.hasNext()) {
        iterator.next().updateImage(g);
    }

    while (FireballIter.hasNext()) {
        FireballIter.next().updateImage(g);
    }

}
//Although sometimes it occurs as a result of updateScene, which is 
//called in another method which moves all the "projectile" sprites
public synchronized void updateScene(int width, int height) {

    Iterator<Sprite> arrowIter = arrows.iterator();
    while(arrowIter.hasNext()) {
        Sprite spriteObject = arrowIter.next(); 
        ((Arrow) spriteObject).updateState();   
        if (spriteObject.overlaps(dragon, 350, 350)) {
            dragon.arrowHit();
            System.out.printf("Dragon was hit at %d, %d%n, while arrow was at %d,%d%n", dragon.getX(), dragon.getY(), spriteObject.getX(), spriteObject.getY());
            arrowIter.remove();
        }
    }

    Iterator<Sprite> fireballIter = fireballs.iterator();
    while(fireballIter.hasNext()) {
        Sprite spriteObject = fireballIter.next(); 
        ((Fireball) spriteObject).updateState();    
    }
}

@Override
public synchronized void run() {
    while (model.getGameNotPaused()) {
        model.updateScene(view.getWidth(), view.getHeight());
        view.repaint();
        try {
            Thread.sleep(2);
        } catch (InterruptedException e) {
            e.printStackTrace();
            Thread.currentThread().interrupt();
            JOptionPane.showMessageDialog(null, "Press R to resume game.");
        }
    }
}

最佳答案

使 createNewFireball 同步没有任何用处:同步仅适用于该方法的执行,而不适用于线程执行的可运行对象(这很好,因为否则其他方法都无法执行)执行直到 while 循环完成)。

fireballs.add 放入 synchronized block 中,注意确保同步正确的内容:

  • 如果您只是使用synchronized (this),您将在Runnable上进行同步。
  • 而是使用 synchronized (TheContainingClass.this)(其中 TheContainingClass 是包含这些方法的类的名称)。

关于java - 更新 Sprite 图像时出现 ConcurrentModificationException,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/55964299/

相关文章:

Java SSL 握手错误 - Tomcat keystore

android - startActivity应该在主线程上执行吗?

python - 如何在 APScheduler 中使用 Tornado?

java - SuperCSV 和推土机

java - 查询 Junit @parameters 方法的返回类型和测试类构造函数参数类型

python - 如何在多处理进程中保护日志记录对象免受垃圾收集器的影响?

multithreading - 如何使用 Entity Framework 在 SQLite 中设置线程模式?

concurrency - Golang 防止 channel 阻塞

java - 如何维护唯一的实际pdf文件?在服务器中

java - 如何将 groovy dsl 脚本从一个 groovy 文件包含到另一个文件