実装ファイルのC++ゲッターセッター

Question

私はC++には比較的新しいので、私の質問は例で分かりやすいと思います。 copyVar1メソッドを実装するときに、私のヘッダファイルでは、私は私の実装の.ccファイルで

class myClass{ 
    public: 
     double getVar1(); 
     void setVar1(double newVar1); 
     void copyVar1(myClass* dat); 

    private: 
     double var1; 
}; 

があると、私はやるべき

void myClass::copyVar1(myClass* dat){ 
    var1 = dat->var1; 
} 

または

void myClass::copyVar1(myClass* dat){ 
    var1 = dat->getVar1(); 
} 

秒で私は代わりにゲッターメソッドを使用します。どちらもVisual C++で正常に動作しますが、どちらが実際に使用する方が良いかを知りたいと思います。

ご意見ありがとうございました！

Answer 1

ベストプラクティス？メソッドを記述するのではなく、代入演算子をオーバーロードします。

myClass & myClass::operator=(const myClass & dat) 
{ 
    var1 = dat.var1; // or dat.getVar1() 
    return *this; 
} 

フィールドを使用したり、セッターを呼び出して...それはすべて個人的な味の問題です。あなたのゲッターに副作用がある場合は、それを呼び出す必要があります。それ以外の場合は、フィールドを使用します。

だから、大きな "依存する"。

Answer 2

によって異なります。変数の代わりにゲッター/セッターを使用することは、プログラミングのエラーや変更に対してより堅牢であり、コードの重複を減らすことを多くの人があなたに伝えます。しかし、これらのgetter/setterがインライン関数でない限り、パフォーマンスの低下が少なく、インプリメンテーションがクラスの内部を知る必要があるため、変数を直接使用するだけではいかがですか。

Answer 3

いくつかの実装の詳細（例えば値アサインの検証を紹介）あなたは冗談がセッター内の1つの場所でそれを変更する必要がある方法あなたの変更を場合、私はセッターを使用して好むとその方法をゲッタリングでしょう...

Answer 4

あなたがしようとしていますコピーコンストラクタを書くには？なぜあなたはsetVar1を持っているときにcopyVar1が必要ですか？コピーコンストラクタを作成しようとしている場合は、getterを使用しない方が良いでしょう。あなたはゲッターとセッターの両方を持っている場合は、なぜvar1がパブリックにしない

myclass(const myclass& other) 
{ 
var1 = other.var1; 
} 

？

Answer 5

クラスの外にいるときには、ほとんどの場合getter/setterメソッドを使用して変数にアクセスする必要があります。そうしなければならないため、しばしば行う必要があります。しかし、あなたがクラスの中にいるときにはどちらかを使うことができ、getterメソッドが何もせずに変数を返すならば、それは違いはありません。

あなたはcopyVar1が呼び出されたときにそのコードが実行されるようにしたい場合は、ゲッターだけ変数を返すよりも何かをする方法、およびのコードを持っている場合あなたの決定が基づくことになるだろう。あなたがわからない場合、私のアドバイスは、のコードを将来変更することを決定した場合でも、getterメソッドを使用することです。今すぐ直接アクセスするだけで正常に機能し、微視的にはパフォーマンスが向上するかもしれませんが、必要なときにゲッターを呼び出さないでください。そして、コンパイラはおそらくあなたがその違いを感じさせないほどに最適化を終わらせるでしょう。 ：D

Answer 6

ここでそれを行う方法はありません。

ただし、getVar1（）が値またはvar1を返すと仮定しています。 var1値の格納方法を変更する場合は、すべてのコードを調べて更新する必要があります。 setter/getterを使用すると、これはほんの数通りの方法になります。その場合、あなたは番目のオプションを使用する必要があります

void myClass::copyVar1(myClass* dat){ 
    setVar1 (dat->getVar1()); 
} 

をして、あなたは完全にVAR1への依存を削除するには、何か他のものにVAR1を変更することができますし、copyVar1はまだ動作します。

Answer 7

あなたが言うように、両方とも正しく動作し、完全に合法です。

メンバー変数に直接アクセスすることとアクセサーを使用することの違いは、クラス内でより強力な「レイヤー」を定義することです。このレイヤは、任意のレイヤとして、依存関係を減らすのに役立ちます。

つまり、ゲッターメソッドは、基本的に実装の詳細を外部に公開しないという要件に応答します。たとえば、var1を二重に保存する代わりに、計算することもできます。したがって、アクセサはあなたにこの種の自由を与えます。ある程度、他のクラスに影響を与えずに（つまり、依存関係を減らして）実装を変更することができます。

getterメソッドをクラス自体で使用する場合、つまりクラススコープ内の依存関係を削除する場合も同じことが当てはまります。そのため、デザインは変更に対してより復元力があります。

Answer 8

セッターとゲッターの抽象化のポイントは、検索の方法が変わる可能性があるということです。メンバー変数に直接アクセスすると、何か変更があった場合は、どこでも使用状況を更新する必要があります。しかし、関数を使用する場合は、変数の取り込み方法を変更するだけで関数を変更する必要があります。

これは同じクラスなので、変数をパブリックにするよりも、この1つの実装ファイルを更新するだけで済みます（クラスのすべてのユーザーを更新する必要がないため）ライブラリクラスをお持ちの場合は可能です）。

これを見るには、突然スレッドセーフにしなければならないクラスを見てください。さて、あなたのコピー機は、

のようになります。あなたはいつも関数で呼び出すと、他の場所に固定する必要はありません。あなたはエンジニアが他の人のオブジェクトをロックすることを忘れないように持たせる場合は、直接変数にアクセスした場合、それがDATに外部の「DAT」のミューテックスをロックし

void myClass::copyVar1(myClass * dat) 
{ 
    scoped_lock lock(mutex); 
    scoped_lock lock2(dat->mutex) 
    var1 = dat->var1; 
} 

ようになり、通常は（非常に良いデザインとは見なされません彼らは忘れるチャンスを開き、それをしないでください）。

同様に、変数がデータベースに格納され始めた場合は、同様にそれを処理する必要があります。あるいは、変数が2つの異なる変数に格納され、検索されたときに構築された場合、複雑さがどのように増加するかを見ることができます。

しかし、アクセサとミューテータを使用する設計思想は、カプセル化によって潜在的な複雑さを減らすことです。クラスを変更しようとする意図がない小規模なプロジェクト（特に個人的なプロジェクト）に取り組むときがあります。それらに直接アクセスすることはそれほど複雑ではないかもしれません。そうすることを恐れてはいけません。あなたがなぜ決定を下すのかを知ってください。