Есть код: ссылка на github. В ходе выполнения выбрасывается исключение Exception in thread "AWT-EventQueue-0" java.util.ConcurrentModificationException
Знающие люди подскажите как побороть проблему. Заранее спасибо)
3 ответа
Его не надо обрабатывать. Это исключение значит, что вы в своем коде удаляете элемент напрямую из коллекции во время использования итератора. Т.к. в этом случае появляется неопределенность, откуда итератор должен продолжать и какие последующие элементы должен пройти, выбрасываетс исключение. Для корректного изменения сущности используйте методы итератора, например, iterator.remove()
.
Например, можно поставить блокировку перед началом модификации 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);
}
В дополнение к ответу etki привожу код для корректного удаления элемента коллекции внутри цикла итератора:
Iterator<String> i = strings.iterator();
while (i.hasNext()) {
String value = i.next(); // должен быть вызван перед тем, как вызывается i.remove()
// какой-то код
i.remove(); // Элемент value был удалён из коллекции strings
}