关于简化 PHP 语句的建议

Question

我是 PHP 新手，正在为 Joomla K2 项目布局编写模板文件。

我配置了一个“额外字段”$extrafields[15]，其输出为“是”、“否”或“”。 $extrafields[16] 是一个文本字符串。

我有这段代码，它可以工作，但会就如何简化它提出建议，因为我知道它可能有点粗糙！

if (!empty($extrafields[15])): 
    if ($extrafields[15] == "Yes") {
        echo "<span class=sgl-bold>Sponsored by:&nbsp;</span>"; 
    }

    if ($extrafields[15] == "Yes"):
        if (!empty($extrafields[16])):
            echo $extrafields[16]; 
        endif;
        echo "<br>";
    endif;
endif;

Answer 1

我倾向于做这样的事情：

if(!empty($extrafields[15]) && !empty($extrafields[16])){ 
    if($extrafields[15] == "Yes"){
        echo "<span class=sgl-bold>Sponsored by:&nbsp;</span>"; 
        echo $extrafields[16]; 
        echo "<br>";
    } //endif not empty
} //endif yes

Answer 2

您可以通过一些简单的调整使您的代码更简洁：

• 尽可能去掉多余的 if 子句，
• 在有意义的地方组合 if 子句条件，并且
• 不要将alternative syntax 用于您的控制结构以减少噪音。

以下 sn-p 保留了与您最初尝试相同的功能，但更容易理解。

if (!empty($extrafields[15]) && 'Yes' === $extrafields[15]) {
    echo '<span class=sgl-bold>Sponsored by:&nbsp;</span>';
    if (!empty($extrafields[16])) {
        echo $extrafields[16];
    }
    echo '<br>';
}

也就是说，从上下文来看，您可能希望使用此处发布的solution BigScar。

为了使这个 sn-p 更容易理解，您应该考虑处理数据结构（尽管我认为这是 Joomla 强加给您的）：

• 而不是像 $extrafields[16] 这样的数值数组中的值，而是使用像 $showSponsor 这样的说话变量名称，并且
• 使用布尔值true、false 和null，而不是'Yes'、'No' 和'' 的字符串值。

记住：

There are only two hard things in Computer Science: cache invalidation and naming things.

Answer 3

if (@$extrafields[15] == "Yes") { 
    echo "<span class=sgl-bold>Sponsored by:&nbsp;</span>"; 
    echo @$extrafields[16]; 
    echo "<br>";
}