Discussion :

[padraig29]

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

[dinobogan]

Il ne faut pas passer par un Iterator :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[Galevsky]

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'

[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.

[dinobogan]

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.

[adiGuba]

Salut,

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

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++

[turbo_chess]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
 
Collection<Salariesalarie> salarieList = null;
for (Salariesalarie  salarie : salarieList) {
    if (est_a_supprimer(salarie)) {
      salarieList.remove(salarie);
   }
}