Discussion :

[kazuzu]

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

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
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;
		}
	}
}

[slim_java]

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

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
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 :

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

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

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

[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

[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().

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

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

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

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 :

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

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;
    }
}

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

[Baptiste Wicht]

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.

[kazuzu]

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

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

[Baptiste Wicht]

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

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