OOPでは、クラス内で「this」を使用することに関するベストプラクティスは何ですか？

Question

私はいつも疑問に思ったことがあります。 '[NAME]'または単に[NAME]のどちらかを使用してメンバーを参照することができるクラスで、これが優先されますか？ Javaで例えば

：

public class foo { 
    public int bars = 0; 
    private void incrementBars(){ 
     bars++; 
    } 
} 

と

public class foo { 
    public int bars = 0; 
    private void incrementBars(){ 
     this.bars++; 
    } 
} 

は、同じ効果を持っている 'ように見えます'。

for (foo f : listOfFoos){ 
    f.incrementBars(); 
} 

、まだ動作しているようです：私は、クラスfooの複数のインスタンスをインスタンス化のケースでは

は、私がこれまでのところ、のような何かを、と思います。

これは技術的に曖昧ですか？その場合は、好ましい方法がありますか？

Answer 1

thisの場合はvariable shadowingの場合。

class MyClass{ 
     int i;//1 
     public void myMethod(){ 
      i = 10;//referring to 1  
     } 

     public void myMethod(int i){//2 
      i = 10;//referring to 2 
      this.i = 10 //refering to 1  
     }  
    } 

もいつかthisはあなたが確実とあなたがフィールドを扱っていることを通信するthisを使っによる英語の考え方

Answer 2

に、コードをより読みやすくなります。

それはあなたが非常にsuccintある

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

のようなセッターを書くことができます。

Answer 3

唯一の時間this.は、現在のスコープにも同じ名前の変数がある場合です。すべてのクラス変数には、_variableという規則を使用することをお勧めします。このようにしてthis.を使用する必要はなく、誤ってクラス変数に触れることはありませんでした。

Answer 4

あいまいさはありません。存在した場合、にはがあり、thisを使用しています。

わかりやすくするために、一部の人にはthisをお勧めします。それ以外の場合は、「ノイズ」が発生するため、必要でないときは反対することをおすすめします。現代のIDEやエディタの中には、構文強調を使用して、引数を分かりやすくするためにフィールドとは異なる（たとえば）引数を付けることができます。

個人的にはthisを避けて@unholysampler's underscore conventionを使用します。あなたの同僚と何かを同意し、あなたのコーディング基準に入れてください。

Answer 5

class Test 
{ 
    private int value; 

    public Test(int value) 
    { 
    // This is wrong 
    // value = value 

    // This is right 
    this.value = value; 
    } 
} 

ローカル変数によって非表示になっているメンバー変数にアクセスするには、「this」を使用します。 は、「これは」OUがこれを行うことができ、現在のオブジェクトへの参照のみですので：

doSomething(this); 

これらは、基本的にあなたがこれを使用する必要があります唯一の方法です。

Answer 6

可変シャドーイングの場合を除いて、技術的にはあいまいです（ジガール氏の答えで指摘されています）。

ほとんどの場合、クラス構造の構文の強調表示を非常によく知っているエディタ/ IDEで作業すると、変数がオブジェクト内のフィールドであるか、メソッド内で定義された変数かどうかがわかります。しかし、より純粋なテキストエディタ（vimなど）では、構文の強調表示はこれを明確にしません。

これは私が通常のIDEの外でコードを読んでいることを知っているので、オブジェクトに属するすべてのフィールドを参照するのにthis.fieldを使用する方が好きです。それは少し冗長ですが、それは私が気にしないトレードオフです。

Answer 7

質問の最初の2つのサンプルはまったく同じものです。属性はローカル変数の影されたときにthisが必須であるところだけがある：

public class Test { 
    private int a; 

    public void noShadow() { 
     System.out.println(a); // attribute 
    } 

    public void shadow(int a) { 
     System.out.println(a); // parameter 
     System.out.println(this.a); // attribute 
    } 
} 

今私は、現代のIDEは、高度な構文の強調表示を提供し、特にとして、絶対的なベストプラクティスがあるとは思いません。たとえば、_、m_などのような属性を接頭辞として使用し、thisを使用しないようにするものがあります。ある者はthisの使用を必須とし、属性に命名規則を追加しないことを好みます。あなたがしようとしていることは、既存のコードまたは既存のコーディング標準に大きく依存します。私は、コーディングの標準がプロジェクトで働くすべての人々にとって同じであることを保証するものは何もありません。