Странный код в 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]

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