改进许多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;