Discussion :

[cyrodil]

Bonjour a tous,

Mon exercise me donne un faut code que je dois corriger. Je dois retirer d'un set tous les etudiants ayant un resultat de -1.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
public void removeFails() // cette methode est incorrecte
   {
      int resultsExam;
      for (String name: students.keySet())
      {
         resultsExam = this.students.get(name).getExamMark();
         if (resultsExam == -1)
         {
            students.remove(name);
         }
      }

Maintenant je sais que l'on ne peut pas utiliser la method remove() sur un key.Set(), mais je ne vois pas comment approcher le problem differement

[eric39]

Bonjour,

Pour pouvoir supprimer des éléments d'une Map, il faut utiliser un Iterator.

Code :

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

1
2
3
4
5
6
 
	Iterator<String> it = students.keySet().iterator();
	while (it.hasNext()) {
		String name = it.next();
		....
	}

[Invité]

Salut,

Bonjour,

Pour pouvoir supprimer des éléments d'une Map, il faut utiliser un Iterator.

Code :

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

1
2
3
4
5
6
 
	Iterator<String> it = students.keySet().iterator();
	while (it.hasNext()) {
		String name = it.next();
		....
	}

Il utilise le for étendu, ce qui revient à utiliser l'itérateur non ?
Je pense surtout que le problème ici est un ConcurrentModificationException. Tu peux utiliser une ConcurrentMap ou alors reconstruire une nouvelle Map au fur et à mesure.

A plus

[eric39]

Salut,

Salut,



Il utilise le for étendu, ce qui revient à utiliser l'itérateur non ?
Je pense surtout que le problème ici est un ConcurrentModificationException. Tu peux utiliser une ConcurrentMap ou alors reconstruire une nouvelle Map au fur et à mesure.

Je suis d'accord pour le for étendu.
En effet, le problème est bien une ConcurrentModificationException, mais l'Iterator solutionne ce genre de problème.
J'aurais du ajouter la totalité du code:

Code :

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

1
2
3
4
5
6
7
8
9
 
Iterator<String> it = students.keySet().iterator();
while (it.hasNext()) {
	String name = it.next();
	resultsExam = this.students.get(name).getExamMark();
	if (resultsExam == -1) {
		it.remove();
	}
}

On peut comme celà accéder à l'intance de l'Iterator et faire un remove sur l'Iterator ce qui solutionne la ConcurrentModificationException.

[Invité]

Ok si tu utilises le remove de l'iterator et pas celui de la Map...

[cyrodil]

Merci a tous pour votre reponse

Je suis desole j'ai oublier de preciser que dans l'exercice ils precisent de garder le foreach loop