Как посчитать среднее значение в базе данных?

Question

[maksutka777]

Привет! У меня есть таблица books (список книг), для каждой книги есть отзывы с оценкой - они хранятся в таблице reviews. Мне нужно для каждой книги вывести среднюю оценку, но считать надо только те оценки, у которых active = true.

Вот этот код, выводит среднюю оценку без учета active, но как исправить так, чтобы учитывалось true? Уже полдня в поисках решения...

public function index() {
        $books = Book::query()->
        where('active', true)->
	withAvg('reviews', 'grade')->
        get();
        return view('books.index', compact('books'));
    }

В шаблоне:
{{ $book->reviews_avg_grade }}

Comments

[Дмитрий]

А поле какого типа у вас? Которое true?

[maksutka777]

Дмитрий, boolean

[Дмитрий]

maksutka777, а бд какая? Это вы где тип его посмотрели? В миграции?

[Дмитрий]

Ну посмотрите какой запрос улетает в бд.

[maksutka777]

Дмитрий, Дмитрий,

я думаю я по-дурацки объяснил что хочу сделать:

public function index() {
        $books = Book::query()->
        where('active', true)-> /* этот пункт работает, он делает выборку для книг с action = true, тут все ок */
  withAvg('reviews', 'grade')-> /*  а вот это считает среднюю оценку из таблицы review колонки grade. Но тут нет ограничения на то, чтобы считались не все оценки, а только те, у которых в этой же таблице (reviews) столбец active = true.  Вопрос в том, что добавить в эту строку, чтобы не все оценки в расчет брались */ 
        get();
        return view('books.index', compact('books'));
    }

[Дмитрий]

maksutka777, я думаю стоит сделать перед запросом DB::enableQueryLog() и dd(DB::getQueryLog()); после. И взять за правило если запрос что то делает не то что вы хотите, смотреть что вообще за запрос выполняется.

[Дмитрий]

maksutka777, попробуйте вместо withavg использовать просто avg

[maksutka777]

Дмитрий, а как laravel узнает что надо брать из reviews только active=true?
Эта строка where('active', true) же только для таблицы books.

[maksutka777]

вот, например, такая строка считает количество оценок с теми же ограничениями. А можно ли ее как то применить к вычислению среднего...

withCount(['reviews' => function ($query) {
            $query->where('active', true);
        }])->

[Дмитрий]

maksutka777, а таблицы как то связанны?
Наверное нужны будут джоины

[maksutka777]

Дмитрий, да, конечно, в моделях. К тому же общая оценка без ограничения для каждой книги высчитывается как надо

[Дмитрий]

maksutka777, то есть у вас есть метод в модели оценок для подсчёта среднего? Что то вы не так сделали

[maksutka777]

Дмитрий, еще раз напишу: представленный код работает как надо, вот код на всякий случай:

public function index() {
        $books = Book::query()->
        where('active', true)->
  withAvg('reviews', 'grade')->
        get();
        return view('books.index', compact('books'));
    }

У него нет ошибок, свою функцию выполняет на 100%. Он выводит в шаблон список книг, и напротив каждой книги пишет ее среднюю оценку. Мне нужно этот код усовершенствовать, сделать так, чтобы он не все оценки подряд брал относящиеся к книге, а только те оценки, которые в столбце active имеют true =) Сейчас же этот код считает и true и false.

[Дмитрий]

maksutka777, судя по коду, он должен работать именно как вы описали, активные книги, и их рейтинг. Как вариант дальше в коллекции убрать лишние, либо разобраться до конца, где что не так

[Алексей Уколов]

Дмитрий, у оценок, похоже, тоже есть активность, не только у книг.

[Дмитрий]

Алексей Уколов, только не понятно зачем оценкам активность)

Answer 1

[nokimaro]

$books = Book::query()
            ->where('active', true)
            ->withAvg(
                  ['reviews' => fn ($query) => $query->where('active', true)],
                  'grade'
            )
            ->get();