problem method remove()

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:

---

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 differement8O

Bonjour,

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

Code:

---

1 2 3 4 5 6

Iterator<String> it = students.keySet().iterator(); while (it.hasNext()) { String name = it.next(); .... }

---

Salut,

Citation:

---

Envoyé par eric39

Bonjour,

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

Code:

---

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

Salut,

Citation:

---

Envoyé par George7

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:

---

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.

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

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:oops:

Citation:

---

Envoyé par 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:oops:

---

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

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:

---

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

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

Code:

---

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

---

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:?

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

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.

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:oops:

J'ai remplacer la boucle students par removeB:

Code:

---

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:aie:

Comment tu ajoutes les informations dans ta Map removeD?

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:

---

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

---

tu fais

Code:

---

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é ;)

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

Code:

---

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

---

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.

Oui, mon variable est declare comme ceci:

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

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

Code:

---

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+

Merci DeathMaker, mais ca ne fonctionne toujours pas.

Citation:

---

Envoyé par cyrodil

Merci DeathMaker, mais ca ne fonctionne toujours pas.

---

Mais encore ? quelle est l'erreur ?

Tu itéres encore sur la Map vide ? Donne nous ton code exact complet

Ton code doit être: (il faut utiliser la Map 'students' déclaré en début de classe)!

Code:

---

1 2 3 4 5 6 7 8 9

Set<String> setToRemove = new HashSet<String>(); for (String name : students.keySet()) { if (students.get(name).equals(-1)) { setToRemove.add(name); } } students.keySet().removeAll(setToRemove);

---

Merci mille fois a tous pour votre aide, apres des heures de prise de tete j'ai finalement compris.

Et merci encore eric39:king::king::king: