Long if语句和PSR-2

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

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

}

写这样的表达式的最佳做法是什么？

来源

2013-02-19 Johni

喜欢，但你想要的。建议使测试语句缩短：无论您如何编写测试语句，都将无法阅读。 – KingCrunch 2013-02-19 14:42:06

问题是，如果“if（”空格不允许，所以我不能将表达式设置为与以下相同的hoirizobntal位置。并且据我了解，我无法在“if（”ether。 – Johni 2013-02-19 14:44:51

如何使其成为一个班轮，以避免这一问题，使表述更加可读：

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

if ($blockModeIsHidevar && $blockNotEnabled) { 

}

备选：

我通常用的方法去做，这可能 looke像这样：

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()]) 
}

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

来源

2013-02-19 14:45:55

这不会影响性能吗？ – Johni 2013-02-19 14:46:54

如果第二个条件在第一个条件为false时没有得到评估，您可以将检查提取到方法/函数而不是变量。 – 2013-02-19 14:48:39

+1。看起来你跟我说的几乎一样 – SDC 2013-02-19 14:50:24

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

即：

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

（注意，我也把你的第一个测试到三相等，因为这很可能是更好的;随意更改回来，如果不为你工作）

来源

2013-02-19 14:46:37 SDC

Long if语句和PSR-2

回答

相关问题