Pull to refresh

Comments 12

Статью не читай@сразу отвечай


Я написал, что логику нужно в сервис выносить.
Просто все это по своему делают, тут конкретные советы давать не стал.

А лучше бы дать.
Ведь те, кто на самом деле «не читал», но изучает, скопипастят это в проект.
Не надо так.

Те, кто не читал — должен прочитать. Это не замена документации, лишь выдержка в виде шпаргалки.
Архитектура бизнес логики не рассматривается в документации, я решил следовать этому.


Про сервис слой я сделаю отдельную статью, спасибо за идею.

А можно попробовать, например, расширить эту статью циклом: «это было для проверки ответа, читать продолжение...». А там раскрыть тему про репозитории, сервисы, di.
Тех, кто копипастит не читая и не думая, уже ничего не спасёт.

тот код что написан в методах контроллера в статье и так короткий. Для чего контроллер делать еще тоньше?

Если планируете дальше развивать статью, наверно можно добавить:
1 после установки добавить вызов:
php artisan preset none
Удаляет не нужные UI компоненты (vuejs, react)

2 Route::resource поменял на вызов apiResource
Route::apiResource('/games', 'GameController');
Добавил в статью, спасибо.
Посредника
\App\Http\Middleware\ForceJsonResponse::class
перед bindings лучше добавить иначе ошибки будут html формате

А смысл? Запрашивай джейсон и будет в ответе тебе джейсон. Не понятно желание форса.

Я так понимаю вместо ForceJsonResponse, лучше использовать «Eloquent: API Resources» который добавили в laravel 5.5?
Sign up to leave a comment.

Articles

Change theme settings