switch文またはlong if…elseチェーンを使用する必要がありますか？

多くの場合、switchステートメントについて聞いたとき、長いif ... elseチェーンを置き換える方法として先送りされています。しかし、switchステートメントを使用する場合、if ... else以外の場合に書くよりも多くのコードを書いているようです。また、すべての呼び出しのすべての変数を同じスコープに保持するなど、他の問題もあります。

ここに私が通常書くフローを表すいくつかのコードがあります（diamのおかげで）

String comment;   // The generated insult.
int which = (int)(Math.random() * 3);  //  Result is 0, 1, or 2.

if (which == 0) {
    comment = "You look so much better than usual.";
} else if (which == 1) {
    comment = "Your work is up to its usual standards.";
} else if (which == 2) {
    comment = "You're quite competent for so little experience.";
} else {
    comment = "Oops -- something is wrong with this code.";
}

それから彼らは私にこれをこれと取り替えてほしい：

String comment;   // The generated insult.
int which = (int)(Math.random() * 3);  //  Result is 0, 1, or 2.

switch (which) {
    case 0:  
             comment = "You look so much better than usual.";
    break;
    case 1:  
             comment = "Your work is up to its usual standards.";
    break;
    case 2:  
             comment = "You're quite competent for so little experience.";
    break;
    default: 
             comment = "Oops -- something is wrong with this code.";
}

はるかに厄介な構文でより多くのコードのようです。しかし、switchステートメントを使用することには本当に利点がありますか？

この特定の状況では、両方のように私には思えるifとはcase貧弱な選択肢です。私は単純な配列を使用します：

String comments[] = {
    "You look so much better than usual.",
    "Your work is up to its usual standards.",
    "You're quite competent for so little experience."
};

String comment = comments[(int)(Math.random() * 3)];

副次的な注意事項として、通常、をハードコーディングするのではなく、配列のサイズに基づいて乗数を計算する必要があり3ます。

ケース/スイッチを使用するタイミングについては、ifステートメントのカスケードとの違い（または少なくとも1つの大きな違い）はswitch、値の数と密度に基づいて半自動で最適化できるのに対して、ifステートメントのカスケードはコンパイラーから離れることです選択肢はほとんどありませんが、記述したとおりにコードを生成し、一致する値が見つかるまで次々に値をテストします。実際のケースが3つしかない場合、それはほとんど問題になりませんが、十分な数がある場合、重要になる可能性があります。

if...else if...チェーンの問題は、私がそれを読みに行くときif、プログラムが何をしているかを理解するために、すべての単一の条件を調べなければならないことです。たとえば、次のようなものがあります。

if (a == 1) {
    // stuff
} else if (a == 2) {
    // stuff
} else if (a == 3) {
    // stuff
} else if (b == 1) {
    // stuff
} else if (b == 2) {
    // stuff
}

（明らかに、このような少数のステートメントについては、それほど悪くはありません）

すべてのステートメントを読むことなく、条件変数を途中で変更したことを知る方法がありません。ただし、switch1つの条件変数のみに制限されるため、何が起きているか一目でわかります。

ただし、1日の終わりには、どちらも、switchまたはのチェーンを好まないでしょうif...else if。多くの場合、より良い解決策は、元の質問や多態性（言語がサポートしている場合）のような場合の何らかの種類のジャンプテーブルまたは辞書です。もちろん、常に可能とは限りませんがswitch、最初のステップとして回避するソリューションを探します...

switch (which) {
  case 0: comment = "String 1"; break;
  case 1: comment = "String 2"; break;
  case 2: comment = "String 3"; break;
  default: comment = "Oops"; break;
}

このタイプのスイッチケースを記述する上記の方法は、かなり一般的です。かさばる場合にスイッチケースを感じた理由は、あなたの体が1行しかないためであり、スイッチケースではbreakステートメントも必要でした。そのため、スイッチケースの本体サイズはif elseの2倍でした。より実質的なコードでは、breakステートメントは本文にあまり追加しません。単一行の本文の場合、caseステートメントと同じ行にコードを記述するのが一般的です。

