Странный код в Nav yii2, как можно сделать альтернативно?

Question

[dev400]

Обнаружил такую штуку

Yii::$app->user->isGuest ? "" : (
                            ['label' => 'Главная', 'url' => ['/site/index']] 
                        ),
                        Yii::$app->user->isGuest ? "" : (
                            ['label' => 'Сотрудники', 'url' => ['/site/about']]),
                        
                        Yii::$app->user->isGuest ? "" : (
                            ['label' => 'Лиды', 'url' => ['/site/contact']]),
                        
                        Yii::$app->user->isGuest ? "" : (
                            ['label' => 'Заказы', 'url' => ['/site/contact']]),
                        
                        Yii::$app->user->isGuest ? "" : (
                            ['label' => 'Настройки', 'url' => ['/site/contact']]),

И сгруппировать их не получается в одно условие... Как можно переделать этот ужас?

Comments

[Евгений Вольф]

Если Вы напишите код целиком, или объясните, куда в итоге деваются все эти массивы - я думаю, многим гораздо легче будет ответить на Ваш вопрос.

[Евгений Вольф]

P.S. Мне кажется вторая буква ("т") в заголовке вопроса - лишняя :))

[dev400]

Евгений Вольф: не понял про букву т)))

[Евгений Вольф]

dev400: ну если букву "т" (вторую по счёту, в первом слове) из заголовка вопроса удалить, то глядя на код - вариант "без буквы т" будет лучше описывать всю эту боль :D

[dev400]

Евгений Вольф: )))

[dev400]

Евгений Вольф: в общем то попросили доделать меню

Answer 1

[bears]

Там скорее всего что-то типа такого:

$nav = [
    'items' => Yii::$app->user->isGuest ? '' : (
                    ['label' => 'Главная', 'url' => ['/site/index']] 
                ),
                Yii::$app->user->isGuest ? "" : (
                    ['label' => 'Сотрудники', 'url' => ['/site/about']]),
                
                Yii::$app->user->isGuest ? "" : (
                    ['label' => 'Лиды', 'url' => ['/site/contact']]),
                
                Yii::$app->user->isGuest ? "" : (
                    ['label' => 'Заказы', 'url' => ['/site/contact']]),
                
                Yii::$app->user->isGuest ? "" : (
                    ['label' => 'Настройки', 'url' => ['/site/contact']]),
];

Это то же самое что и:

if (Yii::$app->user->isGuest) {
    $nav = [
        'items' => [],
    ];
} else {
    $nav = [
        'items' => [
            ['label' => 'Главная', 'url' => ['/site/index']] ,
            ['label' => 'Сотрудники', 'url' => ['/site/about']],
            ['label' => 'Лиды', 'url' => ['/site/contact']],
            ['label' => 'Заказы', 'url' => ['/site/contact']],
            ['label' => 'Настройки', 'url' => ['/site/contact']],
        ],
    ];
}

Какой-то извращенец писал код

Comments

[Евгений Вольф]

> Какой-то извращенец писал код
Полностью Вас поддерживаю в этом высказывании! :))

*лично я не сразу понял, зачем там столько проверок одного и того же Yii::$app->user->isGuest и начал искать какой-то сокральный смысл...

[bears]

Евгений Вольф: А мне сразу непонравились скобки, уж слишком их много, и в этом виде код не рабочий, так только в массиве можно сделать. А насчет кучи проверок согласен, вообще жесть )

Answer 2

[Борис Якушев]

У меня так:

<?php
NavBar::begin([
    'brandLabel' => Yii::$app->name,
    'brandUrl' => Yii::$app->homeUrl,
    'options' => [
        'class' => 'navbar-default navbar-fixed-top',
    ],
]);
$menuItems = [
    ['label' => 'Обо мне', 'url' => ['/site/about']],
    ['label' => 'Контакты', 'url' => ['/site/contact']],
];
if (Yii::$app->user->isGuest) {
    $menuItems[] = ['label' => 'Регистрация', 'url' => ['/site/signup']];
    $menuItems[] = ['label' => 'Вход', 'url' => ['/site/login']];
} else {
    $menuItems[] = ['label' => 'Выйти (' . Yii::$app->user->identity->username . ')', 'url' => ['/site/logout'], 'linkOptions' => ['data-method' => 'post']];
}
echo Nav::widget([
    'options' => ['class' => 'navbar-nav navbar-right'],
    'items' => $menuItems,
]);
NavBar::end();
?>