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/

相关文章:

multithreading - x86_64 CPU 是否使用相同的缓存线通过共享内存在两个进程之间进行通信?

java - 如何将这2个方法转换为泛型方法?

java - 运行jms队列时,生产者发送消息,消费者获取消息,但消息不显示

ios - 我需要使计时器失效/释放吗?

c# - WCF 中的非等待任务会怎样?

linux - 查找 [主机] : no such host error in Go

java - 当字段不再使用时,我是否每次都必须 .clear() HashSet 和 HashMap ?

java - RFID + Android --> 我从哪里开始?

c - C语言三线程通信问题(出现死锁)

java - 终止 Future 并获得中间结果