4

Есть код: ссылка на github. В ходе выполнения выбрасывается исключение Exception in thread "AWT-EventQueue-0" java.util.ConcurrentModificationException Изображение Знающие люди подскажите как побороть проблему. Заранее спасибо)

3 ответа 3

19

Его не надо обрабатывать. Это исключение значит, что вы в своем коде удаляете элемент напрямую из коллекции во время использования итератора. Т.к. в этом случае появляется неопределенность, откуда итератор должен продолжать и какие последующие элементы должен пройти, выбрасываетс исключение. Для корректного изменения сущности используйте методы итератора, например, iterator.remove().

1
  • А если я использую цикл for (T item: expr)?
    – Alex78191
    1 авг 2018 в 15:09
3

Например, можно поставить блокировку перед началом модификации ArrayList и снять блокировку после завершения модификации. Используете неконкурентную структуру данных ArrayList. Такая сруктура данных будет корректно работать если поток один.

    while (i.hasNext()) {
        synchronized (i) {
            Enemy e = i.next();
            if (e.x >= 1600 || e.x <= -1600) {
                i.remove();
            } else {
                e.move(); // поправить позже
                g.drawImage(e.img, e.x, e.y, null);
            }
        }
    }

В этом коде блокировка снимается позже завершения модификации. Можно код переписать так:

    while (i.hasNext()) {
        boolean isDrawImage = false;
        synchronized (i) {
            Enemy e = i.next();
            if (e.x >= 1600 || e.x <= -1600) {
                i.remove();
            } else {
                e.move(); // поправить позже
                isDrawImage = true;
            }
        }
        if (isDrawImage)
           g.drawImage(e.img, e.x, e.y, null);
    }
0

В дополнение к ответу etki привожу код для корректного удаления элемента коллекции внутри цикла итератора:

Iterator<String> i = strings.iterator();
while (i.hasNext()) {
    String value = i.next(); // должен быть вызван перед тем, как вызывается i.remove()
    // какой-то код
    i.remove(); // Элемент value был удалён из коллекции strings
}

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.