php - 长 if 语句和 PSR-2

Question

我即将将一些代码迁移到 PSR-2 标准。在我的代码中，我有带有多行的 if 语句作为表达式：

if (    $field->getBlockMode() == FieldInterface::BLOCK_MODE_HIDEVAR &&
        !isset($this->enabledBlocks[$field->getBlock()])
) {

}

编写此类表达式的最佳做法是什么？

Answer 1

如何使它成为一个单行符以避免该问题并使语句更具可读性：

$blockModeIsHidevar = $field->getBlockMode() == FieldInterface::BLOCK_MODE_HIDEVAR;
$blockNotEnabled = !isset($this->enabledBlocks[$field->getBlock()]);

if ($blockModeIsHidevar && $blockNotEnabled) {

}

选择：

我通常用方法来做，这可能看起来像这样：

if ($this->blockModeIsHidevar($field) && $this->blockNotEnabled($field)) {

}
// ...
private function blockModeIsHidevar($field)
{
    return $field->getBlockMode() == FieldInterface::BLOCK_MODE_HIDEVAR
}
private function blockNotEnabled($field)
{
    return !isset($this->enabledBlocks[$field->getBlock()])
}

这样，优化&&仍然发生。

Answer 2

首先将其提取为较短的布尔表达式，然后在您的if()语句中使用这些变量。

IE：

$hideVarMode = $field->getBlockMode() === FieldInterface::BLOCK_MODE_HIDEVAR;
$enabledBlock = !isset($this->enabledBlocks[$field->getBlock()];
if($hideVarMode && $enabledBlock) {
    ....
}

（请注意，我还将您的第一个测试变成了三等式，因为这可能会更好；如果这对您不起作用，请随时将其改回）