Classe correcte ?

Version imprimable

Classe correcte ?

bonjours, je souhaiterai savoir si ma classe est correct et plus particulièrement la méthode equals merci

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
 
package pModel;
 
public class Mot 
{
	private String francais;
	private String anglais;
 
	public Mot(String francais,String anglais)
	{
		this.francais = francais;
		this.anglais = anglais;
	}
 
	public String getFrancais()
	{
		return francais;
	}
	public String getAnglais()
	{
		return anglais;
	}
 
	public boolean equals(Mot m)
	{
		if(francais.equals(m.francais) || anglais.equals(m.anglais))
		{
			return true;
		}
		else
		{
			return false;
		}
	}
}

Citation:

Envoyé par kazuzu

bonjours, je souhaiterai savoir si ma classe est correct et plus particulièrement la méthode equals merci

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
 
package pModel;
 
public class Mot 
{
	private String francais;
	private String anglais;
 
	public Mot(String francais,String anglais)
	{
		this.francais = francais;
		this.anglais = anglais;
	}
 
	public String getFrancais()
	{
		return francais;
	}
	public String getAnglais()
	{
		return anglais;
	}
 
	public boolean equals(Mot m)
	{
		if(francais.equals(m.francais) || anglais.equals(m.anglais))
		{
			return true;
		}
		else
		{
			return false;
		}
	}
}

Code:

1
2
3
4
5
6
public boolean equals(Mot m)
	{
 
          return ( francais.equals(m.getFrancais() ) || anglais.equals(m.getAnglais() )   );
 
	}

17/06/2010, 18h20
thelvin

Ça, et puis je suppose que la condition ne devrait pas être un OU, mais un ET.
Autrement dit, que le mot français soit le même dans les deux objets ET que le mot anglais soit le même dans les deux objets.

(Mais je peux me tromper. Si ça se trouve c'est voulu que ce soit un OU.)
17/06/2010, 18h20
Rei Ichido

Il y a potentiellement deux problèmes liés à ta méthode equals.

Le premier, c'est que ta méthode n'est pas consistante. Si tu as 3 objets A, B et C, que A.francais = C.francais, que B.anglais = C.anglais, tu peux avoir A =C, B = C mais A <> B. C'est peu judicieux :)

Le second, c'est qu'en général quand tu définis la méthode equals, il faut aussi redéfinir la méthode hashcode. Je te laisse chercher plus de doc à ce sujet :mrgreen:
17/06/2010, 18h45
thelvin

Ce qui m'amène à ma propre remarque annexe :

Là, nous avons une méthode equals(Mot). Ce n'est pas la même chose que la méthode equals(Object).

Les différentes classes Java qui se servent de la comparaison, appellent equals(Object) et rien d'autre. Si ton but est qu'elles appellent ta propre méthode, il faut redéfinir equals(Object).
Et si tu fais ça, il y a effectivement des règles à respecter : symétrie et transitivité de l'égalité, et redéfinir hashCode().
18/06/2010, 00h40
Laurent.B

Bonjour,

Comme cela vient d'être expliqué, la méthode equals nécessite une attention particulière selon la façon dont tu vas l'utiliser.

Cf. http://java.developpez.com/faq/java/...#DIVERS_equals . C'est sans doute un peu ardu quand on débute mais au moins tu as les éléments auxquels il faut faire attention.

Donc ton implémentation n'est pas véritablement fausse, tout dépend de la façon dont tu l'utilises mais par exemple, dans une application JEE, elle ne sera pas exploitable en l'état.
21/06/2010, 20h44
kazuzu
Voila, je me suis concentré sur la méthode hashcode().
Pouvez-vous me dire si ce que j'en ai compris est valable ?

En gros cela permet d augmenter la vitesse de recherche dans un tableau en retournant un index qui renvoi plus ou moin a la bonne place.

Dans mon cas l'implémentation de la méthode hashcode() serait :
Code:

1 2 3 4 public int hashcode() { return (francais.hashcode() + anglais.hashcode()); }
Maintenant je ne vois pas comment appeler cette méthode dans equals(). Peut-être comme ceci ? :
Code:

1 2 3 4 5 6 7 8 9 10 11 public boolean equals(Mot m) { if(this.hashcode()>0) { return true; } else { return false; } }
21/06/2010, 22h01
Laurent.B

Heu, à mon avis tu vas avoir du mal à t'en sortir en faisant comme ça...

La méthode hashcode n'a pas a être invoquée à partir de la méthode equals.
Relis plutôt ce que je t'ai indiqué par le lien.
21/06/2010, 22h18
Baptiste Wicht

Citation:

Envoyé par Laurent.B

Heu, à mon avis tu vas avoir du mal à t'en sortir en faisant comme ça...

La méthode hashcode n'a pas a être invoquée à partir de la méthode equals.
Relis plutôt ce que je t'ai indiqué par le lien.

+1

Deux objets égaux ont le même hashCode(), mais deux objets avec le même hashCode() ne sont pas forcément égaux.
21/06/2010, 22h40
kazuzu

:oops: je ne vois vraiment pas ou je peux invocqué la méthode hashcode() vu que je n utilise pas de map :oops:

ils faudrait que j appel la méthode equals() dans hashcode() ?
22/06/2010, 07h16
Baptiste Wicht

Citation:

Envoyé par kazuzu

:oops: je ne vois vraiment pas ou je peux invocqué la méthode hashcode() vu que je n utilise pas de map :oops:

ils faudrait que j appel la méthode equals() dans hashcode() ?

Ce n'est pas à toi d'utiliser hashCode(), c'est l'implémentation de Map qui va utiliser ta méthode hashCode pour répartir les objets de la map dans des emplacements pour améliorer les performances. En règle général, on n'appelle jamais directement la méthode hashCode().