Как сделать возможность выбора расположения пункта меню?

Question

[Дмитрий]

Добрый вечер.
Возник вопрос, приблизительно представляю как реализовать, но правильное ли это решение?
Всё просто.
Обыкновенный сайт на yii2. Использую виджет Nav для вывода меню. Самих меню на сайте может быть несколько, в "шапке" сайта, в "подвале", слева и справа. Пункты меню редактируются в админке. Поэтому виджет Nav использую следующим образом.

<?=
     Nav::widget([
         'items' => Catalog::getMenu(null)
     ]);
 ?>

Метод getMenu(null) формирует в модели двухуровневое меню, параметром можно передать какие именно пункты будут отображаться в данном меню. То есть, можно передать или "head" или "middle" или "left" или "right".
В админке, при создании нового пункта меню я указываю, в каком месте сайта этот пункт выводить, соответственно в выпадающем списке в форме выбираю или "head" или "middle" или "left" или "right".
Но один и тот же пункт может выводится как в "шапке" сайта, так и в "подвале".
Я думаю, что в форме создания нового пункта переделать выпадающий список вариантов на checkboxList, чтобы была возможность выбора нескольких вариантов. И создать отдельно таблицу, где хранить id пункта меню и место, в какой части сайта его выводить.
Но насколько это правильно? Подскажите, пожалуйста, правильное решение.

Comments

[Максим Федоров]

На лицо явная связь многие ко многим... значит связующей таблице быть!
Если конечно реляционная БД

[Дмитрий]

Максим Федоров, понятно, а нужно ли в связующей таблице выставлять для id пункта и места расположения primaryKey? Или в данном случае это не обязательно?

[Максим Федоров]

slo_nik, ну в таких таблицах нужно оба поля выставить в primaryKey

[Дмитрий]

Максим Федоров, и ещё такой вопрос.
В модели вот эти самые места расположения заданы в виде констант.

const LOCATE_TOP = 0;
    const LOCATE_MIDDLE = 1;
    const LOCATE_BOTTOM = 2;
    const LOCATE_RIGHT = 3;
    const LOCATE_LEFT = 4;

Правильно ли будет передавать в метод такой параметр?
Catalog::getMenu(Catalog::LOCATE_TOP)
И в правилах валидации проверять, чтобы это значение входило в нужный массив

['locate', 'in', 'range' => array_keys(self::getLocateArray())],

[Максим Федоров]

slo_nik, мне кажется это на отдельный вопрос по хорошим парктикам тянет... я слабоват...

но я бы сделал лучше 4 метода: getMenuTop(), getMenuRight(),
если следовать информационному эксперту из SOLID,
как мне кажется так понятней: Catalog::getMenuRight()
чем запись Catalog::getMenu(Catalog::LOCATE_RIGHT )

[Дмитрий]

Максим Федоров, надо подумать.
Пока на сайте четыре меню. Одно, более менее статично. Остальные три отличаются только html структурой. В верхнем есть выпадающие списки, в двух нижних нет.
Получается, что код будет повторяться, меняться будет только расположение и html.

[Дмитрий]

Максим Федоров, или сделать два метода, один будет формировать многоуровневое меню, а второй в один уровень?

[Максим Федоров]

slo_nik, а фиг его знает, переусложнять на ровном месте и правда не стоит конечно

[Максим Федоров]

по идее сделать бы так можно:
Catalog::getMenu('RIGHT')

тогда можно без повторений сделать

[Максим Федоров]

еще вот что в голову пришло для универсального метода получения меню:
php.net/manual/ru/function.get-defined-constants.php

[Дмитрий]

Максим Федоров, я так понимаю, что "RIGHT" это значение константы?

[Максим Федоров]

slo_nik, я думал как-то добавить к названию константы в методе, чтобы он был универсальным... не смотрел, можно ли так :) и вот набрел в ходе поисков на php.net/manual/ru/function.get-defined-constants.php

[Дмитрий]

Максим Федоров, да, спасибо, Вы уже выше дали эту же ссылку))).
Сейчас почитаю, а потом на yiiframework.ru/forum загляну, может ещё там что-то интересное мне расскажут. Хотя не факт.

[Дмитрий]

Максим Федоров, прочитал, но ясности не внесло в мою голову)