if(in_array($str, array(a,b,c))) か if($str == a || $str == b || $str == c) か

Question

###前提
CakePHP2.xで、ボタンを押したときに呼び出される関数内でのことです。
チーム内でコードの書き方を統一していく指標を探っています。

###疑問の内容
if文を使用して実行内容を分岐するとき、次の2パターンのうちどちらが適切なのでしょうか？
「実行速度」「可読性」といった点からお答えいただけると助かります。

//両パターン共通の定数, 変数
define('DEFINE_ONE', 1);
define('DEFINE_TWO', 2);
define('DEFINE_THREE', 3);
define('DEFINE_FOUR', 4);
$str['Model']['key'] = 'test_string';

//パターン1
if(in_array($str['Model']['key']), array(DEFINE_ONE, DEFINE_TWO, DEFINE_THREE, DEFINE_FOUR)){
//処理内容略
}

//パターン2
if($str['Model']['key'] == DEFINE_ONE
 || $str['Model']['key'] == DEFINE_TWO
 || $str['Model']['key'] == DEFINE_THREE
 || $str['Model']['key'] == DEFINE_FOUR){
//処理内容略
}

###補足
定数名は大体上記のような文字数です。in_arrayの第一引数はCakePHPを使用しているので、上記のように配列の形になります。

新しく入ってきたメンバーに、if文の中身が長くなるので、パターン2ではなく1の書き方でコーディングしてもいいかと聞かれたのですが、私は「in_arrayの速度や挙動に不安がある」「コードの長さがそんなに変わらない」「パターン2の方が見やすい」ことからパターン2でよいのではないかと考えています。

より適切なコーディングのために回答していただけると助かります。
よろしくお願いいたします。

Answer 1

条件の数にもよるでしょうが、個人的には判定を関数にして
if文ではそれを呼び出すようにするのが好みかなあ。
関数を使わないならswitch文使うと思う。

Comments

[t_hoshizaki]

回答ありがとうございます。
条件の数は大体4～6個です。
「条件文が横に伸びて読みづらい」や「条件を羅列していてコードが読みにくい」といったことから今回の質問をさせていただいていたので、今回頂いた関数という新しい提案を次の機会にしてみようと思います。
なお、switch文はチーム内ではパターン2に分類されています。

Answer 2

定数とマッチさせるときはビット計算するほうが多いのでは？

<?PHP
define('DEFINE_ONE',  1);
define('DEFINE_TWO',  2);
define('DEFINE_THREE',3);
define('DEFINE_FOUR', 4);
$def_all=(pow(2,DEFINE_ONE -1)| pow(2,DEFINE_TWO-1)| pow(2,DEFINE_THREE-1) | pow(2,DEFINE_FOUR-1));
$num=3;
print $def_all & pow(2,$num-1);
$num=4;
print $def_all & pow(2,$num-1);
$num=5;
print $def_all & pow(2,$num-1);
?>

最初からDEFINE_ALL的な定数を用意してもいいかもしれないですね。

Comments

[t_hoshizaki]

回答ありがとうございます。
ビット計算をするというチーム内では出なかった新しい案に驚いています。
ただチーム内にプログラミング初心者が入る可能性があるので関数や、提案頂いたDEFINE_ALLの案を検討していこうと思います。

[yambejp]

もしビット処理する場合は

define('DEFINE_ONE', 1);
define('DEFINE_TWO', 2);
define('DEFINE_THREE,4);
define('DEFINE_FOUR,8);

$def_all=DEFINE_ONE| DEFINE_TWO| DEFINE_THREE|DEFINE_FOUR;

とビット刻みで元の定数を用意したほうが感覚的にはわかりやすいかもしれません

また最初から要素数がきまっていればALLは
define('DEFINE_ALL,15);
としてもいいですし、上限を設定し
define('DEFINE_ALL,1023);
など余裕をみておいてもよいでしょう。

[t_hoshizaki]

更なる回答ありがとうございます。
具体的に書いていただいたのでより理解できました。
チーム内での導入はまだ難しいかもしれませんが、個人的に制作しているプログラムで使用してみようと思います。