Какой метод лучше и почему?

Question

[Сухроб Саидов]

Какой вариант лучше?
Мне привычно работать с моделью прямо в контроллере (правда почитал недавно что это очень плохо и т.д, но почему это плохо так и не понял)

Вот простой пример:
PostController

use App\Post;
//...
public function create(Request $request)
{
    //validate...
    $post = new Post;
    $post->slug = $request->slug;
    $post->name = $request->name;
    $post->story = $request->story;
    $post->save();

    return redirect()->back();
}

И недавно увидел один ознакомительный проект где немного все иначе:
PostController

use App\Post;
//...
public function __construct(Post $model)
{
    $this->model = $model;
}

public function create(Request $request)
{
    $post = new $this->model;
    $post->slug = $request->slug;
    $post->name = $request->name;
    $post->story = $request->story;
    $post->save();

    return redirect()->back();
}

Правда это укороченная версия ну а так вот пример:
BlogController
BlogRepository

Answer 1

[D3lphi]

Потому что контроллер по своей сути не должен работать с данными, не должен содержать логику. Его задача - принять запрос, возможно, проверить данные на валидность и отдать ответ. Всё! Больше он делать ничего не умеет. Вся бизнес логика должна выноситься в сервисный слой. А из контроллера вы будете лишь вызывать метод этого сервиса и получать ответ. В принципе, в контроллере можно писать логику, если код короткий, тогда, конечно, создавать сервис будет излишним. С данными должен работать репозиторий (как по ссылке, которую вы привели)! Не нужно делать этого в контроллере.

Мне привычно работать с моделью прямо в контроллере

Может быть вам и привычнее, но это не правильно. Контроллеры разрастаются и становятся жирными. Отсюда же вытекает и то, что вам приходится дублировать код.

$post = new Post;
$post->slug = $request->slug;
$post->name = $request->name;
$post->story = $request->story;
$post->save();

Этот кусок вам, возможно, потребуется использовать еще где-то. И да, вот такой способ, если вашей целью является просто вставка данных, юзать не стоит. В этом нет смысла. Лучше воспользоваться методом create() или же insert().

Comments

[Сухроб Саидов]

Можете пример привести как правильней работать, я уже многое успел сделать а тут такое увидел, теперь все переписывать T_T

[D3lphi]

Сухроб Саидов: пример чего?

[Сухроб Саидов]

D3lphi: Пример как работать с репозиторей контроллером и моделью. Если не сложно, простой пример)

[D3lphi]

Сухроб Саидов: Я же уже все описал выше. Вызываете из контроллера метод сервиса, который выполняет определенные действия. Получаете ответ от него, возможно, ловите исключения. Отдаете пользователю ответ.

// Экшен контроллера
public function editPost(Request $request, PostService $service)
{
     $postId = $request->get('id');
     $postBody-> $request->get('body');    
     $result = null;

     try {
          $result = $service->edit($postId, $postBody)
     } catch (PostNotFoundException $e) {
          // Отдаем ответ пользователю, так как пост с таким идентификатором не найден
     }

     // Отдаем ответ пользователю, о том, что пост успешно отредактирован
}

А вот в PostService вы уже пишите логику изменения поста. Этот же сервис будет работать и с репозиторием. Почитать подробнее о паттерне сервисный слой можно ТУТ.

[Сухроб Саидов]

D3lphi: спасибо за ответ это многое прояснило

[D3lphi]

Дмитрий Мязин: если под моделью вы понимаете сущность, то да.

[Сухроб Саидов]

D3lphi: а разве не controller -> service -> model?