pętla na liście z usunięciem

Question

for (String fruit : list) 
    { 
     if("banane".equals(fruit)) 
      list.remove(fruit); 
     System.out.println(fruit); 
    } 

Tutaj pętla z instrukcją usuwania. w czasie wykonywania, ja trochę ConcurrentModificationException, poniżej wyjścia konsoli:

Exception in thread "main" java.util.ConcurrentModificationException 
at java.util.AbstractList$Itr.checkForComodification(AbstractList.java:449) 
at java.util.AbstractList$Itr.next(AbstractList.java:420) 
at Boucle.main(Boucle.java:14) 
abricot 
banane 

pytanie: Jak usunąć jakiś element z pętli?

Answer 1

Musisz użyć iteratora bezpośrednio i usunąć element przez ten iterator.

for (Iterator<String> iterator = list.iterator(); iterator.hasNext();) { 
    String fruit = iterator.next(); 
    if ("banane".equals(fruit)) { 
     iterator.remove(); 
    } 
    System.out.println(fruit); 
} 

Answer 2

Użyj pętli for i zapętlić kolekcję w odwrotnej kolejności. (Oznacza to, że zacznij od ostatniego elementu i zapętlaj do pierwszego elementu.) W ten sposób nie będziesz miał problemów z indeksami, które zmieniają się z powodu usuwania elementów z kolekcji. przykład, że piszesz, bo lista, na której twoi iteracje iteracyjnej, uległa zmianie, co oznacza, że ​​iterator staje się nieważny.

Answer 3

for(Iterator<String> iter = list.iterator(); iter.hasNext();) 
{ 
    String fruit = iter.next(); 
    if("banana".equals(fruit)) 
     iter.remove(); 
    System.out.println(fruit); 
} 

Answer 4

Poza pomocą Iterator bezpośrednio (co polecam) można również sklep elementy, które chcesz usunąć na innej liście.

List<String> toRemove = new ArrayList<String>(); 
for (String fruit : list) { 
    if ("banane".equals(fruit)) 
     toRemove.add(fruit); 
    System.out.println(fruit); 
} 
for (String fruit : toRemove) { 
    list.remove(fruit); 
} 

Pamiętaj, nie polecam tego, to tylko alternatywa. :)

Answer 5

Podobne do sugestii Bombe, ale w mniej wierszy kodu przez iterowanie na liście kopii, ale usunięcie z oryginalnej listy;

List<String> temp = new ArrayList<String>(list); 
for (String fruit : temp) 
{ 
    if("banane".equals(fruit)) 
     list.remove(fruit); 
    System.out.println(fruit); 
} 

Osobiście uważam, że wygląda ładniej niż iteracja z iteratorem.

Answer 6

Wydaje się to nieco skomplikowane, dlaczego nie po prostu zrobić zwykłą pętlę? Wydaje mi się, że wygląda na czystsze i nie rzuci tego błędu. Po prostu odrzuć i jeśli coś usuniesz. przynajmniej moje prace. Te rodzaje automatycznych pętli są bardziej wygodne dla kodowania, pomyślałem, więc jeśli nie są wygodne, to po prostu ich nie używaj.

for (int i = list.size() - 1; i>=0; i--) { 
    String fruit = list.get(i); 
    System.out.println(fruit); 

    if ("banane".equals(fruit)) { 
     list.remove(fruit); 
    } 
} 

Answer 7

ArrayList<String> list = new ArrayList<String>(Arrays.asList("a", "b", "c", "d")); 
Iterator<String> iter = list.iterator(); 
while (iter.hasNext()) { 
    String s = iter.next(); 

    if (s.equals("a")) { 
     iter.remove(); 
    } 
} 

jest najlepszym rozwiązaniem ..