Discussion :

[pot2yaourt]

Bonjour à tous,

Voilà, j'ai une petite question sur les "règles" de programmation que vous utilisez.

Admettons que l'on travaille sur un projet Winform qui contient plusieurs contrôles du type bouton, textbox, liste déroulante préfixés comme suit : btnXXX, txtXXX et cmbXXX.

Pour affecter du texte à une textbox depuis votre form, vous faites plutôt :

Code :

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

txtNom.Text = "toto";

ou plutôt :

Code :

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

this.txtNom.Text = "toto";

Même question pour appeler une méthode privée à l'intérieur d'un objet :

Code :

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

GetAgeDuCapitaine();

ou

Code :

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

this.GetAgeDuCapitaine();

Voilà, j'aurais bien aimé savoir le code que vous utilisez et pourquoi.

Personnellement je suis plutôt partagé :
- l'utilisation de txtNom.Text ou GetAgeDuCapitaine(); signifie implicitement que ce sont des contrôles / méthodes qui appartiennent à l'objet en cours, donc pas besoin de préciser avec this
- d'un autre côté, le this permet d'avoir uniquement les méthodes / propriétés d'un objet sans s'encombrer de toutes les références au framework .Net, mais je le trouve un peu "lourd".

Merci de me donner votre avis sur la question !
Lionel.

[Marsupilami_00]

Bonjour

Moi j'utilise

Code :

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

txtNom.Text = "toto";

Les méthodes sont des accesseurs que j'utilise pour des variables que je publies à l'utilisateur et dont je veux effectuer un traitement particulier. Je ne les utilise jamais dans le cas que tu as évoqué. J'utilise le "This" quand j'ai un conflit de méthode ou de variable avec .Net ou dans les extensions de classe.

[yaniss321]

Pour ma part, j'utilise toujours

Code :

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

this

pour éviter les ambiguités...

Si par exemple j'ai un paramètre de fontion ayant le même nom que la variable de la classe ayant cette fonction ....

[clochettes]

Bonjour,
pour des petits projets, sur lesquels je bosse seule, j'ai tendance aussi à ne pas mettre le this.
Cependant, j'ai déjà travaillé sur de gros projets où les conventions de codage l'imposaient... Et c'était plutôt utile à la lecture. En effet, quand on arrive dans un code qu'on ne connaît pas, pas forcement bien découpé en petits modules, cela permet de différencier très rapidement les variables de classe des variables locales. Pour les fonctions en revanche, je ne vois pas l'intérêt à priori.

[yaniss321]

pour les fonctions je suppose que c'est tout simplement pour utiliser l'intelisense ... si on ne se souvien plus de toute les fonctions de la classe alors un petit

Code :

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

this

et hop!!! c'est réglé
bien que les fonctions soit listées et visibles sur l'éditeur, c'est une habitude que j'ai prise pour ma part

[Nathanael Marchand]

Bonjour,
pour des petits projets, sur lesquels je bosse seule, j'ai tendance aussi à ne pas mettre le this.
Cependant, j'ai déjà travaillé sur de gros projets où les conventions de codage l'imposaient... Et c'était plutôt utile à la lecture. En effet, quand on arrive dans un code qu'on ne connaît pas, pas forcement bien découpé en petits modules, cela permet de différencier très rapidement les variables de classe des variables locales. Pour les fonctions en revanche, je ne vois pas l'intérêt à priori.

C'est que votre convention était pourrie alors
Personellement, j'utiliser Resharper et ses conventions par défaut:
_maVariable pour un membre privé de classe
maVariable pour une instance locale
MaVariable pour une propriété publique ou une constante
A partir de la, c'est totalement inutile d'utiliser this puisque du premier coup d'oeil tu sais ce qu'est ta variable

[Max]

En effet, quand on arrive dans un code qu'on ne connaît pas, pas forcement bien découpé en petits modules, cela permet de différencier très rapidement les variables de classe des variables locales.

Salut.

J'utilise systématiquement this pour cette raison

Après, à mes yeux, il s'agit d'un débat stérile , tu trouveras toujours des pour et des contre...

Tant qu'on est dans les conventions, je rajouterais juste que :

Code :

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

GetAgeDuCapitaine()

S'il s'agit d'une méthode privée, je commence toujours le nom par une minuscule :

Code :

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

private void getAgeDeLaFemmeDuCapitaine()

Afin de les différencier des méthodes publiques :

Code :

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

public void GetAgeDuCapitaine()

Voilà

[yaniss321]

D'accord avec eusebe19...
les conventions vont changer en fonction de la personne ou de la société...

par exemple, pour ma part les membres private commencent toujours avec un _ que ce soit fonctions ou variable

[MetalGeek]

Salut,

pour ma part je suis partisan d'écrire le maximum d'infos afin de lever toutes les ambiguïtés possibles. Du coup, j'utilise systématiquement 'this', que ce soit pour les méthodes, les propriétés ou les champs, quelle que soit leur visibilité.
De la même manière je précise systématiquement la visibilité de tous les membres, même si par ex. un champ est privé par défaut.
Enfin, pour les conventions de nommage, je préfixe les champs privés par des underscores, et je suis un anti-notation hongroise convaincu !