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

[Invité]

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

Alors dans ce cas, utilise une des deux possibilités que j'ai évoquées, ceci dit la solution de eric39 est aussi possible que tu pourras recycler pour une utilisation ultérieure

[cyrodil]

Je n'ai pas encore etudier concurrentMap, j'ai regarder sur le net et j'ai essaye d'implementer mon code le mieux possible:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
public void removeDropouts() 
   {
       Map<String, Integer> removeD = new HashMap<String, Integer>();
       for (String name: students.keySet())
      {
        if (removeD.get(name).equals(-1)) 
         {
            removeD.remove(name);
         }
      }
   }

J'ai ajoute aussi java.util.concurrent.ConcurrentMap; a ma class, mais toujours rien

[eric39]

Il faut utiliser une ConcurrentHashMap à la place d'une HashMap:

Code :

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

1
2
 
Map<String, Integer> removeD = new ConcurrentHashMap<String, Integer>();

[cyrodil]

Map<String, Integer> removeD = new ConcurrentHashMap<String, Integer>();
for (String name: students.keySet())
{
if (removeD.get(name).equals(-1))
{
removeD.remove(name);
}
}

Thanks eric39, I changed that, the student with -1 score is still on the list

[eric39]

Euh, tu es sur qu'il n'y a pas une autre chose quelque part?
Parce que le mécanisme est le bon...

[eric39]

En fait, dans le code que tu as donné,
tu supprimes le éléments de la Map 'removeD', mais tu boucles sur 'students'.!!
Remplace 'students' par 'removeD' dans ta boucle.

[cyrodil]

Je n'ai rien ajouter dans le constructor et rien dans les instances variables. Faut-il peut-etre que je declare removeD.

Aussi l'exercise me donne le test pour la methode:

TutorGroup t = new TutorGroup();
t.populate();
t.displayTutorGroup();
t.removeDropouts();

Ils fonctionnent tous sauf le dernier

[cyrodil]

J'ai remplacer la boucle students par removeB:

Code :

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

1
2
3
4
5
6
7
8
9
 
Map<String, Integer> removeD = new ConcurrentHashMap<String, Integer>();
      for (String name: removeD.keySet())
      {
         if (removeD.get(name).equals(-1)) 
         {
            removeD.remove(name);
         }
      }

Cet etudiant est accroche a l'ecran, il veut pas partir

[eric39]

Comment tu ajoutes les informations dans ta Map removeD?

[Invité]

Ce que je ne comprends pas dans ton code, c'est que tu instancies removeD, sans le remplir et tu itéres dessus, donc ça ne fait rien. Ce que je voulais dire, c'est de changer le type de students (si tu le peux). Au moment de l'instancier au lieu de faire un

Code :

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

1
2
 
students = new HashMap<String, Integer>();

tu fais

Code :

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

1
2
 
students = new ConcurrentHashMap<String, Integer>();

Et le reste du code ne change pas apr rapport à ce que tu avais au début.
Si tu ne peux pas changer le type de students (contrainte de l'énoncé par exemple...), alors la logique est inverse : tu ne retires rien, tu ajoutes les étudiants qui ont une note > -1 dans une nouvelle HashMap, et tu assignes à la fin students avec cette nouvelle HashMap remplie

EDIT : à moitié grillé

[cyrodil]

J'ai changer pour ConcurrentHashMap, mais ca ne fonctionne pas:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
public void removeDropouts() 
   {
      Map<String, Integer> students = new ConcurrentHashMap<String, Integer>();
      int resultsExam;
      for (String name: students.keySet())
      {
         resultsExam = this.students.get(name).getExamMark();
         if (resultsExam == -1)
         {
            students.remove(name);
         }
      }

[eric39]

D'après ton code:
Tu as déjà une variable qui s'appelle 'students' qui je pense est une variable d'instance de classe. C'est cette variable de classe qui doit devenir une ConcurrentHashMap.

Dans le code que tu nous donnes, tu boucles sur une map vide!
En gros la déclaration de la variable students dans la méthode ne sert pas.

[cyrodil]

Oui, mon variable est declare comme ceci:

public class TutorGroup
{
/* instance variables */
private SortedMap<String, Student> students;

[DeathMaker]

Tu peux essayer ce bout de code, c'est ce que je fais généralement quand j'ai plusieurs chose à supprimer:

Code :

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

1
2
3
4
5
6
7
8
9
 
HashMap<String, Integer> removeD = new HashMap<String, Integer>();
Set<String> setToRemove = new HashSet<String>();
for (String name : removeD.keySet()) {
	if (removeD.get(name).equals(-1)) {
		setToRemove.add(name);
	}
}
removeD.keySet().removeAll(setToRemove);

A+