Utiliser une Map pour stocker des informations

bonjour,

Je veux utiliser un Map pour stocker des informations et voici deux traitements possibles pour déclarer Map:

Le premier:

Code:

---

public static HashMap<String,String> maMap = new HashMap<String,String>();

---

Le deuxième:

Code:

---

public static Map maMap = new HashMap();

---

Laquelle c'est la bonne déclaration?


Et voici le traitement pour initialiser Map:

Code:

---

1 2 3

maMap.put("cle1","val1"); maMap.put("cle2","val2");

---

Traitement pour récupérer les données ou bien parcourir Map:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

String valeur; String key; Iterator it; it= maMap.keySet().iterator(); while (it.hasNext) { key = (String)it.next() System.out.println(key); valeur= maMap.get(key); System.out.println(valeur); }

---

Est ce que mon code est bien ou bien il y a la possibilité d'optimiser mon code ?
Merci d'avance

Généralement il vaut mieux faire ceci :

Code:

---

1 2	public static Map<String,String> maMap = new HashMap<String,String>();

---

Sinon, si tu veux parcourir ta Map, oui tu dois passer par un itérateur pas très joli ; en même temps la Map n'est pas spécialement faite pour être ainsi parcourue (ce n'est pas son objectif premier ^^).

Tout de même, pour le parcours on peut faire ça :

Code:

---

1 2 3 4

for(Map.Entry<String, String> entry : map.entrySet()) { System.out.println(entry.getKey()); System.out.println(entry.getValue()); }

---

Ça évite de devoir résoudre toutes les clés, et puis ça évite aussi de se taper soi-même l'itérateur.

Première étape :

Code:

---

1 2	HashMap map = new HashMap(); Map map = new HashMap();

---

Il n'y a pas de différence mis à part que map est en type plus générique sur la deuxième proposition. Il est possible de la réutiliser pour un autre type proche :

Code:

---

map = new TreeMap();

---

Ceci ne fonctionnerait que dans le deuxième cas. Donc en général, il faut déclarer les variables à l'aide d'interface qui rende le code plus souple. Donc utilise la deuxième notation.

Deuxième étape :

Code:

---

1 2	Map<String,String> map = new HashMap<String,String>(); Map map = new HashMap();

---

La première proposition type ses valeurs, c'est préférable si l'on est sur du type de son contenu. D'après ton exemple, tu sembles sur que ta Map contiendra des String donc utilise la première. La deuxième proposition te permet d'insérer à peu près n'importe quoi dans ta Map.

Pour les bonnes façons de parcourir une Map : Cf. ici

Merci pour vos réponses

Mais pour cette déclaration

Code:

---

1 2	Map<String,String> map = new HashMap<String,String>();

---

Elle n’a pas admis pas le compilateur 1.4
J’ai une autre question si on a le compilateur 1.4 alors on travaille par Java4 et pour 1.5 c’est de java 5 ?

Et ce que je peux faire ce traitement, pour voir si la cle n’existe pas avec sa valeur avec j’insère dans Map

Code:

---

1 2 3

If(maMap.get(cle) != null) {maMap.put(cle,val); }

---

Citation:

---

Envoyé par sonia5

Merci pour vos réponses

Mais pour cette déclaration

Code:

---

1 2	Map<String,String> map = new HashMap<String,String>();

---

Elle n’a pas admis pas le compilateur 1.4

---

Normal. Les « generics » (la syntaxe <*>) ne sont valides que pour les versions de Java à partir de 1.5.

Citation:

---

Envoyé par sonia5

J’ai une autre question si on a le compilateur 1.4 alors on travaille par Java4 et pour 1.5 c’est de java 5 ?

---

« Java 4 » n'existe pas : on ne parle que de Java 1.4. Les dénominations en raccourci « Java 5/6/7... » ne sont utilisées que depuis Java 1.5.

Citation:

---

Envoyé par sonia5

Et ce que je peux faire ce traitement, pour voir si la cle n’existe pas avec sa valeur avec j’insère dans Map

Code:

---

1 2 3

If(maMap.get(cle) != null) {maMap.put(cle,val); }

---

