Laravel 8+ Маршрутизатор не видит контроллер, хотя всё сделал как написано на laravel.com?

Question

[echizon]

Создал проект

test (composer create-project laravel/laravel test)

Создал контроллер (php artisan make:controller HomeController), также добавил action

public function index(){
        return 'Hello world!';
    }

Добавил запись в web.php Route::get('/home', 'HomeController@index');
Иду на "site-name.com/home"

Illuminate\Contracts\Container\BindingResolutionException
Target class [HomeController] does not exist.

То есть маршрут не видит контроллер, подскажите в какую сторону мне копать?
Искал вопросы по типу route don't find controller laravel, видел что люди создавали контроллер сами и эти контроллеры laravel не видел, чтобы всё заработало они через php artisan создавали и всё работало. Я пытался и так и сяк не работает. Что интересно, у меня есть другой проект и в нём я могу создавать новые контроллеры и добавлять пути и всё работает. Использую openServer для обоих проектов.

Comments

[JhaoDa]

Покажи, где «написано на laravel.com» для версии 8+, что надо указывать экшен-контроллер именно в виде 'HomeController@index'?

Answer 1

[Сергей delphinpro]

Вы читаете очень старые учебники. Рекомендую сверяться с документацией.

Роуты теперь не нужно задавать строковой записью. Пишите примерно так:

use App\Http\Controllers\HomeController;

Route::get('/home', [HomeController::class, 'index']);

https://laravel.com/docs/8.x/routing#the-default-r...
https://laravel.com/docs/8.x/controllers#basic-con...

Comments

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

Я поправлю.

Существует разница между задать роут каллаблом и строкой.

Каллабл требует обьект, объект требует use, use требует autoloader, autoloader хочет тики процессора, чтение с диска и память. Пока роутов 10 - бог с ним. Но когда роутов полтысячи - там use только штук 50. Это много.

Строка в этом плане лучше, т.к. подгрузка класса произойдет только в момент когда будет резолвится Route::namespace().

Кроме того когда мы задаем каллабл массивом - это считается статическим методом. Если мы хотим создать динамический - в первый параметр каллабла обьект надо. И когда мы пишем свой инжектор зависимостей мы вдруг понимаем, что нет такой формы записи, чтобы отличать статику от динамики, но при этом не передавать объект, и запись через собаку кажется вполне разумной.

Разве что в ларе 8 и более сделали чтобы если первый параметр класс - то автовайрить его и создавать, нарушив немного идею каллабла, то может быть, но по факту с этим есть трабла на уровне пыхи.

====

Я думаю что у ТС нужно вызвать `php artisan optimise`

[Сергей delphinpro]

Григорий Васильков, конструкция use сама по себе не приводит к загрузке класса.

Строка в этом плане лучше, т.к. подгрузка класса произойдет только в момент когда будет резолвится Route::namespace().

Подгрузка класса и так в любом случае происходит только когда отрезолвится нужный роут. Вы можете спокойно прописать несуществующий класс, и пока вы не перейдете на страницу этого роута, вы не словите эксепсш class not exists.

Оптимизацию обычно только на проде запускают.

[echizon]

Спасибо помогло. Для нового проекта так и работает. Но в старом проекта строчное работало и работает, когда новые контроллеры добавляю. Не подскажете в чём может быть причина?

[Сергей delphinpro]

echizon, в старых версиях было свойство $namespace в RouteServiceProvider. Там был записан корневой неймспейс для коонтроллеров. Когда строка парсилась, то к имени класса добавлялся этот неймспейс и класс успешно реезолвился.
В восьмерке все еще можно использовать строки (может и в девятке, не помню). Но нужно раскоментировать это свойство



либо использовать в строковой записи полное имя класса

Route::get('/home', 'App\\Http\\Controllers\\HomeController@index');

[yarovikov]

Сергей delphinpro, в девятке работает

кстати, а как для девятки корректно группировать роуты?

Route::group(['namespace' => 'App\Http\Controllers\Post'], function () {
	Route::get('/posts', IndexController::class)->name('post.index');
	Route::get('/posts/create', CreateController::class)->name('post.create');
});

[Сергей delphinpro]

yarovikov, группировка по неймспейсу не особо актуальна, на мой взгляд.
Все способы группировок описаны здесь https://habr.com/ru/post/671018/

[yarovikov]

Сергей delphinpro, огонь, спасибо

[Сергей delphinpro]

yarovikov, в таком варианте должно работать так (без указания неймспейса в роутпровайдере)

Route::group(['namespace' => 'App\\Http\\Controllers\\Post'], function () {
  Route::get('/posts', 'IndexController')->name('post.index');
  Route::get('/posts/create', 'CreateController')->name('post.create');
});

\\ Не забываем экранировать слэши в строковых неймспейсах

[yarovikov]

Сергей delphinpro, да, у меня так и было, но закрались сомнения, что не по рекомендациям девятки

[yarovikov]

Сергей delphinpro, а как насчет такой записи?

Route::controller(AdminController::class)->prefix('admin')->middleware(['admin'])->group(function () {
	Route::get('/', 'index')->name('admin.index');
	Route::get('/posts', 'posts')->name('admin.posts.index');
});

[Сергей delphinpro]

yarovikov, Это кажется только в девятке появилось

[Сергей delphinpro]

Хотя нет. Проверил на 8.83 - работает. И даже переходы в IDE по ctrl+click работают.

Хотя в моём случае получилось семь строк кода, вместо пяти =)

Route::controller(TourOrderController::class)->group(function () {
  Route::get('confirm', 'view')
    ->name('order.confirm')
    ->breadcrumbs(fn(Trail $trail, $uuid) => $trail->parent('order.form', $uuid)
    ->push(__('crumbs.order_data_validation'), route('order.confirm', $uuid)));
  Route::post('confirm', 'complete');
});
// Route::get('confirm', [TourOrderController::class, 'view'])
//   ->name('order.confirm')
//   ->breadcrumbs(fn(Trail $trail, $uuid) => $trail->parent('order.form', $uuid)
//     ->push(__('crumbs.order_data_validation'), route('order.confirm', $uuid)));
// Route::post('confirm', [TourOrderController::class, 'complete']);