如何寫更長的IF更漂亮？

Question

我早就IF：

if(rand(1, 100) == 22 && $smth < time() && 
$smths > 5 && $sxsxsx > 250 && 
!$_SESSION['false']) 
{ 
    echo "wow, big if just happened!"; 
} 

怎麼寫更 「漂亮」？

Answer 1

按照我的回答對相關

• Multiple condition IF statement

這應該用重構，這意味着你應該將單獨的測試分成不同的功能。你應該擺脫魔術數字和無意義的變量名稱。我會給你一個關於如何爲你的代碼做這個事情的例子，但是代碼是不可理解的。

Answer 2

我更喜歡在布爾運算符之前打破。

if(rand(1, 100) == 22 
    && $smth < time() 
    && $smths > 5 
    && $sxsxsx > 250 
    && !$_SESSION['false'] 
) 

Answer 3

我喜歡列舉我的條件並將它們分組，以清楚它們的目的是什麼。

$is22 = rand(1, 100) == 22; 
$someTime = $smth < time() && $smths > 5; 
$meetsSx = $sxsxsx > 250; 
$inSession = !$_SESSION['false']; 
if ($is22 && $someTime && $meetsSx && $inSession) { 
    // do something 
} 

Answer 4

可能

if(
    rand(1, 100) == 22 && 
    $smth < time() && 
    $smths > 5 && 
    $sxsxsx > 250 && 
    !$_SESSION['false'] 
) { 
    echo "wow, big if just happened!"; 
} 

歡呼

Answer 5

使您的代碼可讀性是一個非常重要的方面，當談到支持你的代碼 - 別人可能做到這一點支持。

看看coding styles（如果必須，請搜索更多信息）。

我個人格式化片斷，像這樣：

if (
    rand(1, 100) == 22 
    && 
    $smth < time() 
    && 
    $smths > 5 
    && 
    $sxsxsx > 250 
    && 
    !$_SESSION['false'] 
    ) 
{ 
    echo "wow, big if just happened!"; 
} 

Answer 6

$isSomethingValid = rand(1, 100) == 22 
    && $smth < time() 
    && $smths > 5 
    && $sxsxsx > 250 
    && !$_SESSION['false']; 

if ($isSometingValid) { 
    // do something 
} 

Answer 7

始終縮進封閉聲明一個較塊的身體多餘的。你會寫這樣的函數：

function (reallylongparam, reallylongparam, reallylongparam, 
     reallylongparam, reallylongparam) { 
    doStuff() 
} 

，所以你會寫你的if語句是這樣的：

if(rand(1, 100) == 22 && $smth < time() && $smths > 5 
     && $sxsxsx > 250 && !$_SESSION['false']) { 
    doStuff(); 
} 

Answer 8

只是封裝布爾邏輯在一個單獨的功能

Answer 9

你也可以讓你的變量名更容易閱讀。