---

« if » doit rester tout en minuscules. À part ça, pour tester l'existence de la clé, il y a plus fiable que « if(maMap.get(cle) != null) » : la méthode Map.containsKey() :

Code:

---

1 2 3

if (maMap.containsKey(cle)) { maMap.put(cle, val); }

---

Merci,
Je pense pour ce test:

Code:

---

1 2 3

if (maMap.containsKey(cle)) { maMap.put(cle, val); }

---

On doit le remplacer par

Code:

---

1 2 3

if (!maMap.containsKey(cle)) { maMap.put(cle, val); }

---

c a d, si il n existe pas on insère

Citation:

---

Envoyé par sonia5

Mais pour cette déclaration

Code:

---

1 2	Map<String,String> map = new HashMap<String,String>();

---

Elle n’a pas admis pas le compilateur 1.4

---

Non, en effet. Mais 1.4 est complètement obsolète, on en est à 1.7 maintenant. Si ça ne t'est pas absolument impossible, tu dois vraiment passer à une version supérieure. Idéalement 1.6 ou 1.7, en fonction de si tu te sens pionnier ou pas.

Citation:

---

Envoyé par sonia5

Et ce que je peux faire ce traitement, pour voir si la cle n’existe pas avec sa valeur avec j’insère dans Map

Code:

---

1 2 3

If(maMap.get(cle) != null) {maMap.put(cle,val); }

---

---

Ça me semble un peu contradictoire. Tu peux nous dire ce que tu essaies de faire en bon français clair ? (Et aussi dans quel but, au cas où...)

D'accord,

J'essaye d’insérer la valeur ou cas ou je ne trouve pas sa clé?

Et je pense la meilleur solution :

Code:

---

1 2 3

if (!maMap.containsKey(cle)) { maMap.put(cle, val); }

---

Citation:

---

Envoyé par sonia5

Merci,
Je pense pour ce test:

Code:

---

1 2 3

if (maMap.containsKey(cle)) { maMap.put(cle, val); }

---

On doit le remplacer par

Code:

---

1 2 3

if (!maMap.containsKey(cle)) { maMap.put(cle, val); }

---

c a d, si il n existe pas on insère

---

Au temps pour moi : si j'avais été plus attentif à l'énoncé (comme l'a été Thelvin), j'aurais vu la confusion dans ton code et ne l'aurais pas répercutée :aie:

Pourquoi
Je veux voir si l 'élément n’existe pas dans ma Map, alors je l’insère sachant que pour chaque valeur à une clé unique
Ou se trouve contradictoire?

Citation:

---

Envoyé par sonia5

Pourquoi
Je veux voir si l 'élément n’existe pas dans ma Map, alors je l’insère sachant que pour chaque valeur à une clé unique
Ou se trouve contradictoire?

---

Dans le fait que ton premier traitement :

Code:

---

1 2 3

If(maMap.get(cle) != null) {maMap.put(cle,val); }

---

donnait ceci : si la clé était présente dans la Map et non "null", alors on lui affectait "val"... Ce qui n'est pas vraiment la même chose :aie:

Désolé, c'est vrai vous avez raison
Je dois faire :

Code:

---

1 2 3 4

if(maMap.get(cle) == null) {maMap.put(cle,val); }

---

ou bien

Code:

---

1 2 3

if (!maMap.containsKey(cle)) { maMap.put(cle, val); }

---

Mais ce deuxième test est mieux que le premier?

Oui, car :

- si tu consultes l'API, tu liras que l'interface Map et la classe HashMap qui l'implémente autorisent des valeurs "null" (voire la clé "null" !). Ce qui implique que le test

Code:

---

maMap.get(cle) == null

---

peut parfaitement retourner "true" si maMap contient la clé "cle", pourvu qu'elle soit associée à la valeur "null" !

- En revanche, le test

Code:

---

!maMap.containsKey(cle)

---

garantit de retourner "true" si et seulement si la clé "cle" est absente de maMap.

Et ne pas oublier de passer le sujet en "Résolu" le cas échéant :mrgreen:

Edit : Je rêvais de la sortir un jour, celle-ci :)