如何编写具有多个条件的单行 if 语句？

Question

我目前正在处理带有嵌套 if 语句的 foreach 循环，但我很确定有更好的方法来编写这些 if 语句块。
我找到了这篇文章：PHP if shorthand and echo in one line - possible?
虽然这篇文章是针对单一条件的，但我想以同样的方式写我的文章（单行）。

我自己在 PHP 方面的经验并不多，所以我有点坚持使用老式的方式：

   if(($colorLevel['name'] === 'ATTR_VPMCV13') && ($colorLevel['level'] >= 80))
    {
        $prominentSideNumberArray[] = 10;
    }

   elseif(($colorLevel['name'] == 'ATTR_VPMCV13') && ($colorLevel['level'] >= 60) && ($colorLevel['level'] <= 70)){
        $prominentSideNumberArray[] = 8;
    }

如果有人能正确解释在何处以及为什么使用什么代码，那真的可以帮助我和/或其他人。我看过手册，但我就是不知道在哪里使用什么。

Answer 1

没有像 "if shorthand" 这样的东西。
?: 是运算符，if 是 control structure。它们是不同的语言概念，具有不同的目的并做不同的事情。包含三元条件运算符 (?:) 的表达式始终可以重写为两个表达式和一个 if/else statement。反之亦然。

---

如果您将$colorLevel['name'] 的通用检查提取到包含其余测试的单独if 中，您发布的代码可以编写得更容易阅读，将$colorLevel['level'] 提取到一个更短的新变量中命名并使使用$colorLevel['level'] 的条件使用相同的规则：

$level = $colorLevel['level'];
if ($colorLevel['name'] == 'ATTR_VPMCV13') {
    // Don't mix '<=' with '>=', always use '<='...
    if (60 <= $level && $level <= 70) {
        $prominentSideNumberArray[] = 8;
    // ... and put the intervals in ascending order
    } elseif (80 <= $level) {
        $prominentSideNumberArray[] = 10;
    }
}

如果有多个if 语句验证$colorLevel['name'] 的不同值，那么使用switch statement 的意图会更清楚：

$level = $colorLevel['level'];
switch ($colorLevel['name'])
{
    case 'ATTR_VPMCV13':
        if (60 <= $level && $level <= 70) {
            $prominentSideNumberArray[] = 8;
        } elseif (80 <= $level) {
            $prominentSideNumberArray[] = 10;
        }
        break;

    case '...':
        // ...
        break;

    default:
        // ...
        break;
}

Answer 2

您可以通过使用三元运算符来实现这一点。看下面的代码：

$prominentSideNumberArray[] = ((($colorLevel['name'] === 'ATTR_VPMCV13') &&
 ($colorLevel['level'] >= 80) )? 10 : (($colorLevel['name'] == 'ATTR_VPMCV13') && 
($colorLevel['level'] >= 60) && ($colorLevel['level'] <= 70)?8:""))  ;

编辑根据 cmets，您必须比较相同的值才能更好地定义名称

$color_name = "ATTR_VPMCV13";
if($colorLevel['name'] == $color_name )
    $prominentSideNumberArray[] = (($colorLevel['level'] >= 80)? 10 : (
($colorLevel['level'] >= 60) && ($colorLevel['level'] <= 70)?8:""))  ;

DEMO 采用不同的方法

编辑

请记住，此解决方案的可读性不如 if-else 语句。