Discussion :

[Faiche]

Bonjour

Quand on génère equals() avec eclipse, on se retrouve avec ça :

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
 
@Override
public boolean equals(Object obj) {
	if (this == obj) {
		return true;
	}
	if (obj == null) {
		return false;
	}
	if (getClass() != obj.getClass()) {
		return false;
	}
	MyObj other = (MyObj) obj;
	if (id == null) {
		if (other.id != null) {
			return false;
		}
	}
	else if (!id.equals(other.id)) {
		return false;
	}
	return true;
}

Le probleme, c'est qu'il considère par défaut que si les deux clés sont nulles, alors les deux objets sont égaux. Pour moi, au contraire, null est systématiquement différent.

J'aimerais qu'il me génère plutôt ça :

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
 
@Override
public boolean equals(Object obj) {
	if (this == obj) {
		return true;
	}
	if (obj == null) {
		return false;
	}
	if (getClass() != obj.getClass()) {
		return false;
	}
	MyObj other = (MyObj) obj;
	if (id == null || other.id == null) {
		return false;
	}
	else {
		return id.equals(other.id);
	}
}

Est ce que vous avez une idée de où je dois taper pour que ça modifier la génération du equals ?

Merci

[LittleBean]

Salut,

Normalement ça devrait se trouver là :
Window>Preference>Java>Code Style>Code templates

Il y a bien les templates de génération getter/setter et autres mais je ne vois pas celui de equals ...

Je ne sais pas si c'est possible juste par configuration ....

Si tu trouve je suis intéressé

[professeur shadoko]

J'ajouterai une critique grave:
- dans 99% des cas utiliser getClass() est incorrect
(ça ne peut être le cas que si le code est effectivement lié au contexte du ClassLoader ET si les sous-classes ne sont pas compatibles pour l'égalité avec
d'autres classes -effet de bords graves garantis dans de très nombreux cas-)
j'utilise rarement eclipse mais j'ai du changer le comportement pour qu'il utilise instanceof)