Если в одном контроллере Laravel нужно получить данные от множества моделей — как лучше группировать код?

Question

[Роман]

Здравствуйте.

Допустим мне в IndexController нужно получить и отправить во вьюху данные из моделей Car, Wheel, Glass, Door - как принято делать:

- Просто получать все данные в одной функции и отправлять во вьюху?

- Или на каждую модель писать отдельную функцию типа getCars - а потом эти функции вызывать внутри метода, который прописан в роутере, и таки отправлять во вьюху?

Как вы обычно делаете такое и почему?

Спасибо.

Answer 1

[jazzus]

Если код не повторяется, то в одном методе и отправлять во вьюху.
Если повторяется в пределах контроллера, я выношу в отдельный метод

public function getUsers()
{
     return User::all();
}

И потом получаю в любом методе
$this->getUsers();
Если повторяется в пределах приложения, выношу в трейты. Это удобно т.к. меняешь в одном месте, а не в 10.
Если код не сложный или изменений не предвидится, никуда не выношу, даже если повторяется.

Comments

[Роман]

А файлы с трейтами куда складываешь?

[Кирилл Арутюнов]

Роман, куда удобнее, туда и кладите :–) Ларавел вас не ограничивает своей структурой. Логично было бы положить в app\Traits, например.

[jazzus]

Роман,
App\Traits
Можно подключать не только к контроллерам, но и моделям

use App\Traits\ModerationTrait;
class User extends Authenticatable implements MustVerifyEmail
{
    use ModerationTrait;
}

Прописать один раз в одном месте

public function isApproved()
{
   return $this->status_id == Status::ID__APPROVED;
}

Подключить 1 трейт ко всем моделям со статусами. И у всех моделей можно делать

If($model->isApproved())
{
…
}

Т.е. обращаться, как к методу модели, но не нужно у всех моделей дублировать этот и другой код.