非常に大きな「switch」ブロックを使用して400ブロックのプログラムコードにアクセスすることを避けることはできますか？

Question

私は現在、C＃で約400の数学関数を含んでおり、それぞれが独自の整数IDを持つアプリケーションを書いています。関数コードにアクセスするために、私のアプリケーションは現在、関数ごとに1つの 'case'ブロックを持つ巨大な 'switch'ブロックに関数のIDを供給しています。これは、シームレスかつ非常に高速に動作すると思われますが、コードブロックを関数識別子に照合するより効率的な方法はありますか？

Answer 1

もっと良い解決策は、各機能をそれ自身のクラスにカプセル化し、次にstrategy patternを使用して必要な機能を交換することです。

それぞれの関数を独自のクラスにまとめると、その関数のコードを1か所で見るのが簡単になり、各クラスがその関数を実装するコードを担当する良いseparation of concernsを維持するという利点があります。これらの機能のテストも簡単になります。

これはまた、再コンパイルすることなく外部ソースから関数を追加する機会があるので、将来的に関数を追加する方が柔軟性があることを意味します。

このアプローチのもう1つの利点は、関数がすべてインターフェイスを実装するため、ロード時にそれらの実装のアセンブリをスキャンして、余分な労力を要することなく利用可能な関数のリストに新しい関数を自動的に追加できることです。

Answer 2

あなたがDictionary<string, Func<double, double>>または似たような...

// Better, use an *enum* of function IDs... 
private static readonly Dictionary<int, Func<double, double>> Functions = 
    new Dictionary<int, Func<double, double>> 
{ 
    { 0, Math.Sqrt }, 
    { 100, x => x * 2 }, 
    ... 
}; 

注意をしたいように、これはMath.Sqrtを指し同じように、それはまた、独自のメソッドを参照することができるとサウンド - あなたは配置する必要はありませんコレクションイニシャライザ内のすべてのロジック。おそらく、単一の式だけではない関数をメソッドに分割することになります。

その後、次のようににそれを適用したい：私は一般的な目的であることを、ここで辞書を使用しました

public void Apply(int functionId, double value) 
{ 
    Func<double, double> function; 
    if (!Functions.TryGetValue(functionId, out function)) 
    { 
     // Throw an exception or whatever 
    } 
    return function(value); 
} 

注意を。関数IDが0 ... nの場合は、代わりに配列を使用できます。

最初のコメントで指摘したように、整数だけではなく関数IDの列挙型を使用するほうが賢明です。

Answer 3

Dictionary,this fellowは、6つ以上の文字列の場合があり、デフォルトの場合にはDictionaryを生成するという結果になります。ですから、コンパイル時にその変換をスキップしてみませんか？

Answer 4

私が知っている限り、いくつかの項目を含むステートメントは、内部的にルックアップテーブルを使用するようにコンパイラによって既に最適化されています。 Dictionaryは最小にする必要があります。しかし、いつものように、両方のアプローチを測定して決定する必要があります。

Answer 5

あなたはこのようなあなたのコード（psudocodeを）書いているように聞こえる：

function doMath(a,b, functionID) { 

    switch (functionID) { 
     case 1: 
      // addition function 
      return a + b; 
     break; 
     case 2: 
      // subtraction function 
      return a - b; 
     break; 
    } 

} 

あなたはこのようにそれを書きたいとき：

function add(a,b) { 
    return a + b; 
} 

function subtract(a,b) { 
    return a - b; 
} 

か、私はあなたの要件を誤解していますか？