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()ステートメントでそれらの変数を使用します。

すなわち：

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

（注：これはより良い可能性があるため、最初のテストをトリプルイコールに変更しました。それでも問題が解決しない場合は、自由に元に戻してください）