Fehler beim löschen eines Arraylist-element

[dTP]

Hallo
habe ein Problem beim löschen einer Person bei meinem Adressbuch
die Personen sind objekte in einer Arraylist
diese Methode kann zwar elemente löschen,
aber nicht das erste und auch nicht das letzte
da kommt eine fehlermeldung:

Exception in thread "main" java.util.ConcurrentModificationException
at java.util.AbstractList$Itr.checkForComodification(AbstractList.java:448)
at java.util.AbstractList$Itr.next(AbstractList.java:419)

die erase Methode sieht folgendermaßen aus:

public void erasePerson(String name)
{
int i=0;
Person x;
Iterator it = personen.iterator();
while(it.hasNext())
{
x = (Person)it.next();
if(x.getNachname().equals(name))
{
personen.remove(i);
}
i++;
}
}

 

[foobar]

Warum deklarierst du das Objekt Person ausserhalb der While-Schleife?

public void erasePerson(String name)
{
Iterator it = personen.iterator();
while(it.hasNext())
{
Person x = (Person)it.next();
if(x.getNachname().equals(name))
{
  it.remove();
}
}
}

 

[bygones]

das mit dem Personen Objekt ist doch in Ordnung....
das Problem ist, dass man nicht die Liste ändern darf, während man über sie iteriert !

also, wie foobar richtig sagt, das löschen nur über den iterator erledigen

 

[foobar]

das mit dem Personen Objekt ist doch in Ordnung....

Es ist zwar nicht direkt ein Fehler, aber auch nicht besonders schön. Bei der Wahl des Scopes gilt wie immer: So viel wie nötig, so wenig wie möglich ;-)

 

[dTP]

verstehe ich such mir also mit dem Iterator das zulöschende Objekt
und lösche es erst danach..
wenn ich das richtig verstanden habe.

und wegen dem Peronen objekt namens x
wenn ich es ausserhalb der schleife deklariere,
dann wird es ja nur einmal erzeugt
und in der schleife wird dann jedesmal das passende zugewiesen
wenn ich das Person Objekt x in der schleife deklaliere
so wie du es vorgeschlagen hast,
dann wird ja bei jedem Durchgang ein Objekt x deklariert und zugeordnet
ist meine Methode nicht schneller?
da das x ja nur einmal deklariert wird und in der schleife nur mehr zugeorndet wird???

 

[bygones]

verstehe ich such mir also mit dem Iterator das zulöschende Objekt
und lösche es erst danach..
wenn ich das richtig verstanden habe.

nicht ganz - du iterierst über die liste und schaust welches element du löschen willst. dann löschst du es über den iterator und nicht direkt über die liste !!