Есть ли аналог scope для коллекций Laravel?

Question

[stdio96]

Потратив некоторое время на поиск ответов в гугле, я так и не нашел ответа на следующий вопрос.

Есть ли аналог scope для коллекций Laravel?

Среди ответов, которые я прочел, было указано, что можно использовать и scope для моделей (не коллекций). К примеру: мне надо получить все активные (со статусом 1) и неактивные (статус 0) коментарии. Как это делается, варианты:

1. $post->comments - получаю все комментарии (relation создан в модели)
   
2. $post->activeComments() - все активные комментарии (scope создан заранее)
   $post->inActiveComments() - все неактивные комментарии (scope создан заранее)
   

При использовании варианта 1, в коде я могу получить оба типа комментариев:

$active_comments = $post->comments->where('status', 1);
$incative_comments = $post->comments->where('status', 0)

В чем, собственно, вопрос: при использовании варианта 1 на странице выполняется всего 1 SQL-запрос к БД. При использовании вариантов 2 и 3 - 2 SQL-запроса. Поэтому и стоит вопрос: "Можно ли как-то сократить запись из кода выше использую что-то типа scope только для коллекций?".

P.S.: как вы считаете, стоит ли использовать вариант со scope для запросов или мой вариант, изложенный выше?

Comments

[Arman]

$post->comments()->active()->get();
$post->comments()->incative()->get();

не нра?

[stdio96]

Arman, выполняется два запроса на странице, а при использовании первого варианта - один

Answer 1

[Александр Аксентьев]

class Post {
    public function scopeCommentsWithStatus($query, $status)
    {
        return $this->comments->where('status', $status);
    }

    public function getActiveCommentsAttribute()
    {
        return $this->comments->where('status', 1);
    }
}

$post = Post::with(['comments'])->first();

$post->commentsWithStatus(1);
$post->commentsWithStatus(0);

$post->active_comments;

извращайтесь как угодно, главное что следить за загрузкой with() надо всегда, в противном случае можно нарваться на ленивую загрузку которая сгенерирует по запросу на каждую модель.

Comments

[stdio96]

Спасибо. Протестирую. А как, на Ваш взгляд, стоит реализовать это: используя scope или таким вариантом? По оптимизации и тп. Насколько это целесообразно и оптимально-ли?

Могу-ли я сделать так:

class Post {
    // comments() relation goes here

    public function getActiveCommentsAttribute()
    {
        return $this->comments->where('status', 1);
    }

    public function getInactiveCommentsAttribute()
    {
        return $this->comments->where('status', 0);
    }
}

$post = Post::with(['comments'])->first();

$post->active_comments;
$post->inactive_comments;

Не совсем понимаю использование scope в Вашем примере.

[Александр Аксентьев]

stdio96,

Могу-ли я сделать так:

да это второй вариант в ответе и есть.
Подводный камень в том что если забудете сделать with() случайно, то получится стопицот запросов вместо одного.

Не совсем понимаю использование scope в Вашем примере.

Обычное использование динамического скопа, только через жопу и фактически никакой запрос не генерирует.
Не уверен что сработает, но скорее всего должно.

[stdio96]

Александр Аксентьев, ага, другой еще нюанс выплыл. Дело в том, что поля status в comments нету. Оно есть в таблице pivot (many to many). И отдельная модель не создана для этого отношения. Соответственно, когда я выполняю этот запрос, оно сообщает, что нету такого поля. Возможно я не совсем корректно объяснил. Возможно-ли тогда работать использовать атрибут? Если да, то в каком месте его прописывать?

И есть-ли вообще в этом смысл? Можно ведь просто 2 запросами обойтись. Интересует мнение Ваше.

Спасибо.

[Александр Аксентьев]

comments->where('pivot.status', 1)
или

comments->where(function ($comment) {
    return $comment->pivot->status === 1;
})

[stdio96]

Александр Аксентьев, спасибо. Все получилось бы еще раньше, если бы я в нужную модель написал атрибут -_-.

[NubasLol]

2 запроса к бд это ничто, такое оптимизировать своими костылями бред

[Александр Аксентьев]

stdio96, или про пивот хоть где-нибудь написали сразу

[Александр Аксентьев]

NubasLol, речь про коллекцию постов вроде как т.е. 2 запроса * количество постов на выходе будет.

[stdio96]

Александр Аксентьев, мне достаточно было прописать только 2 атрибута. Я не использовал scope.

Мой код:

public function getActiveCommentsAttribute()
    {
        return $this->comments->where('pivot.status', 1);
    }

    public function getInactiveCommentsAttribute()
    {
        return $this->comments->where('pivot.status', 0);
    }

@forelse ($post->inactive_comments as $comment)
...
@endfoelse

Дополнительный запрос не появился.

[NubasLol]

public function scopeCommentsWithStatus($query, $status)
    {
        return $this->comments->where('status', $status);
    }

Скоп который возвращает коллекцию????

[NubasLol]

Александр Аксентьев, 2 запроса вида

$post->comments()->where('status', 1)->get();
 $post->comments()->where('status', 0)->get();

Дополнительных запросов не вылезет

[stdio96]

NubasLol, только что попробовал несколько вариантов и Ваш в том числе - дополнительные запросы есть.

Вариант без дополнительных запросов:

$post->comments->where('status', 1); // active
$post->comments->where('status', 0); // inactive

[NubasLol]

stdio96, 2 запроса будет. Это ничто. Экономия на спичках. И то не факт. Нужно тестить.

Возможно даже php медленне будет фильтровать коллекцию чем сделать 2 запроса к бд

[stdio96]

NubasLol, в этом тоже вопрос. Я изначально сомневался в целесообразности реализации этого метода.

Оба варианта (через атрибуты и через предложенный Вами scope) рабочие.

Answer 2

[NubasLol]

Если очень хочется костылять

Collection::macro('inactive', function () {
    return $this->where('status', 1);
});

Answer 3

[jazzus]

Бояться запросов не стоит бд для того и создана) Если данных много запрос будет быстрее чем фильтровать коллекцию (проверено). Вообще лучше думать так – нет скопов, значит их не должно быть. С различного рода новаторскими идеями без опыта можно потом нарваться на нехилый рефакторинг (проверено). Я бы сделал так

// В модели Post
public function activeComments()
{
  return $this->commnets()->ofActive();
}

public function inactiveComments()
{
  return $this->commnets()->ofInactive();
}

// в контроллере/классе
$relations = ['activeComments', 'inactiveComments'];

Post::with($relations)
    ->withCount($relations)
    ->get();

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