ArrayList ConcurrentModificationException

[simons]

Guten Tag

Ich habe ein kleines Problem bei meinem Programm welches ich gerade am schreiben bin.

Man kann ein Produkt in einen Warenkorb legen. Das Objekt wird in ein ArrayList gespeichert. Die Anzahl welche gekauft wurde in eine andere ArrayList.

Nun zu meinem Problem:

Ich habe eine Methode megreProducts in der Klasse Warenkorb. Diese Methode soll Prdukte mit dem gleichen Code "zusammenfügen" (die Anzahl zusammenzählen und einen Eintrag in der ArrayListe löschen).

Code:

public void mergeProducts()
	{
		String code = null;
		int anz = 0;
		int pos = 0;
		int pos2 = 0;
		int oldpos = 0;
		
		
		for(Fakturierbar f : dbw)
		{
		code = f.getCode();
		pos = oldpos;
		anz = anzahl.get(pos).intValue();
		pos += 1;
		oldpos = pos;
			for(pos2 = pos; pos2 < dbw.size() ; pos2++)
			{
				if(dbw.get(pos2).getCode().equals(code))
				{
					anz += anzahl.get(pos2).intValue();
					anzahl.set(oldpos - 1, anz);
					dbw.remove(pos2);
					anzahl.remove(pos2);
					mergeProducts();
				}
			}
		}
	}

Dies gibt mit nun jedoch die Exception java.util.ConcurrentModificationException aus, was ja noch logisch ist, da ich ein Element aus dem Array lösche. Ich dachte jedoch, dass es klappen sollte, wenn ich die Methode einfach neu aufrufe. Klappt leider auch nicht. Bin nun mit meinem Lautein am Ende..

Hoffe Ihr habt mir einige gute Ideen wie ich das lösen kann.

Besten Dank
MfG
Simon

 

[maki]

Du iterierst über dbw, dann löscht du Einträge aus dbw, und benutzt denn gleichen Iterator weiter.
Der Iterator merkt das sich die Collection verändert hat seitdem er erzeugt wurde und wirft die Exception.

Ändere deine Collection nicht während du drüber iterierst.

 

[Guest]

Du iterierst über dbw, dann löscht du Einträge aus dbw, und benutzt denn gleichen Iterator weiter.
Der Iterator merkt das sich die Collection verändert hat seitdem er erzeugt wurde und wirft die Exception.

Ändere deine Collection nicht während du drüber iterierst.

ok, danke werde das einmal versuchen.

 

[Wildcard]

Ändere deine Collection nicht während du drüber iterierst.

Ist absolut valide die Collection zu ändern, dazu müssen allerdings die Methoden des Iterators verwendet werden.
Sprich: keine foreach Schleife

 

[maki]

Hast natürlich recht Wildcard, Änderungen nur über den Iterator machen.