改進許多case switch語句

Question

我正在研究一些我自己沒有寫過的代碼示例，並且我發現這個函數在我看來很糟糕，至少效率不高。你能提出改進方法嗎？我認爲這可能適用於關聯數組，但不知道如何。

其他任何可以重寫的方法？

$op = ''; 
    switch($style) { 
     case 'f': 
     case 'F': { 
      $op = 'f'; 
      break; 
     } 
     case 'f*': 
     case 'F*': { 
      $op = 'f*'; 
      break; 
     } 
     case 'B': 
     case 'FD': 
     case 'DF': { 
      $op = 'B'; 
      break; 
     } 
     default: { 
      if (!empty($default)) { 
       $op = $this->dosomething($default, ''); 
      } else { 
       $op = ''; 
      } 
     } 
    } 
    return $op; 

Answer 1

要使用assoc命令數組，你需要做到以下幾點：

$styles = array(
    'S' => 'S', 
    'D' => 'S', 
    's' => 's', 
    'd' => 's', 
    ... etc ... 
); 

if (isset($styles[$op])) { 
    return $styles[$op]; 
} elseif (!empty($default)) { 
    return $this->getPathPaintOperator($default, ''); 
} else { 
    return ''; 
} 

Answer 2

沒有任何需要添加額外的大括號，你可以改變

case 'df': { 
     $op = 'b'; 
     break; 
    } 

到

case 'df': 
     $op = 'b'; 
     break;