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

Question

[Владислав Нагорный]

Есть функция A и в ней есть пару запросов в базу и работа с счетом юзера. Но когда много одновременных запросов скрипт работает как хочет, т.е скрипт не успевает обрабатывать счет (списание денег) юзера ... Как можно исправить ситуацию?

\DB::beginTransaction();
if($user->money < 100) return false;
try {
$user->money = $user->money - 100;
$user->save();
\DB::commit();
} catch (\Exception $e) {
\DB::rollback();
dd($e);
}
Что то не так работает

Answer 1

[Alex Wells]

DB::transaction(function() use ($user) {
    $user->money -= 100;
    $user->save();
});

В чем проблема то?

PS: фасад не забудьте

Answer 2

[Oleg]

В зависимости от логики.

1. Транзакции
2. Снимать и проверять одновременно:
Update table set money=money-10 where user=123 and money>=10
И проверка кол-ва строк измененных

Answer 3

[ThunderCat]

для начала поправить код,
$use->money = $user->money - 100;
пишите в блокноте? Любая нормальная IDE подсвечивает такой косяк.

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

Comments

[Владислав Нагорный]

Код нормальный, просто когда здесь для примера писал, ошибся ...

[Владислав Нагорный]

И есть ссылки на примеры, где почитать, посмотреть?

Answer 4

[Вячеслав Плиско]

User::where('money',  '>', 100)
    ->update('money', \DB::raw('money - ?', 100));

Плюс плюс посмотрите в сторону локов https://laravel.com/docs/5.3/queries#pessimistic-l...

Comments

[Владислав Нагорный]

Пока сделал так, вроде работает, не знаю правильно ли ...

return DB::transaction(function () use ($id) {
$user = User::where('id', $id)->lockForUpdate()->first();
///код код код
});