列挙型とcase文

Question

スイッチ/ケースステートメントを避け、列挙型でこれを行うことを避けようとすると良い理由があります。次の例を考えてみましょう： （このスイッチ/ケースはすべて

switch (theEnum) 
{ 
    case MyEnum.Enum1: 
     // dosomething 
    case MyEnum.Enum2: 
     // do something 
    case MyEnum.Enum3: 
     // do something 
    default: 
     throw new Exception("Unsupported enumeration: " + theEnum.ToString()); 

}

対）コードベースに散らばっ

public Dictionary<MyEnum, StrategyBase> BuildMapper() 
{ 
    var mapper = new Dictionary<MyEnum, StrategyBase>(); 
    mapper[MyEnum.Enum1] = new Strategy1(); 
    mapper[MyEnum.Enum2] = new Strategy2(); 
    mapper[MyEnum.Enum3] = new Strategy3(); 
return mapper; 
} 

BuildMapper()[MyEnum.Enum1].DoSomething(); 

オプション2は、より多くのOOはあるが、私は他の人が考えているかと思いました私たちがこれをするかどうかに努めなければならない理由がある場合は、このアプローチを使用してください。

switch/elseのような原則は、例えばオープン・クローズに違反すると主張するかもしれません。 OOのために

Answer 1

私はそれをより読みと明示的に見えるので、第一の変形が良いと思いVisitor Pattern

Answer 2

を使用する必要があります。

Answer 3

私は常に同じ辞書セットを返すので、BuildMapperを静的（読み込み専用）にします。

なぜこれが良いのかは、物事があるものから別のものへとマッピングされるような設計をしているときに、その理由が単純であることです。このマッピングは常に静的であり、/elseまたはそれが単なるマッピングであるという意図を隠すスイッチケース。

Answer 4

私はそれがすべてあなたが達成したいものになると言います。最初のサンプルを読みやすく、コードベースを他の人と共有することが重要です。 BuilderMapperは、あるパターンに従っている場合に便利です。

Answer 5

オレンジとリンゴを比較しているようです。

スイッチは、変数といくつかの定数データの逐次比較です。 1Kケーススイッチがない限り、それは辞書よりもはるかに優れているはずです。

もちろん、ディクショナリは非常に複雑なオブジェクトよりも、スイッチのメモリ消費量がはるかに少なくなります。

最後に、Enumは宣言されたオプションに制約されませんが、 "Flags"として組み合わせることができます。この機能は、ハッシュマップを使用して使用できません。

満足ですか？

Answer 6

ちょうど1つか2つの場所にあれば、私はスイッチのために行くだろう。あなたが言うように、「このスイッチ/ケースがコードベース全体に分散する可能性がある」場合、2番目の解決策に進みます。

あなたの状況では、それは良いですが、維持しやすいだけではありません。この場合、パフォーマンスとメモリ消費に関する懸念事項はまったく関係ありません。まずメンテナンスを容易にし、必要に応じて最適化してください。を最適化する必要はありません。

また、この場合、可読性の損失は無関係です。第2のアプローチではほとんど時間がかかりません。