Suppression d'éléments dans une map

Bonjour,
J'ai une question concernant les HashMap, plus particulièrement les map.

J'initialise au départ une map "map" avec 6 éléments et des valeurs d'objets Voiture à null.

puis j'appelle cette méthode

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

public Set<String> getEntries() { Map<String, Voiture> mapCopie = new HashMap<String, Voiture>(); mapCopie.putAll(map); Iterator<String> iterator = mapCopie.keySet().iterator();   while (iterator.hasNext()) { System.out.println("iterateur " + iterator.next().toString());   if (mapCopie.get(iterator) == null) { mapCopie.remove(iterator); }   System.out.println(mapCopie); System.out.println(""); }   return mapCopie.keySet(); }

---

j'ai autant d'éléments après l'appel. pourtant j'aimerais que la map soit vide. Ou est mon erreur ? merci de votre aide

Salut.

Et bien je l'ai fait l'autre jour,

tu fais simplement macopie.clear(); pour vider ta map.

:P

En fait je ne veux pas tout vider systématiquement. Je fais le tri avec la condition

Code:

---

if (mapCopie.get(iterator) == null)

---

Au départ toutes les valeurs sont nulles donc la map devrait être vide mais ensuite je la remplis avec ceratines valeurs non null.

Le clear() efface toute la map et ce n'est pas ce que je voudrais...

Salut,

@patril, ton erreur vient du fait que l'iterateurr ne sert qu'à itérer (sur les clefs, puisqu'il est obtenu à partir du keyset) : il n'est pas, et ne représente pas non plus, une clef dans la map.

Code:

---

1 2 3 4 5 6 7 8 9

Iterator<String> iterator = mapCopie.keySet().iterator();   while (iterator.hasNext()) { System.out.println("iterateur " + iterator.next().toString());   if (mapCopie.get(iterator) == null) { mapCopie.remove(iterator); } }

---

Pour récupérer l'élément courant, on appelle next() : attention, cette méthode consomme l'élément, ce qui signifie que le next() suivant récupérera l'élément suivant, ou, au pire, soulèvera une exception NuSuchElementException (plus d'élément...).
Ensuite, il ne faut jamais (sauf exception d'implémentation prévue pour) modifier le nombre d'éléments d'une collection sur laquelle on itère, sauf par la méthode remove() de Iterator. Tu obtiendras une ConcurrentModificationException.

Donc le code équivalent qui fonctionne est :

Code:

---

1 2 3 4 5 6 7 8 9 10

Iterator<String> iterator = mapCopie.keySet().iterator();   while (iterator.hasNext()) { String key = iterator.next(); System.out.println("iterateur " + key);   if (mapCopie.get(key ) == null) { iterator.remove(); // Note bien ici sur quelle instance on appelle remove() } }

---

Remarque que si tu n'as pas besoin d'afficher, tu pourras faire :

Code:

---

1 2 3 4 5 6 7 8

Iterator<String> iterator = mapCopie.keySet().iterator();   while (iterator.hasNext()) {   if (mapCopie.get(iterator.next()) == null) { iterator.remove(); // Note bien ici sur quelle instance on appelle remove() } }

---

Enfin, lorsqu'on parcourt les clefs d'une map dans le but d'exploiter les valeurs mappées, il est bien plus efficace de parcourir l'entrySet que le keySet :

Code:

---

1 2 3 4 5

for(Iterator<Map.Entry<String,String>> iterator = mapCopie.entrySet().iterator(); iterator.hasNext();) { // le for est préférable au while, pour limiter le scope de la variable iterator if ( iterator.next().getValue()==null ) { iterator.remove(); } }

---

On peut encore faire plus simple, puisque ton code n'exploite que la valeur (et non la clef), en itérant sur les valeurs directement :

Code:

---

1 2 3 4 5

for(Iterator<String> iterator = mapCopie.values().iterator(); iterator.hasNext();) { if ( iterator.next()==null ) { iterator.remove(); } }

---

juste parce que j'ai vu la porte ouverte....

Code:

---

1 2 3 4 5

for(Iterator<String> iterator = mapCopie.values().iterator(); iterator.hasNext();) { if ( iterator.next()==null ) { iterator.remove(); } }

---

devient

Code:

---

mapCopie.values().removeAll(Collections.singleton(null));

---

Ou peut-être simplement :

Code:

---

mapCopie.values().removeIf(e -> e == null);

---