24

我使用 laravel5.5 的数据库事务进行在线支付应用。我有一个 company_account 表来记录每笔付款（type、、、、amount）。创建新记录时，我需要访问最后一条记录。所以我需要在事务的时候用读写表锁来锁表，以避免同时进行多次支付。create_atgross_incomegross_income

我参考了 laravel 的文档，但我不确定事务是否会锁定表。如果事务将锁定表，锁定类型是什么（读锁、写锁或两者兼有）？

DB::transaction(function () {
    // create company_account record

    // create use_account record
}, 5);

代码：

DB::transaction(function ($model) use($model) {
    $model = Model::find($order->product_id);
    $user = $model->user;

    // **update** use_account record
    try {
        $user_account = User_account::find($user->id);
    } catch (Exception $e){
        $user_account = new User_account;
        $user_account->user_id  = $user->id;
        $user_account->earnings = 0;
        $user_account->balance  = 0;
    }
    $user_account->earnings += $order->fee * self::USER_COMMISION_RATIO;
    $user_account->balance += $order->fee * self::USER_COMMISION_RATIO;
    $user_account->save();

    // **create** company_account record
    $old_tiger_account = Tiger_account::latest('id')->first();

    $tiger_account = new Tiger_account;
    $tiger_account->type = 'model';
    $tiger_account->order_id = $order->id;
    $tiger_account->user_id = $user->id;
    $tiger_account->profit = $order->fee;
    $tiger_account->payment = 0;
    $tiger_account->gross_income = $old_tiger_account-> gross_income + $order->fee;
    $tiger_account->save();
}, 5);

参考：
 如何将参数传递给 Laravel DB::transaction()

4

3 回答 3

13

由于您要更新 2 个表，因此您仍然需要使用事务来保持更改同步。考虑以下代码：

DB::transaction(function () {
    $model = Model::find($order->product_id);
    $user = $model->user();

    DB::insert("
        insert into user_account (user_id, earnings, balance) values (?, ?, ?)
        on duplicate key update
        earnings = earnings + values(earnings),
        balance = balance + values(balance)
    ", [$user->id, $order->fee * self::USER_COMMISION_RATIO, $order->fee * self::USER_COMMISION_RATIO]);

    DB::insert(sprintf("
        insert into tiger_account (`type`, order_id, user_id, profit, payment, gross_income)
            select '%s' as `type`, %d as order_id, %d as user_id, %d as profit, %d as payment, gross_income + %d as gross_income
            from tiger_account
            order by id desc
            limit 1
    ", "model", $order->id, $user->id, $order->fee, 0, $order->fee));

}, 5);

有 2 个原子查询。第一个将记录插入到user_account表中，另一个将记录插入到tiger_account.

如果这两个查询之间发生了可怕的事情，您需要事务来保证不会应用任何更改。可怕的不是并发请求，而是 php 应用程序、网络分区或其他任何阻止执行第二个查询的突然死亡。在这种情况下，更改从第一个查询回滚，因此数据库保持一致状态。

这两个查询都是原子的，这保证了每个查询中的数学运算都是独立完成的，此时没有其他查询会更改表。说有可能 2 个并发请求同时为同一用户处理 2 笔付款。第一个将在user_account表中插入或更新记录，第二个查询将更新记录，两者都将向 中添加一条记录tiger_account，并且在提交每个事务时，所有更改都将永久设置在数据库中。

我做了几个假设：

  • user_id是user_account表中的主键。
  • 中至少有 1 条记录tiger_account。在 OP 代码中调用$old_tiger_account的那个，因为当数据库中没有任何内容时，不清楚预期的行为是什么。
  • 所有货币字段都是整数，而不是浮点数。
  • 它是 MySQL 数据库。我使用 MySQL 语法来说明该方法。其他 SQL 风格的语法可能略有不同。
  • 原始查询中的所有表名和列名。不要记得阐明命名约定。

一句警告。这些是原始查询。随着一些应用程序逻辑从命令式 PHP 转移到声明式 SQL，您将来应该特别注意重构模型，并编写更多的集成测试。我相信保证没有比赛条件是一个公平的价格，但我想明确表示它不是免费的。

于 2017-11-01T12:12:02.133 回答
8

我遇到了 MySQL: Transactions vs Locking Tables 这个问题的答案，它解释了事务和锁定表。它显示了这里应该使用的事务和锁定。

我参考Laravel lockforupdate (悲观锁定)和How to pass parameter to Laravel DB::transaction()，然后得到下面的代码。

我不知道这是否是一个很好的实现，至少它现在有效。

DB::transaction(function ($order) use($order) {
    if($order->product_name == 'model')
    {
        $model = Model::find($order->product_id);
        $user = $model->user;

        $user_account = User_account::where('user_id', $user->id)->lockForUpdate()->first();

        if(!$user_account)
        {
            $user_account = new User_account;
            $user_account->user_id  = $user->id;
            $user_account->earnings = 0;
            $user_account->balance  = 0;
        }

        $user_account->earnings += $order->fee * self::USER_COMMISION_RATIO;
        $user_account->balance += $order->fee * self::USER_COMMISION_RATIO;
        $user_account->save();

        $old_tiger_account = Tiger_account::latest('id')->lockForUpdate()->first();
        $tiger_account = new Tiger_account;
        $tiger_account->type = 'model';
        $tiger_account->order_id = $order->id;
        $tiger_account->user_id = $user->id;
        $tiger_account->profit = $order->fee;              
        $tiger_account->payment = 0;

        if($old_tiger_account)
        {
            $tiger_account->gross_income = $old_tiger_account->gross_income + $order->fee;
        } else{
            $tiger_account->gross_income = $order->fee;
        }

        $tiger_account->save();
    }
}, 3);
于 2017-11-06T06:17:35.553 回答
3

在我看来，如果你单独计算每条记录的总收入，你甚至不需要锁表，你知道锁表会直接降低你的网站速度。

DB::transaction(function () use($order) {
    $model = Model::find($order->product_id);
    $user = $model->user;

    // **update** use_account record
    try {
        $user_account = User_account::find($user->id);
    } catch (Exception $e){
        $user_account = new User_account;
        $user_account->user_id  = $user->id;
        $user_account->earnings = 0;
        $user_account->balance  = 0;
    }
    $user_account->earnings += $order->fee * self::USER_COMMISION_RATIO;
    $user_account->balance += $order->fee * self::USER_COMMISION_RATIO;
    $user_account->save();

    // **create** company_account record
    $tiger_account = Tiger_account::create([
        'type' => 'model',
        'order_id' => $order->id,
        'user_id' => $user->id,
        'profit' => $order->fee,
        'payment' => 0,
    ]);

    $tiger_account->update([
        'gross_income' => Tiger_account::where('id', '<=', $tiger_account->id)->sum('fee'),
    ]);
});
于 2017-11-04T14:21:48.143 回答