Можно ли так выбирать данные?

Question

[Дмитрий Демиденко]

Здравствуйте.
Часто вижу такую реализацию и задумался нормальна ли такая практика в Laravel.

Есть например переменная, полученная при помощи отношения

// Переменная в контролере
$favorites = $user->favorites; // Получает все записи, которые пользователь добавил в избранное

// метод отношения в модели User
public function favorites(){
        return $this->hasMany('\App\Models\Favorite');
}

// К-во избранных исполнителей в представлении
Исполнители ({{ $favorites->where('performer_id', '<>', null)->count() }})

Интересует именно то, нормально ли дополнительно в представлении вызывать метод where.
Спасибо!

Answer 1

[Stanislav]

Нет. Как минимум следует сделать count в контроллере. А еще лучше воспользоваться Scope ларавеля и писать нечто вроде:

public function scopeHasPefrormer($query)
{
    return $query->where('performer_id', '<>', null);
}

$performersCount = $user->favorites()->hasPerformer()->count();

View отвечает за отображение данных, а не за их поиск.

Comments

[Дмитрий Демиденко]

А если это же описать на уровне контролера? Или всё же лучшей практикой для этого остается использовать модель и в контролере просто вызывать метод?

Допустимо ли для такой задачи, написать например статический метод в модели Favorite, который будет возвращать нужные данные, а в контролере уже передавать во вью на вывод?

return view('profile.my_favorites', [
            'locale'    => $locale,
            'user'	    => $user,
            'favorites' => [
                'performers' => $favorites->where('performer_id', '<>', null)
            ]
        ]);

[Stanislav]

Дмитрий Демиденко, Local scope прописывается непосредственно в модели, а global scope - в отдельном классе. В вашем конкретном случае сильно париться по этому поводу - оверинжиниринг. Код, что у вас в примере - более предпочтителен, нежели тот, что в вашем основном вопросе. (Еще count() забыли).

[Дмитрий Демиденко]

Stanislav, спасибо за ответ, теперь моя совесть спокойна. Count не забыл. У меня в этом же шаблоне будут выводится избранные "записи" в табах.

[Stanislav]

Дмитрий Демиденко, думать о презентере и о более адекватной инкапсуляции стоит тогда, когда запросов таких у вас много и когда код перестает быть простым. А для таких нескольких запросов - достаточно и прямо в контроллере всё организовать.