Supprimer des élements d'une liste

Bonjour

J'ai une liste de type Arraylist composée de 100 éléments de type objet "salarié" par exemple.

Je souhaite parcourir cette liste et supprimer de temps en tant de objets qui ne correspondent pas à certains critères.
J'ai vu qu'il existait une méthode remove mais apparemment je ne sais pas bien l'utiliser car après quend je vais sur l'élément suivant ça plante.
Je parcours ma liste avec une boucle for
for (Iterator iter=liste.iterator(); iter.hasNext();) {
Salariesalarie= (Salarie) iter.next();
....
Qualqu'un a t'il un exemple ou bien alors est plus façile d'utiliser une 2eme liste pour les objets que je souhaite conserver?
Merci

Il ne faut pas passer par un Iterator :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

for( int i = 0 ; i < liste.size() ; ) { if( est_a_supprimer( liste.get( i ) ) ) { liste.remove( i ); } else { ++i; } }

---

Tu ne peux pas être en train de cycler sur une collection est en même temps lui retirer des éléments.... car tu modifies la taille de la collection, et l'itérateur ira trop loin. Je passe par un clone, une deuxième instance de la liste, qui -elle- peut être modifiée à loisir.

Gal'

Je n'ai pas trop compris ce que tu entends par clone.
Mais la solution que l'on m'a aussi donné c'est de créer une 2eme liste dans laquelle je mettrais (add) les objets qui m'intéressent.
Mais j'aurais bien aimé utiliser cette méthode remove.

Citation:

---

Envoyé par padraig29

Je n'ai pas trop compris ce que tu entends par clone.
Mais la solution que l'on m'a aussi donné c'est de créer une 2eme liste dans laquelle je mettrais (add) les objets qui m'intéressent.
Mais j'aurais bien aimé utiliser cette méthode remove.

---

La méthode que je t'ai donnée ci-dessus fonctionne. Et elle a l'avantage de ne pas obliger à créer une nouvelle liste. En plus elle utilise la méthode "remove", comme tu le souhaites.

Salut,

Citation:

---

Envoyé par dinobogan

Il ne faut pas passer par un Iterator :

---

Au contraire : il FAUT passer par un Iterator, qui doit normalement gérer cela le plus proprement possible (sauf implémentation spécifique). De plus cela évites de faire des manipulations sur les index (qui donne de très mauvaise perfs sur les LinkedList par exemple) :

Code:

---

1 2 3 4 5 6 7

Iterator<TonType> it = liste.iterator(); while (it.hasNext()) { TonType current = it.next(); if (est_a_supprimer(current)) { it.remove(); } }

---

Citation:

---

Envoyé par padraig29

J'ai vu qu'il existait une méthode remove mais apparemment je ne sais pas bien l'utiliser car après quend je vais sur l'élément suivant ça plante.

---

C'est à dire ?
"ça plante" ne veut rien dire et ne t'apportera aucune aide.
Si ton application plante tu devrais avoir une exception et donc un stacktrace qui permettront d'en trouver la cause précise, et non pas de faire des suppositions...

a++

et si on passe par un for/each , normalement c'est la même chose ?? :roll:

Code:

---

1 2 3 4 5 6 7

Collection<Salariesalarie> salarieList = null; for (Salariesalarie salarie : salarieList) { if (est_a_supprimer(salarie)) { salarieList.remove(salarie); } }

---

Citation:

---

Envoyé par turbo_chess

et si on passe par un for/each , normalement c'est la même chose ?? :roll:

---

Non : tu ne dois pas modifier une liste pendant son parcours, sous peine d'obtenir une ConcurrentModificationException...

a++

Citation:

---

Envoyé par adiGuba

Non : tu ne dois pas modifier une liste pendant son parcours, sous peine d'obtenir une ConcurrentModificationException...

a++

---

c'est bon à savoir.

Merci ;)

Bonjour et merci à tous pour ces infos

J'ai appliqué la méthode de Dinobogan ça fonction bien.
Si j'arrive à avoir du temps sur mon projet( la minute compte !) j'essaierai celle de adiGuba.