Discussion :

[fabaroulettes]

Hello,

Une petite question qui me trotte dans la tête...

Ca concerne l'utilisation du "this." même quand il n'est pas nécessaire.

Par exemple, je suis dans une méthode de ma classe et je veux appeler un getter.
Est-ce que je mets "getMonAttribut()" ou "this.getMonAttribut()" ?!?

Et surtout, et c'est nettement plus intéressant, POURQUOI vaut-il mieux utiliser l'un plutôt que l'autre ?

Merci d'avance.

ps : j'imagine que la question a déjà été posée... mais quand je cherche "this", ça me retourne systématiquement 0 résultat !

[yann2]

Bonjour

Je le mets systèmatiquement. Pourquoi ? Parce que ça permet, en seul coup d'oeil, de savoir si la méthode est de classe ou d'instance, si une variable est locale ou est une variable d'instance etc.
Ceci implique de mettre systèmatiquement le nom de la classe lorsque l'on utilise un membre de classe.

yann

[Baptiste Wicht]

Non, tu n'es pas obligé de le mettre, car le compilo va automatiquement le rajouter derrière toi

Il y a un cas ou tu es obligé de le mettre, c'est quand tu as une variable d'instance avec le nom qu'une variable locale.

Si tu fais comme ça :

Code :

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

1
2
3
public void setX(int x){
x=x;
}

Tu auras pas d'erreur, mais ça sert à rien. Il faut attribuer la valeur locale à la valeur globale :

Code :

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

1
2
3
public void setX(int x){
this.x=x;
}

[sironimo]

http://java.developpez.com/faq/java/...sClesThisSuper

[fabaroulettes]

ok, ok... je sais très bien que je ne suis pas obligé de le mettre, mais que dans certains cas il faut quand même le faire.

pour être plus précis, j'ai pour habitude de TOUJOURS le mettre (obligatoire ou non), et un collègue vient de me dire que "non, c'est pas bien, faut pas le mettre systèmatiquement", sans me donner bcp plus d'explications.

bref, quels arguments y'a-t-il pour dire qu'il ne faut pas le mettre quand ce n'est pas nécessaire ?

[maxf1]

Si tu utilises le this c'est plus sur je trouve. Car si jamais tu ne fais pas très attention et que tu appelles un parametre de methode de la meme facon ton code ne sera pas atteint.

Alors que sans le this tu peux avoir des surprises.

[thibaut]

Il n'y a aucun argument autre pour le mettre ou non.

Moi je ne le mets jamais sauf dans les cas ou il y a collision entre un paramétre de méthode et une variable locale, ou entre un attribut d'instance et un variable locale.

Pour moi ça alourdi le code d'autant plus, comme les gens qui mettent this devant l'appel à une méthode d'instance dans une autre méthode de la même instance, je trouve ça trop lourd.

De plus eclipse averti le programmeur en cas de collision de nom, donc ce n'est asboluement pas indispensable.

A+

[gailuris]

Oui je pense pas qu'il n'y est vraiment d'argument pour ou contre le mettre, par contre je pense que quand on travail à plusieurs c'edst pas mal de ce mettre d'accord histoire que les sources restent cohérentes.

[thibaut]

Tout à fait d'accord, mais pour moi le this est superflu et rend le code d'autant moins lisible...

A+

[adiGuba]

bref, quels arguments y'a-t-il pour dire qu'il ne faut pas le mettre quand ce n'est pas nécessaire ?

Aucun. C'est juste une préférence ou l'adoption d'une règle de codage, mais cela donnera exactement la même chose à la compilation...


Et si tu veux une preuve, regarde ce code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
public class Main {
 
	private String attr;
 
	public void method() {
		System.out.println( attr );
		System.out.println( this.attr );
 
	}
 
}

Si tu le compiles :

Code :

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

javac Main.java

Puis que tu désassemble son bytecode :

Code :

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

javap -c Main

Tu verra que tu obtiens exactement la même chose :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public void method();
  Code:
 
   0:   getstatic       #2; //Field java/lang/System.out:Ljava/io/PrintStream;
   3:   aload_0
   4:   getfield        #3; //Field attr:Ljava/lang/String;
   7:   invokevirtual   #4; //Method java/io/PrintStream.println:(Ljava/lang/String;)V
 
   10:  getstatic       #2; //Field java/lang/System.out:Ljava/io/PrintStream;
   13:  aload_0
   14:  getfield        #3; //Field attr:Ljava/lang/String;
   17:  invokevirtual   #4; //Method java/io/PrintStream.println:(Ljava/lang/String;)V
 
   20:  return

a++

[adiGuba]

Salut,

Et surtout, et c'est nettement plus intéressant, POURQUOI vaut-il mieux utiliser l'un plutôt que l'autre ?

Si on ignore le cas où il est obligatoire (c'est à dire pour différencier un attribut d'instance d'une variable locale du même nom), il n'y a aucune différence entre les deux cas (avec ou sans this). Donc c'est principalement une question de lisibilité et de préférence personnelle...

a++