Как построить цепочку роута не прибегая использованию модели для алфавитной литеры?

Question

[powercoder]

Здравствуйте. Подскажите, как построить цепочку роута так, чтобы ссылка на модель была следующей структуры /genres/f/fantasy, не прибегая использованию модели для алфавитной литеры?

Пробовал способом ниже, но проблема заключается в том, что на третьем уровне вложенности, когда мы изменяем литеру, например, так /genres/g/fantasy- не выводится 404 ошибка, страница всё-равно доступна.

PS. У данной цепочки ещё планируется дополнительная вложенность - модель книги

Route::group(['namespace' => 'Books', 'prefix' => 'genres'], function() {
    //Все жанры
    Route::get('/', 'IndexController')->name('pages.books');

    //Все жанры -> {литера}
    Route::get('/{litera}', 'LiteraController', function ($firstLetter = null) {
        return $firstLetter;
    })->where(['litera' => '[a-z0-9]+'])->name('pages.books.litera');

    //Жанр
    Route::get('/{litera}/{genre}', 'IndexController')->name('pages.genre');
});

Comments

[Дмитрий]

Не совсем понятно /genres/f/fantasy что из этого будет параметром

[powercoder]

Дмитрий, первая часть, поправил код

[Дмитрий]

powercoder, то есть - вам надо что бы срабатывало /genres/f/fantasy - ибо первая буква fantasy - f и у нас литера f, но не должно срабатывать /genres/g/fantasy ибо здесь литера g - а не f?

[powercoder]

Дмитрий, да, верно, а ещё после fantasy будет книга, и нужно также, чтобы она была доступна по одному url без дублей

Answer 1

[Григорий Васильков]

Как я вижу тут два способа.

Способ первый. Дерево есть дерево. Сначала строится дерево всего что может быть, потом контроллеры работают с деревом. Если нету в дереве - 404. Здесь будет работать механизм SubstituteBindings, когда ты на входе пишешь LiteraController::literaAction(Category $model), которая по твоей букве ищет категорию, у которой slug - твоя литера - и т.к. её нет - автоматически выбрасывается 404. Делать труднее, изменять труднее, зато потом получаешь все плюшки работы с деревом и возможностью по нему всячески бегать, подсчитывать суммы элементов в нем и так далее.

Способ второй. Твой. Роут работает с любой литерой, а внутри роута pages.books.litera тянет всё что начинается на букву такую-то, и если ничего не вытянул принудительно кидает 404. Это нормальное абсолютно поведение, когда ты бросаешь 404 вручную, а не автоматически. Делать легче, менять легче, плюшек нет. Зато дешевле. Заказчик захочет изменить - надо ПЕРЕ-делывать и наращивать функционал, доделывать - не прокатит.

Comments

[powercoder]

Под деревом вы имеете в виду, что нужно создавать модель для литеры и привзяывать к ним жанры? С SubstituteBindings пока что не знаком, пока что джун в Laravel, буду узнавать про неё. Возможно, будут какие-либо полезные ссылки у вас по этой теме?

[Григорий Васильков]

Не, если делать первым способом то у вас таблица и модель Categories, (и может быть таблица для дерева и модель к ней CategoryClosure либо вы в категории делаете поле `path` или например `url` и в нем держите путь через слеш). И вы сначала отдельным скриптом пробегаетесь по всем вашим книгам и заполняете таблицу categories по вложенности, а потом с ней работаете.

Биндинги это не кнопка, это есть такой мидлвар, который подкидывает на фход действия модельку, запросив из базы по тому, что является частью роута (то есть по стандарту на вход функции приходит то, что есть в роуте, а этот биндинг использует это, чтоб в базе поискать и дать тебе сразу модель), и его еще настраивать надо (т.к. у тебя в роуте идет slug, а он по-дефолту ищет по id). Там внутрянка, вам тогда лучше сделать вторым способом.

Answer 2

[Дмитрий]

Ну если вы хотите валидировать это на уровне роутов чисто теоретически можно прописать /genres/{literaAndGenre} где регуляркой проверять что первый символ совпадает с начальным символом слова. Регулярка для этого простая
([a-z])\/\1[^\/]+
Но ввиду того что laravel эту регулярку под себя подправляет - она бьется и не работает в простом виде. Но так сказать нет препятствий патриотам для костылирования.

Route::any('/genres/{literaGenre}', function ($literaAndGenre) {
dump('Got it', explode('/', $literaAndGenre));
})->where(['literaGenre' => '(?P<litera>[a-z]{1})\/(?P=litera)[^\/]+']);

/genres/f/fantasy - будет работать
/genres/g/fantasy - не будет работать

Но есть один минус - от такого костылирования умирает route:cache, надо патчить

З.Ы. Решение дерьмо конечно. Не легче ловить все подобное в контроллер и там при не совпадении выбрасывать 404? Или на уровне middleware или custom model binding?