使用字符串選項優化if-else/switch-case

Question

這段代碼會帶來哪些修改？在最後幾行，我應該用更多的的if-else結構，而不是 「如果 - 如果 - 如果」

if (action.equals("opt1")) 
{ 
    //something 
} 
else 
{ 
    if (action.equals("opt2")) 
    { 
     //something 
    } 
    else 
    { 
     if ((action.equals("opt3")) || (action.equals("opt4"))) 
     { 
      //something 
     } 
     if (action.equals("opt5")) 
     { 
      //something 
     } 
     if (action.equals("opt6")) 
     { 
      //something 
     } 
    } 
} 

後來編輯：這就是Java。我不認爲switch-case結構可以和Strings一起使用。

後來編輯2：

的開關的工作原理與字節，短， 炭，和INT基本數據類型。它 也與枚舉類型 （中類和繼承討論） 和一些特殊類作品是「包裝」 某些基本類型：字符， 字節，短，和Integer（以簡單的數據對象討論 ）。

Answer 1

即使你沒有使用switch語句，是的，如果避免無用的比較使用else：如果第一個如果被採用，你不希望所有其他ifs在這裏被評估，因爲他們會永遠是假的。同時你並不需要縮進各如果使得最後一塊如此縮進，你不能看到它不滾動，下面的代碼是完全可讀：

if (action.equals("opt1")) { 
} 
else if (action.equals("opt2")) { 
} 
else if (action.equals("opt3")) { 
} 
else { 
} 

Answer 2

使用switch語句，假設您的語言支持切換字符串。

switch(action) 
{ 
    case "opt6": 
     // 
     break; 
    case "opt7": 
     // 
    ... 
    ... 
    ... 
} 

Answer 3

您可以使用開關。

switch (action) 
{ 
case "opt3": 
case "opt4": 
doSomething; 
break; 
case "opt5": 
doSomething; 
break; 
default: 
doSomeWork; 
break; 
} 

Answer 4

這取決於你的語言，但它看起來類似C的，所以你可以嘗試的switch語句：

switch(action) 
{ 
case "opt1": 
    // something 
    break; 

case "opt2": 
    // something 
    break; 

case "opt3": 
case "opt4": 
    // something 
    break; 

case "opt5": 
    // something 
    break; 

case "opt6": 
    // something 
    break; 
} 

然而，有時switch語句沒有提供足夠的清晰度和靈活性（和正如Victor所指出的，在某些語言中不適用於字符串）。大多數編程語言將有說「否則，如果」，因此而不是寫

if (condition1) 
{ 
    ... 
} 
else 
{ 
    if (condition2) 
    { 
     ... 
    } 
    else 
    { 
     if (condition3) 
     { 
      ... 
     } 
     else 
     { 
      // This can get very indented very fast 
     } 
    } 
} 

...其中有凹痕的堆的方式，你可以寫這樣的事情：

if (condition1) 
{ 
    ... 
} 
else if (condition2) 
{ 
    ... 
} 
else if (condition3) 
{ 
    ... 
} 
else 
{ 
    ... 
} 

在C/C++，我相信C＃，它是else if。在Python中，它是elif。

Answer 5

建議使用開關的答案聲明是要走的路。 switch語句比現在使用的if和if...else語句更容易閱讀。

簡單的比較是快，//something代碼將不爲所有，但一個情況下執行，所以你可以跳過「優化」，去「維護」。

當然，這是假設action.equals()方法做一些小事，如==便宜。如果action.equals()很貴，您還有其他問題。

Answer 6

如果您指定語言，它可能會有所幫助......因爲它看起來像C++，所以可以使用開關。

switch (action) { 
    case "opt1": 
     // something 
     break; 
    case "opt2": 
     // something 
     break; 
    ... 
} 

而如果你想if語句，我想你可以提高可讀性和性能一點，如果你習慣用「否則，如果」無花括號，如：

if (action.equals("opt1")) { 
    //something 
} else if (action.equals("opt2")) { 
    //something 
} else if ((action.equals("opt3")) || (action.equals("opt4"))) { 
    //something 
} else if (action.equals("opt5")) { 
    //something 
} else if (action.equals("opt6")) { 
    //something 
} 

