サイクロマティック複雑スイッチ/ケースとの違いとin_array

Question

シナリオ私は私の$type_id変数は、IDの特定のセットの1つであるかどうかを確認する必要があり

。

可読性以外の理由もなく、私は

それらのほとんどは一般的なケースにダウンカスケード

switch($type_id) { 
    case Type::SOME_TYPE: 
    case Type::SOME_OTHER_TYPE: 
    ... 
     //do stuff 

と一緒に行きました。

しかし、これにより、PHPMDがうぬぼれ始めるまでの循環的複雑さが増します。

代わりにin_array()を使ってみましょう。

if (in_array($type_id, [ 
    Type::SOME_TYPE, 
    TYPE::SOME_OTHER_TYPE, 
    ... 
    ])) { 
    //do stuff 
} 

質問

この時点でPHPMDは文句を停止し、ちょうどin_array()機能の後ろに隠れて、まだそこに循環的複雑度ではないでしょうか？

Answer 1

はい。しかし、PHPMDのルールは、単一のメソッド/関数内のCCのためのルールです。コールグラフ全体にCCを適用しません。一般的に、PHPMD CCの警告は、独自のメソッドに分岐を抽出して解決できます。

注：replace the conditional with polymorphismを考慮してください。