Правильное массовое обновление MySQL в Laravel?

Question

[FANTASANTA]

Есть около 10.000 товаров, нужно по категориям обновить цены, для каждой группы товаров по категориям определенные ставки повышения цен. Но правильно ли делать обновление внутри цикла?

Сейчас в голову лезет только одно решение:

// И так более 10 тысяч запросов за раз
foreach($products as $p) {
Products::where('id', $p->id)->update(['price' => $newPrice]);
}

Правильно ли таким образом производить массовое обновление?

Comments

[Akina]

Десять тыщ товаров по одному обновлять? да ты состаришься и умрёшь за монитором...

Конечно, неправильно. Надо как минимум обновлять всю группу одним запросом. А если есть таблица, которая содержит новые цены (или, скажем, коэффициенты изменения погруппно), то и сразу всю таблицу.

[FANTASANTA]

Akina, нет, я же показываю в цикле. Я 10000 собираюсь обновлять сразу в одном цикле, там цены не готовые, я сначала вытаскиваю товар с нынешней ценой, и через формулу создаю новую цену ($newPrice) потом внутри цикла меняю цену. Но правильно ли это?
Сейчас я так не делаю, но идея в голове только такая, поэтому спрашиваю здесь.

[Дмитрий]

FANTASANTA, ну вытащить все цены, посчитать, и на основе новых данных обновить

[FANTASANTA]

Дмитрий, 10-20 тысяч запросов это нормально?) вопрос в этом и заключается, в дальнейшем товаров будут более 100 тысяч

[Akina]

там цены не готовые, я сначала вытаскиваю товар с нынешней ценой, и через формулу создаю новую цену ($newPrice) потом внутри цикла меняю цену.

Формулу покажите. Ибо в 99% случаев это какая-то тривиальная арифметика, и в этом случае тащить цену на клиента, чтобы пересчитать её и потом отправить обратно - очевидный идиотизм.

[FANTASANTA]

Akina,

$percentage = 2;
$price = 1000;

$newPrice = ($percentage / 100) * $price;

[N]

FANTASANTA, Читай ответ ниже...там сразу в UPDATE это можно делать...
Но тут могут быть тоже нюансы...
Я бы обновлял "пачками" с "лимитом"...фоново...

[Akina]

FANTASANTA, что-то бредовое... текущая цена 1000, а показанная формула даст новую цену какую? может, $newPrice = (1 + $percentage / 100) * $price;
Может, не маяться фигнёй, а просто выполнить RAW-запрос типа
UPDATE products SET price = price * 1.02

[N]

Akina, FANTASANTA,
Акина дело говорит...
Только я бы ещё хранил "базовую" цену в отдельном поле...чтобы всегда можно было вернуться или пересчитать от "исходной" цены...

[FANTASANTA]

Akina,

UPDATE products SET price = price * 1.02

я так и сделал в итоге, спасибо за подсказку.
Вышло таким вот образом:

Products::whereIn('category' , [1,2,3,4])->update([
                    'data->price->retail' =>  DB::raw('data->"$.price.retail" '.$type.' (data->"$.price.retail" / 100) * '.$percentage)
                ]);

$type - тут я указываю + или - для повышения или понижения цены соответственно.

Answer 1

[Антон Антон]

Если нет хуков на события, связанных с этими полями, то лучше сделать все через массовые операции (и да, там тоже можно использовать произвольные выражения)