Как получить порядковый номер объекта из коллекции?

Question

[jazzus]

Нужно получить
Объекты, которые принадлежат юзеру
С порядковыми номерами каждого объекта из всей коллекции объектов в заданной сортировке.
Цель: показать юзеру позиции его объектов в общем каталоге

Код

// Запрос все коллекции
           Project::with(..)
                  ->withCount(..)
                  ->orderBy(..)
                  ->get()
                  // добавляем номер к элементам
                  ->map(function ($item, $index) {
                       $item['number'] = $index + 1;
                       return $item;
                   })
                  // фильтруем по id авторизованного юзера
                  ->where('user_id', $user->id);

Это правильный способ или есть другой? Как в эту систему встроить пагинацию?

Answer 1

[vism]

Лень писать код.
Суть.
Вам надо сделать выборку с порядковым номером заданным в базе данных динамически. Что-то типо num_row, не помню.
Эту выборку использовать в качестве temporary-таблицы и по этой temporary-таблице делать выборку по пользователю.

Comments

[jazzus]

я правильно загуглил? Пока не знаю с какой стороны подойти. Думал гораздо проще делается

[jazzus]

т.е. нужно создать временную таблицу, отсортировать, добавить поле номера, забрать объекты юзера и таблицу удалить?

[JhaoDa]

jazzus, или использовать оконные функции вместо временных таблиц.

[jazzus]

JhaoDa,

$select = \DB::raw("@i := coalesce(@i + 1, 1) ranking, projects.*");
$projects = Project::select($select)
->orderByDesc('lift_at')
->where('user_id',$user->id)
->paginate(5);

Оно?

[JhaoDa]

jazzus, нет, но если такой вариант устраивает, то почему бы и да.

[jazzus]

JhaoDa, как 5.0 на 5 заменить? Или доки читать?

[jazzus]

В доках пишут про sql инъекции и "крайне осторожно". Также загугленный запрос считает до сортировки, а значит считает не правильно.

[vism]

@jazzus

что-то типо такого

->leftJoin('
Select
                p1.product_ident_number
                FROM products p1
                LEFT JOIN warehouse_products wp ON wp.`product_id`=p1.id
                LEFT JOIN orders o ON wp.`order_id`=o.id
                WHERE wp.deleted_at IS NULL AND wp.stock_out_dt IS NULL AND wp.id IS NOT NULL
                GROUP BY p1.product_ident_number)
                AS temp_warehouse_amount'),
                //AND (o.status != "' . Order::STATUS_SENDED . '" OR o.status IS NULL)
                function ($query) {
                    /** @var JoinClause $query */
                    $query->on('products.product_ident_number', '=', 'temp_warehouse_amount.product_ident_number');
                }
            )

ну тут немного raw sql и left join

гуглите 2 вещи.
1. mysql select row numeration
2. laravel select from sub query table

Мне очень лень, простите. надеюсь вы поймете путь в котором двигаться)

[vism]

тоесть вы делате выборку и на основе этой выборки делаете следующую.
Если я правильно понял, что вам нужно нумерация по всем, а потом выборку по пользователю.

[jazzus]

vism, мне советуют через запись в бд делать. Доводы железные – зачем часто считать то, что можно считать редко. Думаю сделать rank в projects и при смене даты (на которую ориентируется rank) – делать перезапись ранка у всех проектов в базе. Это будут редкие обновления. Хотя с другой стороны там придется всю таблицу форычить.. Плюс все равно запрос делать, чтобы проекты у юзера получить. Я вчера пробовал делать temporary таблицу

DB::statement('CREATE TEMPORARY TABLE projects_temp LIKE projects');
DB::statement('INSERT projects_temp SELECT * FROM projects');

Не получилось projects отфильтровать/сортировать по нужным параметрам. В sql запросах вообще не разбираюсь и боюсь sql инъекций)) Все решал через билдер Ларавел всегда. В общем буду думать-гуглить спасибо.

[jazzus]

vism, пробую вариант записи в бд

$projects=Project::orderBy('lift_at', 'desc')
                     ->get()
                    ->map(function ($item, $index) {
                         $item['rank'] = $index + 1;
                          return $item;
                     });

foreach ($projects as $project)
{
     // dd($project);
    $project->update(['rank']);
}

Тормозов не вижу (250 записей), но думаю заменить foreach пока не знаю чем.

[vism]

jazzus,

Select
                p1.product_ident_number
                FROM products p1
                LEFT JOIN warehouse_products wp ON wp.`product_id`=p1.id
                LEFT JOIN orders o ON wp.`order_id`=o.id
                WHERE wp.deleted_at IS NULL AND wp.stock_out_dt IS NULL AND wp.id IS NOT NULL
                GROUP BY p1.product_ident_number)
                AS temp_warehouse_amount

AS temp_warehouse_amount

[vism]

в 1 запросе все делать можно.

[jazzus]

vism, спасибо за пример! Я внедрил уже бд, но попробую запросом на днях для эксперимента. Т.е. вы считаете, что лучше такое считать через запрос не бд?

[vism]

jazzus, запрос к бд выполняется в бд.

[jazzus]

vism, получать расчетом ранг или сохранять в бд

[jazzus]

vism, как переделать ваш пример не понял. Т.к. вообще не знаю sql и учился на Laravel моделях. А вам код лень писать. Выходит я не умею, вы не хотите. Но это и правильно. Нефиг требовать решений, которые все равно не понимаешь. От рангов в бд отказался по понятным причинам. Не тормозит только если ранг не менялся. А если менялся то еще как тормозит.

[jazzus]

сделал пока так

$projects=Project::with()
                     ->withCount()
                     ->orderBy('lift_at', 'desc')
                     ->get()
                     ->map(function ($item, $index) {
                          $item['ranking'] = $index + 1;
                          return $item;
                     })->where('user_id',Auth::id());
                     
    $projects=$this->paginate($projects, 10);

    public function paginate($items, $perPage = 15, $page = null, $options = [])
    {
    $page = $page ?: (Paginator::resolveCurrentPage() ?: 1);
    $items = $items instanceof Collection ? $items : Collection::make($items);
    return new LengthAwarePaginator($items->forPage($page, $perPage), $items->count(), $perPage, $page, $options);
    }

[vism]

jazzus, Согласен. Еслиб это мне было на 5 минут работы, я бы написал, но тут я бы сам минут 30 провозился с запросом, не помню достаточно хорошо, как еще счетчик сделать.
Вобще без sql в больших проектах никуда, статистика часто нужна различная, и если на пхп обсчитывать нагрузки будут пипец какие

[jazzus]

vism, я бы sql давно выучил, были моменты, которые обходил с помощью билдера, отношений и т.д., но везде пишут про sql инъекции и что использовать нужно осторожно. Об sql инъекциях узнал раньше, чем о существовании переменных) Поэтому фреймворк стал изучать. Так в целом понятно, что происходит в рав запросах. Синтаксис непонятен, последовательность и нужно знать тонкости безопасности. Что касается коллекции - каких-то отрицательных моментов не заметил. Код не особо красивый с этим user_id.. А так вообще нормально делать? Считать в коллекции я имею ввиду? По-идее там в расчете рейтинга нет запросов к бд, значит быстрее должен работать. Или наоборот нагрузка больше?