これは、この単純なIF OR文を実行する方法ですか？

Question

私はorが入っているifのステートメントを持っています。

すなわち

if($count == 1 || $count == 3 || $count == 7) { 
    // do stuff 
} 

私はちょうど好奇心が強い - これは、これを行うための最良の方法ですか？上記の単純な例では、これを行うより速い方法がありますか？あれば、それは何ですか？

Answer 1

コードは正常に動作します。代わりに、あなたは少しクリーナーで、スケールin_array()を、使用することができ、より良い：

if (in_array($count, array(1,3,7))) { ... } 

Answer 2

あなたは、アレイ内のすべての可能な値を代入し、array_search機能

$array=array(1,3,7); 

if (array_search($count,$array) !== FALSE) 
{ 
    //do stuff 
} 

Answer 3

あなたが書いたコードを使用して確認することができます結構です。 Paul Schreiberによれば、ちょっとばかりいろいろなオプションがあります。

考えてみてください（これは単なる例に過ぎません）。なぜあなたがチェックしている値が重要なのでしょうか。彼らはすべてあなたがチェックしている共通のいくつかのプロパティを持っていますか？そうであれば、記号的にプロパティを記述すると、誰かがコードを理解しやすくなります。たとえば：

if (is_odd($x) && $x < 10) { 
    //... 
} 

ではなく

if ($x == 1 || $x == 3 || $x == 5 || $x == 7 || $x == 9) { 
    //... 
} 

は、これは非常に不自然な例ですが、うまくいけば、あなたは私が取得していますかを参照してください。

より具体的な例として、代わりのようなものやっての

：あなたがしたいかもしれません

if ($user->age > 65 
    || $user->years_of_custom > 3 
    || $num_items > 5) { 
     // Give this user a discount .... 
} 

を：

if (eligible_for_discount($user, $num_items)) { 
    // Give this user a discount 
} 

あなたはこれだけはできる、この一箇所で関数を使用する場合でも、コードの読みやすさを向上させます。明らかに、あなたは判断を下す必要があります。なぜなら、保守すべきコード行が増えてしまうために読みやすさが増しているからです。常に適切な選択であるとは限りません。条件がお互いに関係がない場合、それらを別々の関数にバインドすることは意味をなさないかもしれませんし、コードを難しくするのではなく、簡単にすることができます。コードが実際に意味するものと、人間がそれをどのように理解するべきかに焦点を合わせます。

Answer 4

switchステートメントが優れていませんか？

switch ($count) { 
    case 1: 
    case 3: 
    case 7: 
    echo "do stuff"; 
    break; 
}