if/else文をパターンに置き換えてください

Question

以下のコードでは、月日（mtd）インジケータがtrueである場合、または前2営業日（btd）インジケータは真です。

現在、別のインジケータが表示された場合、別のelse if文を含める必要があります。

私は、このコードの保守を容易にする方法はファクトリデザインパターンを使用することだと読んだことがあります。しかし、パターンを実装するためにどのようなステップを取るべきかわかりません。

public static string GetBusDate(bool mtdIndicator, bool prev2BusDtInd) 
    { 
     string busDate = Helper.PreviousBusinessDate; 

     if (mtdIndicator) 
     { 
      busDate = Helper.FOMBusDt; 
     } 
     else if (prev2BusDtInd) 
     { 
      busDate = Helper.Previous2BusinessDate; 
     } 

     return busDate; 
    } 

Answer 1

ファクトリパターン（ファクトリメソッド& &抽象ファクトリー）このパターンは唯一のカップルを作成することを強制されますので、ご使用のケースのために働くつもりはないされています。ここでは

コードです各ifステートメントのクラスとパターンは通常、消費するクライアントコードに抽象オブジェクトを返します（単純な文字列を返すので、あなたのケースではありません）。

シンプルファクトリ（デザインパターンではありません）は、条件付きロジックを1つのクラスに移動するだけなので、コードデザインが改善されないため、使用できません。

デザインパターンを使用すると、コードの複雑さが増し、柔軟性が損なわれます。

私は単純にif/else文を1行のコードで残しておきます。

return mtdIndicator ? Helper.FOMBusDt : 
     prev2BusDtInd ? Helper.Previous2BusinessDate : 
         Helper.PreviousBusinessDate; 

あなたは多型と条件文を交換したい場合は、試してみてください：

• Chain of Responsibilityパターン
• Strategyパターン
• Decoratorパターン

...またはその他のパタ​​ーン（全体的なアーキテクチャに依存）

また、「尋ねないで尋ねる」に従って、ifを避けることができます。

• あなたはそれが内部 コンテキスト/状態だについてあなたは、低レベルのコンポーネントを頼むべきではありませんタスクX
• を行うには、低レベルのコンポーネントへの要求を送信する必要があります（あなたがisStateXOk():Booleanのような機能を避けていることを行う）

ヒント：3から10+のクラスを作成してを作成して短いif/elseステートメントを回避しないでください。ケースブランチごとにクラスを作成してスイッチを回避してください。