我即将将一些代码迁移到PSR-2标准.在我的代码中,我有if多个行的语句作为表达式:
if ( $field->getBlockMode() == FieldInterface::BLOCK_MODE_HIDEVAR && !isset($this->enabledBlocks[$field->getBlock()]) ) { }
写这样的表达式的最佳做法是什么?
如何使它成为单行代码以避免该问题并使语句更具可读性:
$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()]) }
这样,优化&&仍然发生.
版权声明:本文内容由互联网用户自发贡献,该文观点与技术仅代表作者本人。本站仅提供信息存储空间服务,不拥有所有权,不承担相关法律责任。如发现本站有涉嫌侵权/违法违规的内容, 请发送邮件至 dio@foxmail.com 举报,一经查实,本站将立刻删除。