他の人がすでに述べたように、スイッチのケースは意図をより明確にします。単一の変数/式の値に基づいて決定をしたいのです。私のコメントは純粋に読みやすさの観点からであり、パフォーマンスベースではありません。

この場合、switchステートメントはコードの意図により明確に一致します。単一の値に基づいて実行するアクションを選択します。

一方、ifステートメントは読みにくいです-何が起こっているのかを確認するために、それらすべてを確認する必要があります。私にとっては、精神的に解析する必要が少ないため、コードが少なくなります（文字数がわずかに多い場合でも）。

私は、この特定の場合には文字列の配列の方が優れているが、一般的にはelseifのチェーンよりもswitch / caseステートメントを使用する方がよいというジェリーに同意します。読みやすく、時にはコンパイラーがそのように最適化するより良い仕事をすることができますが、別の利点もあります：それはデバッグするのがずっと簡単です。

そのスイッチを押すと、一度に複数のifステートメントを1つずつ慎重にステップオーバーするのではなく、1回ステップするだけで正しいブランチに到達します。やり直す。

このような場合は切り替えが好きです。コードのポイントにはるかにマッチし、入力値ごとに異なるステートメントを実行します。if..else同じ効果を達成するために、より「トリック」のような役割を果たします。

switch 文もきれいです。それらすべてにタイプミスを隠すのは簡単です ==

また、Cの大きなブロックの場合、切り替えは高速です。

else..if範囲（1から100の間、これを行う、100から200の間）のようなものがある場合、またはCで文字列のような要素（他の言語で可能）を切り替えようとする場合により適切です。どちらも同じです。

Cでプログラミングするときは、多くのスイッチを使用する傾向があります。

効率的で簡潔なものを選び、自分がやったことだけでなく、その理由を文書化します。

コードは再訪できますが、必ずしも元の作者によってではありません。

存在しないコードを前向きに考えているため、ある実装を別の実装よりも慎重に選択する場合があります。

私は一般的にどちらのアプローチも好きではありません。長いスイッチまたはifステートメントは、オブジェクト指向の抽象化にリファクタリングされるように頼みます（ただし、あなたの例では、長いではなく短いと分類します）。

私は個人的にこの種のコードを別のヘルパーメソッドにラップします。

private string GetInsult()
{
    int which = (int)(Math.random() * 3);  //  Result is 0, 1, or 2.

    switch (which) {
        case 0: return "You look so much better than usual.";
        case 1: return "Your work is up to its usual standards.";
        case 2: return "You're quite competent for so little experience.";
        default: return "Oops -- something is wrong with this code.";
    }
}

public void Foo()
{
    string comment = GetInsult();
    Print(comment);
}

スイッチを別のメソッドに配置すると、returnステートメントをswitchステートメント内に直接配置でき（少なくともc＃で）、breakステートメントの必要性がなくなり、コードが読みやすくなります。

そして、これはif / else if / else ifアプローチよりもはるかに優れています。

Pythonには、if / elif / elseが便利なため、switchステートメントはありません。

a = 5

if a==1:
    print "do this"
elif a == 2:
    print "do that"
elif a == 3:
    print "do the other"
elif 3 < a < 9:
    print "do more"
elif 9 <= a < 15:
    print "do nothing"
else:
    print "say sorry"

シンプルでしょ？

C / C＃スタイルをswitch特に迷惑にしているものの1つは、case値がリテラルであるという主張です。VB / VB.NETの優れた点の1つは、select/case各ケースを任意のブール式にすることができることです。 それは便利です。相互に排他的な一連のブール式が役立つ場合が多い限り、一連のif / else ifsはより柔軟であり、入力と読み取りがより効率的であることは言うまでもありません。