Как правильно сделать роутинг в Laravel?

Question

[Юрий Смит]

Здравствуйте. При создании сайта изначально были роуты для категорий и подкатегорий:

Route::pattern('category', '[a-zA-Z0-9\-]+'); // Такие паттерны у всех роутов
Route::get('/{auto}/{country}/', ['as' => 'country', 'uses' => 'AutoController@country']);
Route::get('/{auto}/{country}/archive/', ['as' => 'country.archive', 'uses' => 'AutoController@countryArchive']);
Route::get('/{auto}/{country}/{mark}/', ['as' => 'mark', 'uses' => 'AutoController@mark']);
Route::get('/{auto}/{country}/{mark}/archive/', ['as' => 'mark.archive', 'uses' => 'AutoController@markArchive']);

В контроллере если данных нету то вызываю App::abort(404); Когда добавил новые роуты, то перестали работать старые (выдавалась 404 ошибка), выше написаны чем прошлые.

Route::get('/{category}/', ['as' => 'category.list', 'uses' => 'ArticleCategoriFrontendController@list']); 
Route::get('/{category}/{article_slug}/', ['as' => 'article', 'uses' => 'ArticleCategoriFrontendController@article']);

Вот у меня и возник вопрос как этого можно избежать, уже использовал стандартные функции как link_to_route, link_to_action но по прежнему ничего. Код:

<?php echo link_to_route('category.list', 'obuchenie', ['obuchenie'] ); ?>
<?php echo link_to_action('ArticleCategoriFrontendController@list', 'obuchenie2', ['obuchenie'] ) ?>
<a href="{{route('category.list',['obuchenie']) }}">obuchenie3</a>

Оказывается оно перекидывает на другой роут

Route::get('/{page}/', ['as' => 'page', 'uses' => PageController@page']);

Comments

[Алексей Уколов]

Покажите все свои текущие роуты в том порядке, в котором они объявлены.

[Юрий Смит]

Алексей Уколов:

Route::get('/{auto}/', ['as' => 'auto', 'uses' => 'AutoController@auto']);
Route::get('/{auto}/archive/', ['as' => 'auto.archive', 'uses' => 'AutoController@autoArchive']);
Route::get('/{page}/', ['as' => 'page', 'uses' => 'PageController@page']);
Route::get('/{auto}/{country}/', ['as' => 'country', 'uses' => 'AutoController@country']);
Route::get('/{auto}/{country}/archive/', ['as' => 'country.archive', 'uses' => 'AutoController@countryArchive']);
Route::get('/{auto}/{country}/{mark}/', ['as' => 'mark', 'uses' => 'AutoController@mark']);
Route::get('/{auto}/{country}/{mark}/archive/', ['as' => 'mark.archive', 'uses' => 'AutoController@markArchive']);
Route::get('/{category}/', ['as' => 'category.list', 'uses' => 'ArticleCategoriFrontendController@list']); 
Route::get('/{category}/{article_slug}/', ['as' => 'article', 'uses' => 'ArticleCategoriFrontendController@article']);

Answer 1

[Александр Талалаев]

Как он узнает {auto} отличается от {page} или {category}? Ответ никак, так как они идентичны поэтому всегда срабатывает auto.
Тебе нужно придумывать либо префикс Route::prefix('auto') (типа auto_{auto}), либо для каждого писать паттерн правило именования(на переменные что формирует путь, ), либо внутри писать обработчик которые если не найдет auto по правилам перекидыват на page и тд).
Короче говоря, изучи https://laravel.com/docs/5.4/routing , есть возможность расписать расписать свою логику работы роутера, но это все усложнит.
Я думаю твой вариант перейти на префиксы и все будет красиво и быстро работать.

Comments

[Юрий Смит]

Я понял, спасибо.

Answer 2

[Алексей Уколов]

Роутер не сможет никак различить /{auto}/, /{page}/ и /{category}/. Вам нужно для каждого раздела сайта сделать постоянный уникальный префикс. Что-то вроде такого: /cars/{auto}/, /pages/{page}/ и /categories/{category}/.