Atascado con "java.util.ConcurrentModificationException"

linked-list

Question

Aquí está mi código:

// eventList is a LinkedList

public void run() {

    Iterator<Event> it = eventList.iterator();
    int size = eventList.size();

    while(size > 0) {
        while(it.hasNext()) {
            Event e = it.next(); //flaged line

            if(e.ready()) {
                System.out.println(e);
                e.action();
                eventList.remove(e);
                --size;
            }
        }
    }
}

El error java.util.ConcurrentModificationException se arroja a la bandera alineada ( Event e = it.next(); ). ¿Ve un error en mi código que hace obvio el motivo de esa excepción?

Answer 1

Debe eliminar el elemento a través del iterador, de lo contrario, el iterador se restablece porque la colección subyacente cambió.

Answer 2

Está modificando eventList mientras usa eventList.remove() mientras itera sobre él. No debe hacer esto, o el Iterator queda inutilizable.

Simplemente reemplace eventList.remove(e) con it.remove() y debería estar bien.

Además, puede ejecutar fácilmente un bucle sin fin si uno de sus eventos no está listo en la primera ejecución porque it.hasNext() nunca volverá a ser true una vez que haya resultado false , pero el size tampoco se modificará. Una solución sería mover todo el Iterator it = ... línea dentro del primer ciclo while.

También modificaría el bucle while externo para usar while (!e.isEmpty()) lugar de intentar seguir el tamaño de eventList manualmente.