Discussion :

[sonia5]

bonjour,

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

Le premier:

Code :

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

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

Le deuxième:

Code :

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

public static Map maMap = new HashMap();

Laquelle c'est la bonne déclaration?


Et voici le traitement pour initialiser Map:

Code :

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

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

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

Code :

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

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

[Rei Ichido]

Généralement il vaut mieux faire ceci :

Code :

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

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

[thelvin]

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

Code :

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

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.

[Robin56]

Première étape :

Code :

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

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 :

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

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 :

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

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

[sonia5]

Merci pour vos réponses

Mais pour cette déclaration

Code :

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

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 :

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

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

[MonsieurSmith]

Merci pour vos réponses

Mais pour cette déclaration

Code :

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

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.

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.

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 :

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

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 :

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

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

[sonia5]

Merci,
Je pense pour ce test:

Code :

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

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

On doit le remplacer par

Code :

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

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

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

[thelvin]

Mais pour cette déclaration

Code :

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

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.

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 :

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

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

[sonia5]

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 :

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

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

[MonsieurSmith]

Merci,
Je pense pour ce test:

Code :

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

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

On doit le remplacer par

Code :

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

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

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

[MonsieurSmith]

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 :

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

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

[sonia5]

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

Code :

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

1
2
3
4
 
if(maMap.get(cle) == null)
{maMap.put(cle,val);
}

ou bien

Code :

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

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

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

[MonsieurSmith]

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 :

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

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 :

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

!maMap.containsKey(cle)

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

[MonsieurSmith]

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

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