オブジェクト指向プログラミング：ゲッター/セッターまたは論理名

私は現在、書いているクラスへのインターフェースについて考えています。このクラスには、キャラクターのスタイル、たとえば、キャラクターが太字、斜体、下線などが含まれます。これらのスタイル。私は論理名を好む傾向がありますが、それは効率的でも論理的でもないコードを書くことを意味します。例を挙げましょう。

私はクラスの持っているCharacterStylesメンバ変数があるbold、italic、underline（およびいくつかの他の人を、私はそれをシンプルに保つためにそれらを残しておきます）。最も簡単な方法は、あなたが行うことができるように、書き込みgetter / setterメソッドになり、これらの変数にアクセスするには、プログラムの他の部分を可能にstyles.setBold(true)してstyles.setItalic(false)。

しかし、私はこれが好きではありません。多くの人がゲッター/セッターがカプセル化を破ると言っているだけでなく（それは本当にそんなに悪いのですか？）私は、styles.format("bold", true)これらのすべての方法ではなく、1つの方法などでキャラクターのスタイルを設定することを期待しています。

ただし、1つの問題があります。C ++の文字列の内容ではオブジェクトメンバー変数にアクセスできないため、すべてのスタイルに対して大きなifステートメント/スイッチコンテナーを記述するか、スタイルを連想配列に格納する必要があります（地図）。

最善の方法がわからない。ある瞬間、私はゲッター/セッターを書くべきだと思い、次の瞬間、私は他の方法に傾く。私の質問は：あなたは何をしますか？そして、なぜあなたはそれをするのでしょうか？

はい、ゲッター/セッターはカプセル化を解除します-基本的には、基本フィールドに直接アクセスするための単なる追加レイヤーです。直接アクセスすることもできます。

ここで、フィールドにアクセスするためのより複雑なメソッドが必要な場合、それは有効ですが、フィールドを公開する代わりに、クラスが提供するメソッドを代わりに考える必要があります。すなわち。プロパティを使用して公開されるMoney値を持つBankクラスの代わりに、Bankオブジェクトが提供するアクセスの種類（お金の追加、引き出し、バランスの取得）を考え、代わりにそれらを実装する必要があります。Money変数のプロパティは、Money変数を直接公開する場合と構文的にのみ異なります。

DrDobbsにはさらに多くの記事があります。

あなたの問題のために、可能なスタイルの列挙型をとる2つのメソッドsetStyleとclearStyle（または何でも）があります。これらのメソッド内のswitchステートメントは、適切なクラス変数に関連する値を適用します。この方法では、後でHTMLで使用するために文字列として保存することにした場合、スタイルの内部表現を別のものに変更できます-使用した場合、クラスのすべてのユーザーも変更する必要があるものプロパティを取得/設定します。

任意の値を取得する場合、大きなif-thenステートメント（いくつかある場合）、またはstd :: mem_fun（またはstd ::を使用したメソッドポインターへの文字列値のマップ）のいずれかを使用する場合、ストリングをオンに切り替えることができますfunction）そのため、「bold」は、値がsts :: mem_funであるマップキーに格納され、変数boldをtrueに設定するメソッドになります（文字列がメンバー変数名と同じ場合は、書く必要があるコードの量を減らすための文字列化マクロ）

あなたが考慮しなかったかもしれない一つのアイデアは、デコレータパターンです。オブジェクトにフラグを設定してから、作成しているものにそれらのフラグを適用するのではなく、デコレータで書き込みを行うクラスをラップして、スタイルを適用します。

呼び出し元のコードは、テキストを囲むラッパーの数を知る必要はありません。外側のオブジェクトのメソッドを呼び出すだけで、スタックを呼び出します。

擬似コードの例：

class TextWriter : TextDrawingInterface {
    public:
        void WriteString(string x) {
            // write some text somewhere somehow
        }
}

class BoldDecorator : TextDrawingInterface {
    public:
        void WriteString(string x) {
            // bold application on
            m_textWriter.WriteString(x);
            // bold application off
        }

        ctor (TextDrawingInterface textWriter) {
            m_textWriter = textWriter;
        }

    private:
        TextWriter m_TextWriter;
}

など、装飾スタイルごとに。その最も簡単な使用法では、あなたは言うことができます

TextDrawingInterface GetDecoratedTextWriter() {
    return new BoldDecorator(new ItalicDecorator(new TextWriter()));
}

そして、このメソッドを呼び出すコードは、受け取ったものの詳細を知る必要はありません。WriteStringメソッドを介してテキストを描画できるのは、SOMETHINGのみです。

私は2番目の解決策に傾くでしょう。よりエレガントで柔軟に見えます。太字、斜体、下線（上線？）以外の型を想像することは困難ですが、メンバー変数を使用して新しい型を追加することは困難です。

私は最新のプロジェクトの1つでこのアプローチを使用しました。複数のブール属性を持つことができるクラスがあります。属性の数と名前は時間とともに変化する場合があります。辞書に保存します。属性が存在しない場合、その値は「false」であると想定します。利用可能な属性名のリストも保存する必要がありますが、それは別の話です。

あなたは問題を考え過ぎていると思います。

styles.setBold(true) そしてstyles.setItalic(false)大丈夫です