ConcurrentModificationException

[Evolver]

Ich habe in meinem Programm eine Liste von Objekten, konkret eine ArrayList. Innerhalb eines Threads wird die Liste immer wieder mit einem Iterator durchlaufen, manchmal werden auch Elemente entfernt. Nun ist neu hinzu gekommen, dass über Benutzereingaben (MouseListener) zur Liste neue Elemente hinzugefügt werden können.

Jetzt tritt manchmal beim Durchlaufen der Liste eine java.util.ConcurrentModificationException auf.
1. Ist das so, weil die ArrayList nicht threadsafe ist?
2. Verhindere ich dieses Problem endgültig durch Benutzung von von Vector anstelle der ArrayList?

 

[tfa]

1. Ist das so, weil die ArrayList nicht threadsafe ist?
2. Verhindere ich dieses Problem endgültig durch Benutzung von von Vector anstelle der ArrayList?

Nein, tust du nicht. Sobald ein Iterator die Liste durchläuft und sie von einem anderen (oder dem auch dem selben)Thread aus geändert wird, kommt diese Exception. Du könntest die Collection kopieren, bevor du den Iterator erzeugst.
Da du aber mit mehreren Threads arbeitest brauchst du sowieso eine Art Synchronisation, z.B. Vector oder Collections.synchronizedList(..).

 

[maki]

Oder du änderst die Collection ausschliesslich über den iterator.

Iterator bietet nur remove() an, ListIterator kann auch add(Object).

 

[Marco13]

Auch bei einem Vector (oder besser: einer Collections.synchronizedList) wirst du synchronisation brauchen, wenn du einen Iterator (d.h. auch die neue foreach-Schleife) verwendest.... So, wie es auch in der Doku zu Collections.synchronizedList steht:

// Thread 1:
    list.add(new Element());
    list.remove(3);

// Thread 2:
    synchronzied(list)  // <--- Wischtisch
    {
          for (Element element : list) element.doSomething();
    }

 

[JavaFred]

synchronized nur in einem Thread zu verwenden bringt gar nichts. Auch Thread 1 muss Zugriffe auf den Vector in einem synchronized-Block realisieren!

 

[Marco13]

Ich habe ja von einem Vector oder einer synchronizedList gesprochen. Da sind die Zugriffe automatisch auf die 'list' selbst synchronized. Bei einem Iterator muss man aber trotzdem(!) noch explizit den synchronized-Block (eben AUCH auf die 'list') drumrumschreiben.

 

[Evolver]

Also ich habe es jetzt bei einer ArrayList belassen und jeden zugriff darauf in einen synchronized-Block gepackt. Das Problem sollte jetzt also nichtmehr auftreten, wenn ich eure Antworten richtig gedeutet habe. Ja?

Aber das erscheint mir bissel umständlich. Könnte ich nicht irgendwie zentral festlegen, dass jeder Zugriff darauf nur synchonized erfolgt?