Discussion :

[rune93]

Bonsoir,
Je rencontre actuellement un problème avec un HashTable ou HashMap qui fais la correspondance entre une adresse mac et le port auquel il est connecté.
Celà toujours dans le cadre d'une simulation d'un commutateur.
Le problème est que quand je veux get le numéro du port, je me retrouves toujours avec null.

Code :

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

1
2
3
4
5
6
7
8
9
10
 
private Hashtable<MacAdr,Integer> macadr;
macadr = new Hashtable(100);
 
	public synchronized void ajouter(MacAdr a, int port) {
		macadr.put(a, port);
	}
	public synchronized int getport(MacAdr m) {
		return macadr.get(m);
	}

En cherchant la raison du problème, j'ai comme contenu de la HashTable, celà en rajoutant cette source :

Code :

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

1
2
3
4
5
6
7
 
for (Enumeration e = macadr.elements() ; e.hasMoreElements() ;) {
	System.out.println(e.nextElement());
}
for (Enumeration e = macadr.keys() ; e.hasMoreElements() ;) {
	System.out.println(e.nextElement());
}

Qui affiche bien le bon contenu :

1
0
3
2

00:00:00:00:00:01
00:00:00:00:00:00
00:00:00:00:00:03
00:00:00:00:00:02

En faisant macadr.get(MacAdr), je devrais avoir comme valeur de retour le port (en Integer) mais j'ai toujours un null , est-ce bien comme sa qu'on récupère un élément dans une hashtable/hashmap ?

Merci

[®om]

Redéfini la méthode hashCode() de MacAdr.

[adiGuba]

Salut,

Redéfini la méthode hashCode() de MacAdr.

Ainsi que la méthode equals() !

Plus d'info dans la FAQ :

• Pourquoi et comment redéfinir la méthode hashCode() ?
• Pourquoi et comment redéfinir la méthode equals() ?

Il s'agit de deux impératif pour qu'un objet puisse être utilisé en tant que clef d'une Map !

a++

[rune93]

Re,
J'ai réjouté dans la classe MacAdr, le hashCode cependant j'obtiens toujours l'erreur , ai-je bien re-défini la méthode hashCode() ?

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 class MacAdr {
	String address;
 
	public MacAdr(String str) {
		address = str;
	}
 
	public boolean equals(MacAdr mac) {
		return (this.toString().compareTo(mac.toString())==0);
	}
	public int hashCode() {
		return address.hashCode();
	}
	public String toString() {
		return address;
	}
}

Merci

[adiGuba]

ai-je bien re-défini la méthode hashCode() ?

C'est equals() qui n'est pas redéfini equals(MacAdr) n'est pas une redéfinition de equals(Object), mais une surcharge (et donc une autre méthode).

Code :

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

1
2
3
4
5
6
7
public boolean equals(Object other) {
	if (other instance MacAdr) {
		MacAdr mac = (MacAdr) other;
		return (this.toString().compareTo(mac.toString())==0);
	}
	return false;
}

a++

[rune93]

!! il fallait bien re-définir et non surcharger comme je l'ai fait !