Est-ce une mauvaise pratique d'utiliser `this` en Java?

Question

Je suis juste curieux au sujet de l'utilisation de this, parce que l'exemple de code et la documentation d'Android ne font pas usage du mot-clé this (J'aime croire que les ingénieurs androïdes sont généralement intelligents, ainsi mon les utilisant comme base). Je suis juste curieux de savoir la raison sous-jacente que les ingénieurs Android ne font pas usage de this beaucoup.

Answer 1

C'est principalement une différence stylistique. Je n'ai pas encore vu un argument pour savoir pourquoi c'est bon ou mauvais. Personnellement, j'aime l'utiliser et il y a quelques cas où vous devez l'utiliser. Par exemple si vous avez une classe Anonymous Inner qui veut appeler une méthode de la classe parent, vous avez besoin de this. Exemple:

class MyClass{ 
    protected void doSomething(){ 

    } 

    OnClickListener n = new OnClickListener(){ 
     public void onClick(View v){ 
      MyClass.this.doSomething(); 
     } 
    }; 

} 

Une autre raison d'utiliser this est si un masque d'arguments de méthode d'une variable de membre. Dans ce cas, vous pouvez utiliser this pour faire la distinction entre les deux, bien que je vous recommande de renommer l'argument en quelque chose qui ne masque pas.

class MyClass{ 
    protected int val; 

    public void setVal(int val){ 
     this.val = val; // set the member variable equal to the method argument 
    } 

} 

Une autre raison d'utiliser this est en passant un pointeur vers l'instance de classe actuelle à partir de cette instance. Un exemple de cela est lorsque vous créez un nouveau Intent

Intent i = new Intent(this, NewActivity.class); 

Il y a probablement plus d'endroits où vous aurez besoin d'utiliser this. Ce sont les premiers qui me sont venus à l'esprit.

Answer 2

Peu importe, mais si vous utilisez 'this' dans votre code, assurez-vous de l'utiliser partout. Cela rend votre code clair.

Answer 3

C'est un goût plus personnel qu'une mauvaise pratique.

Answer 4

Non ce n'est pas, il y a des références à «ceci» dans l'exemple de code pour Android, mais il n'y a aucune raison de penser que c'est une mauvaise pratique. Cependant, la plupart du temps, cela n'est pas nécessaire et cela a peut-être été omis pour des raisons de concision.

Par exemple, un appel à methodA() à partir de la classe définie est appelé this.methodA(). Ce n'est pas nécessaire dans ce cas car cela est implicite lorsque vous faites l'appel. Il est couramment utilisé dans les setters où le nom de l'argument est le même que le nom de la variable que vous définissez.

public void setName(String name) { 
    this.name = name; 
} 

Answer 5

Qu'est-ce qui vous fait penser que c'est le cas? Par exemple. SDK coding style guide ne mentionne pas cela et j'ai vu cela étant utilisé dans divers endroits. Cependant, en suivant les directives de nommage pour les variables, cela n'est souvent pas nécessaire et peut être omis, ce qui est souvent le cas. Lorsque vous faites référence à une instance utilisant ce mot-clé, il est bon pour la clarté, mais comme le document mentionne qu'il n'y a pas de style vraiment appliqué et que les règles de nommage ne le rendent pas nécessaire.

par exemple. un setter totalement valable après la nomination avec et sans cela pourrait être

private int mCounter; 
    public void setCounter(int counter) { 
     mCounter = counter 
    } 

private int increment; 
public void setIncrement(int increment) { 
    this.increment = increment 
} 

Notez que dans le premier exemple, vous pouvez, mais ne doivent l'utiliser. Dans la seconde vous devez et le premier il est probablement plus proche du guide de style ..

Answer 6

L'utilisation this est plus d'une mesure de sécurité:

class MyClass{ 
    void doSomething() { 
    } 

    OnClickListener n = new OnClickListener() { 
     void doSomething() { 
     } 

     public void onClick(View v){ 
      MyClass.this.doSomething(); // MyClass#doSomething() 
      this.doSomething(); // OnClickListener#doSomething() 

      doSomething(); // OnClickListener#doSomething() if present, 
          // MyClass#doSomething() otherwise - danger! 
     } 
    }; 
} 

Comme il y a des cas où vous ne pouvez pas contrôler la présence d'un membre ou d'une méthode (par exemple, lorsque le OnClickListener est une classe tiers), à obtenir un comportement prévisible, vous devriez toujours qualifier une telle référence. À moins que le comportement variable soit exactement ce que vous voulez.