Как упростить запрос laravel?

Question

[Nubbin]

$items = DataRelationship::where('link_id', $link->id)->get()->filter(function($user) {
                return (isset($user->user));
            })->sortByDesc(function($item) {
                return $item->club->where('status', 1)->count();
            })->take(10)

Этот запрос показывает данные и сортирует по большим количествам.
В базе 75 000 данных и это занимает чуть ли не 5 минут. Как можно упросить этот вариант чтобы она быстрее подгружало их

Comments

[Vanya Huk]

для начала почитать логи и узнать что грузить сервак, php или mysql

[Nubbin]

Очевидно же запрос.
Почему "DataRelationship::where('link_id', $link->id)->get()" - этот запрос из базы и собирает все
"->filter(function($user) {
return (isset($user->user));
})->sortByDesc(function($item) {
return $item->club->where('status', 1)->count();
})->take(10)" - это настройка в коллекции. (Я хочу чтобы запросом было но незнаю как сделать)

[Дмитрий Евграфович]

1. Использовать wherehas для user и не получать лишнее.
2. У вас нагрузка создаётся в sortbydesc, можно снизить ее, загрузив данные о count отдельным запросом, вставив айдишники нужных записей через wherein, например. Получить 10 записей из базы куда быстрее, чем 75000 запросов сделать и потом сортировать их методами php.

Как вариант, можно написать запрос через club, но не имея компа под рукой и кода я вам его не накидаю на бумажке. Суть в том, чтобы сортировку делать методом db::raw(), а пользователей фильтровать методом wherrhas на стадии формирования запроса, а не после получения всех данных.

Answer 1

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

Потому что надо делать одним запросом нормально.

А у вас выбирается вообще всё что только возможно ->get()
А потом начинаются фильтры и еще подзапросы как минимум по одному на каждую строку которая нашлась в базе изначально по одному. Т.е. 75 000 запросов на выходе, а может даже в 2 раза больше с учетом первого фильтра.

Читайте доку по with(), orderBy() и наверно leftJoin() и делайте один нормальный запрос.

Comments

[Nubbin]

Пожалуйста, помогите да это сделать.

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

Nubbin, get уберите свой первый, и всё что после него переписывайте с помощью query builderа, чтобы это бы запрос в базу, а не вытаскивание из базы с последующей фильтрацией.

https://laravel.com/docs/5.6/queries

[Nubbin]

$items = DB::table('data_relationship')
            ->leftJoin('club','data_relationship.id_user', 'club.id_user')
            ->where('club.status', '=', 1);

По моему не ошибся, теперь помогите пожалуйста как отсортировать их как я делал ее в первом примере.

Answer 2

[Koteezy]

DataRelationship::select(['id', 'name', etc...])->whereLinkId($link->id)->whereHas('user')
            ->addSelect(
                DB::raw( '(select count(id) from clubs where data_relationships.id = clubs.data_relationship_id) as clubs_count' )
            )->orderBy('clubs_count', 'desc')->get();

Как-то так. В DB::raw поменяйте на свой sql, я не знаю какие у вас таблицы, и ключи.

Comments

[Nubbin]

Ваш запрос, тоже долго грузит данные почему та.

[Koteezy]

Nubbin, Ну так в конце get, а не take(10)

[Nubbin]

я в конце добавил ->paginate(10)

[Nubbin]

извините что вас мучаю, помогите пожалуйста.

[Koteezy]

Nubbin, хм, а сколько записей?

[Nubbin]

сейчас удалил половину где та в районе "5473"

[Koteezy]

Nubbin, Маловато как-то для слишком долгих запросов, даже не знаю в чем может быть проблема

[Nubbin]

$items = DataRelationship::select(['user_id'])->where('link_id', $link->id)->whereHas('user')
        ->addSelect(
            DB::raw( '(select count(id) from club where data_relationships.user_id = club.id_user and tasks.status = 1) as count_app' )
        )->orderBy('count_app', 'desc')->paginate(10);

[Koteezy]

Nubbin, можно попробовать:

->withCount('clubs')->orderBy('clubs_count', 'desc')->orderBy('clubs_count', 'desc')

[Nubbin]

Откуда взять "->orderBy('clubs_count', 'desc')-"

[Nubbin]

Попробовал не помогает, все равно долго обрабатывается

[Koteezy]

Nubbin, От withCount

[Nubbin]

Koteezy, тоже самое не помогает.

[Koteezy]

Nubbin, тут я бессилен

[Nubbin]

Koteezy, В phpmyadmin по limit 50 выдал за 19 секунд.

Answer 3

[Yan-s]

Напишите сперва чистый запрос SQL и выполните его напрямую, а не из PHP. Вот когда он у вас будет и будет выполнятся за устраивающее вас время и вернет именно то, что вам надо, тогда подумаем как его в ORM засунуть.

А так.. пустое занятие в вашем случае.