Discussion :

[rvfranck]

Salut,
Je veux conserver pour chaque utilisateur de mon site sa session dans une hashmap (mahashmap.put(user,session).
Le problème c'est que je veux savoir si un user a déjà une session et quand je fais un test(hashmap.containsKey(user)) sa marche pas même si user est déjà dans la hashmap.
Comment je fais pour redefinir la méthode de comparaison des objets dans la hashmap? un user à un numero et un pseudo et la comparaison doit se faire sur le numero.

Merci

[adiGuba]

Salut,


Si tu utilises tes propres classes dans une Map, il faut que tu redéfinnisses correctement les méthode equals(Object) et hashCode()...

a++

[rvfranck]

Merci, mais comment?
j'ai redéfini la methode equals de la classe user qui implemente l'interface comparable.

Code :

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

1
2
3
4
5
6
7
 
public boolean [b]equals[/b](Object o1)
{
      User u = (User)o1;
      if (this.numero.compareToIgnoreCase(u.numero)) return true;
      else return false;
}

c'est bon ce code? et je sais pas comment faire pour la methode hashCode(). Est ce qu'il ne faut pas définir la methode compare() aussi?

[bouye]

En retournant numero par exemple ou le haschode du nom de l'utilisateur au choix.

[rvfranck]

je ne comprend pas. vous pourriez pas m'envoyer un peu de code? Merci

[bouye]

Code :

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

1
2
3
4
5
6
 
/** @inheritDoc
*/
@Override public int hashCode() {
  return numero;
}

ou

Code :

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

1
2
3
4
5
6
 
/** @inheritDoc
*/
@Override public int hashCode() {
  return nickname.hashCode(); // Attention si nickname== null !!!!!!!!!!!!!!!!!!
}

[bouye]

Pour etre precis si tu utilises Comparable et si tu as Java 5.0, il te faut ecrire :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
implements Comparable<User>
 
...
 
 
/** @inheritDoc
 */
public int compareTo(User o) {
  return o.numero-this.numero;
}

Mais je ne suis pas sur qu'il soit necessaire d'implementer Comparable pour les Map (lire la doc).

[rvfranck]

Ok, je relis la doc et j'essaie les codes.

[rvfranck]

salut, j'ai essayé ça

public boolean equals(Object o1)
{
User u = (User)o1;
if (this.numero.compareToIgnoreCase(u.numero)) return true;
else return false;
}

public int hashCode() {
return Integer.parseInt(numero);
}

ça marche, parce que le numero des users est un String composé de 6 chiffres dans mon projet. et si le numéro n'était composé que de lettres comment j'aurais pu procéder sachant que je dois retourner un int dans la methode hashCode()?

Merci

[adiGuba]

Salut,

et si le numéro n'était composé que de lettres comment j'aurais pu procéder sachant que je dois retourner un int dans la methode hashCode()?

Tout simplement en utilisant la méthode hashCode() de la classe String :

Code :

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

1
2
3
public int hashCode() { 
  return numero.hashCode(); 
}

a++

[vincent63]

Si je peux me permettre, et juste pour chipotter un peu...

Code :

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

1
2
3
4
5
6
public boolean equals(Object o1){
if (!(o1 instanceof User))
  return false;
User u = (User)o1;
return numero.equals(u.numero);
}

La tradition veut que l'on teste toujours le type d'objet passé en paramètre, juste pour ne pas caster en user un truc qui n'en est pas.

Ensuite, ton test d'égalité repose sur l'égalité d'une chaine de caractères. Comme String a déjà défini equals, autant le réutiliser.

[rvfranck]

Merci beaucoup.