使用 PHP（对于 Joomla）将多个 if 语句压缩为一个？

Question

以防万一有人不熟悉 Joomla，countModules 是一个内置函数。另外请记住，我对 PHP 不是很熟悉。而且我对使用它还很陌生。请尽量保持任何解决方案简单易懂。谢谢。

我的问题是我可以浓缩一下吗：

if($this->countModules('Top1A and Top1B') >= 1) $modCellRow1 = "col2";
if($this->countModules('Top1B and Top1C') >= 1) $modCellRow1 = "col2";
if($this->countModules('Top1A and Top1C') >= 1) $modCellRow1 = "col2";

进入这个：

if ($this->countModules('Top1A and Top1B') >= 1) || ($this->countModules('Top1A and Top1B') >= 1) || ($this->countModules('Top1A and Top1C') 
{
$modCellRow1 = "col2";
}

如果您知道 Joomla 和 countModules...或者即使您不知道，请记住，我正在检查每个模块位置是否有任何内容，而不是计算实际模块的数量。换句话说，我确实需要使用 AND 作为运算符。另外，我不想包含所有三个，因为这是我的代码中的另一行......例如：

if($this->countModules('Top1B and Top1B and Top1C') >= 1) $modCellRow1 = "col3";

我正在检查那些其他陈述是在两个模块位置中是否有东西......（即三个中只有两个等于或大于一个......但不是全部三个） .

补充意见/问题：

这会起作用吗？

$topRow1Count = 0;
foreach(array('Top1A','Top1B','Top1C','Top1D') as $position) {
    if ($this->countModules($position)) {$colCount += 1;}
}
$modColCountTopRow1 = "col" . $topRow1Count;
if ($topRow1Count == 0) { $modTopRow1 = " hidden"; }

$topRow2Count = 0;
foreach(array('Top2A','Top2B','Top2C','Top2D') as $position) {
    if ($this->countModules($position)) {$colCount += 1;}
}
$modColCountTopRow2 = "col" . $topRow2Count;
if ($topRow2Count == 0) { $modTopRow2 = " hidden"; }

if ($topRow1Count + $topRow2Count == 0) { $topModCont1 = " hidden"; }

Answer 1

是的，它会起作用。但是让我们考虑一下性能和可读性。

性能

对于“col2”案例，您调用countModules 3 次，对于“col3”案例，假设您从“col1”开始，它将是4 次调用，每个调用都有多个“位置”要检查。 现在countModules() 代码已经非常优化，回退到单个静态类，不会在每次迭代时查询数据库；但是即使在模块很少的小型站点中，您的程序仍然比应有的速度慢 10 倍。

可读性和代码质量

您可能已经读过代码不应看起来像是复制和粘贴的。这至少是有道理的，因为它更容易阅读避免重复的代码，另外它更不容易出现拼写错误。

本着浓缩多条语句的精神，与其连接线，不如考虑一下：

$colCount = 0;
foreach(array('Top1A','Top1B','Top1C') as $position) {
    if ($this->countModules($position)) {
        $colCount += 1;
    }
}
$modCellRow1 = "col" . $colCount;

如果没有模块位于任何位置，这将返回"col0"，如果模块仅存在于一个位置，则返回"col1"，依此类推。

我写了它，所以它最容易阅读，