boucle sur liste avec suppression

Question

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

Ici une boucle avec instruction remove. Au moment de l'exécution, je reçois une ConcurrentModificationException, en dessous de la sortie de la console:

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 

Question: Comment faire pour supprimer un élément avec une boucle?

Answer 1

Vous devez utiliser directement l'itérateur et supprimer l'élément via cet itérateur.

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

Answer 2

Utilisez une boucle for et faites une boucle sur la collection dans l'ordre inverse. (Cela veut dire, commencer par le dernier élément, et la boucle au premier élément. Ce faisant, vous ne serez pas avoir des problèmes par les indices qui changent en raison de la suppression d'éléments de la collection.

Vous obtenez l'exception l'exemple que vous publiez, parce que la liste sur laquelle votre itère iterator, a changé, ce qui signifie que le iterator devient invalide.

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

En plus d'utiliser la Iterator directement (que je recommande), vous pouvez également stocker éléments que vous souhaitez supprimer dans une liste différente:

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); 
} 

Rappelez-vous, je ne le recommande pas, c'est juste une alternative. :)

Answer 5

Similaire à ce que Bombe a suggéré, mais en moins de lignes de code en itérant sur la copie de liste, mais en supprimant de la liste originale; Personnellement, je pense que cela semble plus agréable que d'itérer avec un itérateur.

Answer 6

Cela semble un peu compliqué, pourquoi ne pas faire une boucle normale? Je pense que cela semble plus propre et ne va pas jeter cette erreur. Décimale juste si tu enlèves quelque chose. Au moins, la mine fonctionne, de toute façon. Ce genre d'auto-boucles sont plus pour la commodité de codage, je pensais, donc si elles ne sont pas pratiques, alors ne les utilisez pas.

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(); 
    } 
} 

est la meilleure approche ..