我認爲一些編譯器可以比else { if更好地優化else if。無論如何，我希望我能幫上忙！

Answer 7

我只想清理一系列的if/else語句：

if(action.equals("opt1")) 
{ 
    // something 
} 
else if (action.equals("opt2")) 
{ 
    // something 
} 
else if (action.equals("opt3")) 
{ 
    // something 
} 
etc... 

Answer 8

使用字典用字符串作爲鍵類型和代表*爲值類型。 - 從使用字符串檢索方法將採取O（1 +負載）。

在類的構造函數中填充字典。

• Java不支持委託，因此作爲一項工作可能需要定義幾個內部類 - 每個案例一個，並傳遞內部類的實例而不是方法作爲值。

Answer 9

在Java中有很多方法可以做到這一點，但這裏是一個整潔的方法。

enum Option { 
    opt1, opt2, opt3, opt4, opt5, opt6 
} 

... 

switch (Option.valueOf(s)) { 
case opt1: 
    // do opt1 
    break; 
case opt2: 
    // do opt2 
    break; 
case opt3: case opt4: 
    // do opt3 or opt4 
    break; 
... 
} 

注意valueOf(String)將拋出IllegalArgumentException如果參數 不枚舉的成員之一的名稱。在底層，valueOf的實現使用靜態哈希映射將其String參數映射到枚舉值。

Answer 10

實際上這取決於分支分析。如果99％的決定是「opt1」，這個代碼已經很不錯了。如果你的決定中有99％是「opt6」，那麼這段代碼很糟糕。

如果在第一次比較中經常出現「opt6」並很少「opt1」出現「opt6」，並根據執行數據流中字符串的頻率進行下列比較。

如果你有很多選擇，並且都具有同等的頻率，你可以在選項進行排序，並將其分成二叉樹這樣的形式：

if (action < "opt20") 
{ 
    if(action < "opt10") 
    { 
    if(action == "opt4") {...} 
    else if(action == "opt2") {...} 
    else if(action == "opt1") {...} 
    else if(action == "opt8") {...} 
    } 
} 
else 
{ 
    if(action < "opt30) 
    { 

    } 
    else 
    { 
     if(action == "opt38") {...} 
     else if(action == "opt32") {...} 
    } 
} 

在此示例範圍拆分減少需要比較「opt38」和「opt4」爲3.這樣做後，您可以在每個分支中獲得log2（n）+1比較。這對於選項的相同頻率是最好的。

不要做二進制吐到最後，最後使用4-10「正常」否則，如果構造按選項的頻率排序。二叉樹中的最後兩個或三個級別沒有太多進展。

摘要

至少也有對這種比較兩個優化。

1. 二元決策樹
2. 訂貨由於二進制決策樹用於大的switch-case選項

頻率構建由編譯器。但編譯器不知道任何選項的頻率。因此，如果一個或兩個選項比其他選項頻繁得多，那麼根據頻率排序可以是使用開關盒的性能優勢。在這種情況下，這是一個解決辦法：

if (action == "opt5") // Processing a frequent (99%) option first 
{ 
} 
else // Processing less frequent options (>1%) second 
{ 
    switch(action) 
    { 
    case "opt1": ... 
    case "opt2": ... 
    } 
} 

警告

，直到你做了剖析不要優化你的代碼，它是真的有必要。最好使用開關盒，否則 - 如果直接向前，並且您的代碼保持清潔可讀。如果你已經優化你的代碼，在代碼中放置一些好評，這樣每個人都可以理解這種醜陋的代碼和平。一年後，你將不知道分析數據，一些評論將會非常有幫助。

Answer 11

像這樣的過程切換通常更好地由多態處理 - 而不是由一個字符串表示的動作來代表具有可以專門化的「某種」方法的對象的動作。如果您發現您確實需要將字符串映射到該選項，請使用Map<String,Option>。

如果你要堅持程序代碼，並在你的實際代碼的選項真的都是「OPTX」：

if (action.startsWith("opt") && action.length() == 4) { 
    switch (action.charAt(3)) { 
     case '1': something; break; 
     case '2': something; break; 
     case '3': something; break; 
     ... 
    } 
} 

這將是確定的東西像一個解析器（其中破串起來的部分的問題域），並且應該是快速的，但不是內聚的（對象action與行爲之間的連接基於其表示的部分，而不是對象的任何內在）。

Answer 12

請注意，在switch語句的情況下使用字符串是將在下一個Java版本中添加的新功能之一。

見Project Coin: Proposal for Strings in switch

Answer 13

如果您發現本機的java開關構建物太多限制給一目瞭然的lambdaj Switcher，讓他們與一些hamcrest的匹配匹配任何物體上以聲明方式進行切換。