Laravel支付验证功能求助：更新状态并累加用户钱包余额

Hey there, let's break down what's going wrong with your payment verification logic and fix it up:

核心问题分析 #

• 逻辑顺序错误：Right now, as soon as you successfully save the updated wallet status, you redirect away from the controller. That means the code that updates the user's wallet balance never runs—it's stuck after the return statement! This is why your users aren't seeing their balances increase.
• 缺乏事务保障：如果钱包状态更新成功，但用户余额更新失败，你会得到不一致的数据（支付标记为已验证，但用户钱包没收到钱），这会给业务带来隐患。
• 错误处理不完整：当前代码只在钱包保存失败时提示错误，但完全没处理用户余额更新失败的情况。

修复后的控制器代码 #

下面是调整后的代码，附带注释说明逻辑：

use Illuminate\Support\Facades\DB; // 记得引入DB门面

public function verify_payment($user_id, $payment_data, $id)
{
    // 开启数据库事务，保证所有操作要么全部成功，要么全部回滚
    DB::beginTransaction();

    try {
        // 1. 更新支付状态为verified
        $wallet = Mpesa::findOrFail($id);
        $wallet->status = 'verified';
        $wallet->save();

        // 2. 计算并更新用户钱包余额
        $top_up = User::findOrFail($user_id);
        // 为了更可靠，建议直接从支付记录中获取金额，而非依赖外部传入的$payment_data
        $amountToAdd = $wallet->amount; 
        $top_up->balance += $amountToAdd;
        // 只有当你需要在User模型上存储本次充值金额时，才保留这行代码
        // $top_up->amount = $amountToAdd; 
        $top_up->save();

        // 所有操作成功，提交事务
        DB::commit();
        flash(__('Payment has been approved successfully'))->success();
        return redirect()->route('all.payments');
    } catch (\Exception $e) {
        // 任何一步失败，回滚所有操作
        DB::rollBack();
        // 可选：添加日志记录方便调试 Log::error('Payment verification failed: ' . $e->getMessage());
        flash(__('Something went wrong: ') . $e->getMessage())->error();
        return back();
    }
}

额外优化建议 #

• 利用模型关联：如果User和Mpesa模型有关联关系（比如一个用户对应多条支付记录），可以用$wallet->user代替User::findOrFail($user_id)，代码更简洁。
• 校验输入数据：在更新余额前，确保待添加的金额是有效数字，避免意外错误。
• 记录错误日志：在catch块中添加错误日志，方便后续快速排查问题。

内容的提问来源于stack exchange，提问作者elly