Transactions Yii2中带有回调的Db事务

Transactions Yii2中带有回调的Db事务,transactions,yii2,yii2-model,Transactions,Yii2,Yii2 Model,我有一个简单的代码,它在Yii2中使用DB事务,这会更新用户余额,并向用户余额历史记录中添加一条新记录 //User model public function changeBalance(UserBalanceHistory $balance) { $balance->balance = $this->balance; $balance->user_id = $this->id; $this->balance = $this->

我有一个简单的代码,它在Yii2中使用DB事务,这会更新用户余额,并向用户余额历史记录中添加一条新记录

//User model
public function changeBalance(UserBalanceHistory $balance)
{
    $balance->balance = $this->balance;
    $balance->user_id = $this->id;
    $this->balance    = $this->getBalance() + $balance->getDelta();

    $transaction = Yii::$app->db->beginTransaction();
    try {
        if ($balance->save() && $this->save()) {
            $transaction->commit();
            return true;
        }
    } catch (Exception $e) {
        Yii::error($e->getMessage());
    }

    $transaction->rollBack();
}
我应该经常使用DB事务来保存数据完整性。但处理上述DB事务需要大量代码行,因此我创建了以下函数来调用我的代码:

function dbTransaction(callable $callback)
{
    $transaction = Yii::$app->db->beginTransaction();

    try {
        //if callback returns true than commit transaction
        if (call_user_func($callback)) {
            $transaction->commit();
            Yii::trace('Transaction wrapper success');
        }
    } catch (\Exception $e) {
        $transaction->rollBack();
        throw $e;
    }
    $transaction->rollBack();
}
使用此功能,我可以处理如下事务:

//User model
public function changeBalance(UserBalanceHistory $balance)
{
    dbTransaction(
        function () use ($balance) {
            $balance->balance = $this->balance;
            $balance->user_id = $this->id;
            $this->balance    = $this->getBalance() + $balance->getDelta();

            return $balance->save() && $this->save();
        }
    );
}

正如您所看到的,第二种方法非常适合使用事务。但在这一点上,我不确定
dbTransaction
函数是否正常工作?代码审查并注意哪些点是潜在问题。谢谢

删除您的代码
$transaction->rollBack()在最后一行之前,因为此代码总是回滚并取消事务

function dbTransaction(callable $callback)
{
    $transaction = Yii::$app->db->beginTransaction();

    try {
        //if callback returns true than commit transaction
        if (call_user_func($callback)) {
            $transaction->commit();
            Yii::trace('Transaction wrapper success');
        }
    } catch (\Exception $e) {
        $transaction->rollBack();
        throw $e;
    }
    //$transaction->rollBack();
 }
应该移到