三元内部条件不良做法

Question

将

三元放在条件中是不好的做法吗？

请考虑以下代码：

        <?php if ((isset($hideEditButton) ? $hideEditButton : true) && $this->request->session()->read('Auth.User.id') == $user->id): ?>
            <div class="hr-line-dashed"></div>
            <div class="row">
                <div class="col-sm-4">
                    <?= $this->Html->link(__('Edit Profile'), ['action' => 'account'], ['class' => 'btn btn-xs btn-primary']) ?>
                </div>
            </div>
        <?php endif; ?>

我关心的代码的特定部分在条件内：(isset($hideEditButton) ? $hideEditButton : true)

有没有更好的方法来实现我上面尝试做的事情？我很想知道，谢谢！

Answer 1

是的，有更好的方法可以做到这一点。您的陈述

if ((isset($hideEditButton) ? $hideEditButton : true) && $this->request->session()->read('Auth.User.id') == $user->id)

可以重写为

if ((!isset($hideEditButton) || $hideEditButton) && $this->request->session()->read('Auth.User.id') == $user->id)

在 PHP 7 中，你可以更进一步，使用 null 合并运算符：

if ($hideEditButton ?? true) && $this->request->session()->read('Auth.User.id') == $user->id)

在这种情况下，三元运算符只会使读取条件变得更加困难。在我看来，这尤其违反直觉，因为您在运算符的第三部分中使用了文字true;这与你通常期望看到true结果(第二个点(相反。

Answer 2

非常多。代码变得多余且难以阅读。