Discussion :

[patril]

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 :

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

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

[batou22003]

Salut.

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

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

[patril]

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

Code :

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

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

[joel.drigo]

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 :

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

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 :

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

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 :

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

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 :

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

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 :

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

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

[Lorantus]

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

Code :

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

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

devient

Code :

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

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

[thelvin]

Ou peut-être simplement :

Code :

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

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