supprimer un élément d'une liste a l'aide de Iterator

Salut tous le monde :)
je parcours ma liste avec une expression régulière et je veux qu'à chaque fois qu'il trouve le mot correspondant à l'expression régulière, il le supprime de la liste

Code:

---

1 2	String[] items = chaine.split("\\s+"); List<String> a11 = Arrays.asList(items);

---

Code:

---

1 2 3 4 5 6 7 8 9 10

Pattern t = Pattern.compile("^[*/](.*)[*/]$"); for (int i = 0; i < a11.size(); i++) { String o = (String) a11.get(i);   if (t.matcher(o).find()){   a99.add(o);   }}

---

Bonjour

Je ne vais pas rentrer dans le détail de ton code, mais l'exemple suivant t'aidera

Code:

---

1 2 3 4 5 6 7

for(Iterator<String> it = collection.iterator(); it.hasNext();) { String str = it.next(); if(ta condition) { it.remove(); } }

---

Dans ton code, tu ne peux pas supprimer un element via un iterateur si tu parcour ta boucle avec un indice !
Attention certaines collection peuvent fournir des iterator qui ne supporte pas la fonction de suppression. Dans ce cas une exception est levée lors du remove (cf. Javadoc Iterator)

Citation:

---

Envoyé par EyZox

Attention certaines collection peuvent fournir des iterator qui ne supporte pas la fonction de suppression. Dans ce cas une exception est levée lors du remove (cf. Javadoc Iterator)

---

C'est d'ailleurs le cas de Arrays.asList(), qui sert à voir un tableau à travers l'interface d'une liste. Comme un tableau est de taille fixe, il n'est pas possible de faire en sorte qu'il ait un élément de moins.

donc je fais comment pour supprimé l’élément trouvé de ma liste ?

Au lieu de créer ta liste comme ça List<String> a11 = Arrays.asList(items);, fais :

• soit List<String> a11 = new ArrayList<>(Arrays.asList(items));
• soit List<String> a11 = Arrays.stream(items).collect(Collectors.toList()); en Java 8

Ce qui te permettra d'utiliser la boucle indiquée par @EyZox.

Autre solution, au lieu de mettre tous les items dans une liste, puis supprimer certains, ajouter seulement les autres (ceux qu'on ne veux pas supprimer) :

Code:

---

1 2 3 4 5 6

List<String> al1 = new ArrayList<>(); for(String item : items) { if ( <ta condition sur item> ) { // la condition qui dit qu'on garde l'élément (l'inverse de celle qui dit qu'on le supprime) al1.add(item); } }

---

On peut aussi faire en Java 8 :
List<String> a11 = Arrays.stream(items).filter(item-> <ta condition sur item>).collect(Collectors.toList());

ça marche
j'ai déclaré la liste a11 comme tu ma dit

Code:

---

1 2	List <String> a11 = new ArrayList <>(Arrays.asList(items));

---

et j'ai utiliser le .remove pour la suppression

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

Pattern t = Pattern.compile("^[/*](.*)[*/]$|"); for (int i = 0; i < a11.size(); i++) { String o = (String) a11.get(i);   if (t.matcher(o).matches()){   a99.add(o); a11.remove(o);   }} System.out.println( "les commentaire"); for( int i = 0; i < a99.size(); i++) { System.out.println( a99.get(i)); }

---

Merci Joel :)

Euh, non. Déjà, ce que je t'ai indiqué c'était dans l'hypothèque où tu utiliserais un Iterator et sa méthode remove(). Ensuite, ta boucle peut fonctionner dans le principe, sauf que là, tu sautes des éléments (tous les éléments qui sont juste après un élément retiré), parce que tu incrémentes l'index systématiquement, alors qu'il ne faudrait le faire que quand tu retires pas l'élement courant !

Solution à base d'Iterator :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

for (Iterator<String> iter = a11.iterator(); iter.hasNext()); ) { String o = iter.next();   if (t.matcher(o).matches()){   a99.add(o); iter.remove();   }   }

---

Solution à base de for :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

for (int i = 0; i < a11.size(); ) { String o = (String) a11.get(i);   if (t.matcher(o).matches()){   a99.add(o); a11.remove(o);   } else { i++; }   }

---

Il y a éventuellement cette solution, si a99 n'est remplie que dans cette boucle (nouvelle/vide avant) :

Code:

---

1 2 3 4 5 6 7 8

for (String o : a11 ) {   if (t.matcher(o).matches()) { a99.add(o); }   } al1.removeAll(a99);

---