如何写更长的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

你也可以让你的变量名更容易阅读。