自分のゲッターを使用したクラス/セッター

Question

はのは、私は次のクラスがあるとしましょう。これを実現する方法は2つあります。

bool DataIsZero() const 
{ 
    return Data == 0; 
} 

または：

より良い練習と見なされ

bool DataIsZero() const 
{ 
    return GetData() == 0; 
} 

？変数自体を使用するだけで可読性が向上することがわかりますが、ゲッターを使用するとコードを簡単に保守することができます。

Answer 1

私はここには行かない理由のためにゲッター/セッターが本当に好きではありません。彼らは他の質問で覆われています。しかし、あなたがそれらについて尋ねてきたので、私の答えはゲッター/セッターを使うと仮定します。それはすべての可能な選択肢を訪れることはありません。

あなたが暗示するメンテナンス上の理由から、ゲッタを使用したいと思います。実際、抽象化は最初にゲッターを持つことの目的の半分（アクセス制御をわずかに厳密にすることに伴い）です。

変数を使用する方がゲッターを使用するより読みやすい場合は、ゲッター関数名が不良であり、再考する必要があります。

他にも、メンバーを初期化し、事実の後でコンストラクター本体に割り当てないのが最善です。あなたにも今起動して、一貫性を維持する可能性があるので、実際には、あなたは、定数であることを行うにはを持っている：

class MyClass 
{ 
private: 
    int Data; 

public: 

    MyClass(int Init) : Data(Init) {} 

    int GetData() const { 
     return Data; 
    } 
}; 

は、コンストラクターがどのように変化したかを参照してください。

Answer 2

getterを使用する必要があります。これは、クラスがゲッターのより複雑なロジックに移動すると、変更から隔離されるためです。しかし、あなたのクラスがパブリックゲッターを提供しているなら、私はこのメソッドを作るロジックに疑問を抱いています。

Answer 3

です。

前者は単純なクラスで十分です。

後者は実装を隠し、メソッドが仮想の場合は多態性をサポートできます。