独自のgetメソッドを持たないsetメソッドを持つのは悪い習慣ですか？

Question

だが、私は（名前円がランダムだったとは意味を持たない）クラスがあるとしましょう。一方で

Class circle{ 
    double colorFrequency_; 
public: 
    void setColor(double colorFrequency){ colorFrequency_=colorFrequency; } 
    void setColor(string colorName){ colorFrequency_=colorNameToFrequency(string colorName); } 
    double getColorFrequency() 
    string getColorName(){ /* converts color frequency into a string containing the name of a color it's close to, like "red" */ } 

} 

を、setColorを呼ばれる方法ではなく、GETCOLORと呼ばれるものを持っている矛盾したようだが、他の代わりにsetColorFrequencyとsetColorNameという関数があれば、setColorNameはgetColorNameの出力だけでなく、setColorFrequencyの出力にも影響するため、これらの名前は完全に説明的ではありません。

setColorという関数を持つのは奇妙ですが、getColorという関数はありませんか？

Answer 1

メンバーの値をクラスのユーザーが取得するかどうかによって異なります。

珍しいかどうか、あなたのデザインが要求するものに固執する必要があります。

Answer 2

あなたがSetModel()機能が必要な場合がありますが、意味がGetModel()ことはありません1つのよく知られたデザインです：最後のモデル - ビュー - プレゼンター（MVP）パターンまたは（例えばMVC、MVVMなどのような）その変種

私は主にUIに取り組んでいます。ここでは、プリズムアーキテクチャ（WPFとSilverlight）に続いてMVPを実装しました。私たちのデザインは、このような何かを行く：

、ビューのためのモデルとしての地位を設定するビューの SetModel関数を呼び出し、そのプレゼンターですが、それはdoesnのため、ビュークラスには GetModel機能はありません

//C# code, but this design can written in C++ as well 
public class QueryPresenter : IQueryPresenter, /*other interfaces */ 
{ 

    public QueryPresenter(IQueryView view, /*other parameter*/) 
    { 
    //... 
    view.SetModel(this); 
    } 
} 

まったく意味がありません。

はい、それは設定機能を必要とする可能性のある設計に完全に依存しますが、機能を取得することは意味がありません。通常、このようなセット関数は、MVPパターンのように、他の方法で使用されるオブジェクトを「構成する」ために使用されます。集合関数は、協調的に動作するクラスが少ない設計でも使用できます。そのようなクラスは、お互いにアクセスできるように、friendとすることができます（必要に応じて）privateメンバーです。

Answer 3

どのように奇妙なのか分かりませんが、ゲッター/セッターの "ペアリング"がないことを明確にするために、ミュータントの名前をchangeColor（または同様のもの）に変更することを検討するかもしれません。

個人的に、私はCircleの完全indepent変換を残して、呼び出し側がそれを処理させることを好むだろう：

double colorNameToFrequency(const string& name) { ... } 
string colorFrequencyToName(double frequency) { ... } 

class Circle 
{ 
    double colorFrequency_; 
public: 
    void setColor(double colorFrequency); 
    double getColor() const; 
}; 

// Client code 
Circle c; 
c.setColor(colorNameToFrequency("mint-green")); 
string name = colorFrequencyToName(c.getColor()); 

これは色の人間が読める形式として選択したものは何でもの独立したCircleクラスになります。