Boucle HashMap paramétrée

Bonjour,

Je définis une HashMap paramétrée de la sorte :

Code:

---

HashMap<String,ArrayList<String>> personnes = new HashMap<String,ArrayList<String>>();

---

Jusque là, tout va bien. Après l'avoir remplis, j'ai une partie du code qui boucle sur les clés/valeurs de cette HashMap, mais j'ai un problème :

Code:

---

1 2 3 4 5 6 7 8

for ( Iterator iter = personnes.entrySet().iterator(); iter.hasNext(); ) { Map.Entry ent = (Map.Entry) iter.next(); //La clé de la HashMap String idPerson = (String)ent.getKey(); //La Valeur de la HashMap ArrayList<String> valeur = (ArrayList<String>) ent.getValue(); //Traitement }

---

Je suis obligé de laisser les cast, provoquant donc un warning pour "(ArrayList<String>)". Où ai-je raté une étape ?

Salut,

Citation:

---

Envoyé par JohnNC

Où ai-je raté une étape ?

---

L'erreur vient simplement du fait que tu n'utilises pas de paramétrage sur l'Iterator. Le compilateur ne peut donc pas vérifier la cohérence des types, ce qui amène aux casts et aux warnings...

Ton code devrait plutôt ressembler à cela :

Code:

---

1 2 3 4 5 6 7 8

for (Iterator<Map.Entry<String, ArrayList<String>>> iter = personnes.entrySet().iterator(); iter.hasNext();) { Map.Entry<String, ArrayList<String>> ent = iter.next(); // La clé de la HashMap String idPerson = ent.getKey(); // La Valeur de la HashMap ArrayList<String> valeur = ent.getValue(); // Traitement }

---

Quoique je te conseillerais d'utiliser la boucle for étendu de Java 5.0 :

Code:

---

1 2 3 4 5 6 7

for (Map.Entry<String, ArrayList<String>> entry : personnes.entrySet()) { // La clé de la HashMap String idPerson = entry.getKey(); // La Valeur de la HashMap ArrayList<String> valeur = entry.getValue(); // Traitement }

---

Et j'aurais même tendance à te conseiller de faire un maximum d'abstraction des implémentation. C'est à dire d'utiliser les interfaces Map et List à la place des types concret HashMap et ArrayList :

Code:

---

1 2 3 4 5 6 7 8 9

Map<String, List<String>> personnes = new HashMap<String, List<String>>();   for (Map.Entry<String, List<String>> entry : personnes.entrySet()) { // La clé de la HashMap String idPerson = entry.getKey(); // La Valeur de la HashMap List<String> valeur = entry.getValue(); // Traitement }

---

a++

Ha merci ! Je n'avais pas pensé à l'itérateur, puisque l'info-bulle d'eclipse me donnait bien les paramètres :aie: Du coup, je vais prendre ta seconde solution.

Par contre, pour la troisième (utiliser les interfaces au lieu des types concret), j'ai une question. Dans mes recherches, j'ai lu un post où celà posait problème. Je ne retrouve plus le post exact, où le problème était bien détaillé, mais j'en ai un qui ressemble sur ce post.
Je cite :

Citation:

---

Envoyé par gronono

Bonjour,

Lorsque tu fais A a1 = new A1() la variable a1 est du type A. Donc la méthode appelée est transform(A, Object).

Par contre si tu déclares A1 a1 = new A1() la variable a1 est du type A1. Donc la méthode appelée est transform(A1, Object).

Si si tu déclares A a1 = new A1() et que tu appelle transform par transform(a1, "test") alors tu force le type de a1 en A1.

Dans le premier cas, la JVM considère que a1 est de type A
Dans le second, de type A1 (il est déclaré explicitement)
Dans le troisième de type A1 (c'est toi qui force)

L'important est le type de la déclaration et non le type de l'instance.

Il existe des exemples dans l'API standard:
soit une ArrayList l.
Si tu la déclare ArrayList l = new ArrayList(), tu peux utiliser la méthode add(int index, E element)
Par contre si tu la déclare Collection l = new ArrayList(), tu ne peux utiliser la méthode add(int index, E element) .

---

Est-ce que je risque pas d'avoir des erreurs toutes bêtes (genre une méthode que je ne retrouve pas, car non présente dans l'interface) ? (Autrement dis, comment tu les gères ?)

Il y a quelque cas particulier où l'on a besoin du type précis... mais pas dans 95% des cas !


En particulier pour les collections, où les interfaces sont plutôt complète.

Ton code est super réducteur : tu ne peux utiliser qu'une HashMap et qu'une ArrayList. Or il existe un grand nombre de Map et de List que tu pourrais utiliser...

a++

D'accord, merci pour l'info ! Quitte à passer sur JAVA 5, je vais en profiter pour approfondir mes connaissances et utilisations :king: