关于数据库事务提交位置的疑惑

事物提交了以后还可以回滚吗？

赞成你同事的写法

if ($book) {} 这里是判断数据是否有问题

如果不判断数据是否存在问题，事务的作用会打折的。

你们的争论焦点应该是这个，如果数据存在问题， 还需要写入数据库吗？

可以试试事务的闭包写法，全局捕捉异常。

DB::transaction(function () {
    $user = UserInfo::query()->create([
        "name" => "foo",
        "age" => 18
    ]);

    $user->books()->create([
        "name" => "bar"
    ]);
});

只有我看不明白为什么要加判断吗？
还有这个异常什么情况才会抛出呢？
如果 $book 保存成功会走到 return，如果不成功框架应该会抛出异常吧

throw new Exception("提交失败");

所以直接这样是不是就行了。

public function test() {
    DB::beginTransaction();
    try {
        $user = UserInfo::query()->create([
            "name" => "李大",
            "age" => 18
        ]);

        $book = Book::query()->create([
            "name" => "测试书籍1",
            "user_id" => $user->id
        ]);
        DB::commit();
        // 这个 return 写法我也没见过
        return("提交成功");
    } catch (Exception $exception) {
        DB::rollback();
        throw new Exception($exception->getMessage());
    }
}

首先你这个业务创建为什么会失败？如果失败了是属于什么异常？外层的异常捕获是捕获你运行发生的异常但是你主动抛出的这个异常明显就很奇怪无论它有没有创建成功你都不应该抛出异常而是应该提示，再一个你这个关联是必须有的吗？第一个创建失败那么第二个其实成不成功都无所谓这种怎么能成为一个事务，如果第一个用户创建失败那么这本是肯定创建失败所有看你业务怎么需要怎么实现。。

我会这样写：

    public function test()
    {
        try {
            DB::beginTransaction();
            $user = UserInfo::query()->create([
                "name" => "李大",
                "age" => 18
            ]);
            Book::query()->create([
                "name" => "测试书籍1",
                "user_id" => $user->id
            ]);
            DB::commit();
            return '提交成功';
        } catch (Exception $e) {
            DB::rollback();
            // log $e->getMessage()
            return '提交失败';
        }
    }

明确告诉你把，insert语句在没有产生异常的情况下，是一定有影响记录数返回的，你那个 if 就是多余的。然后说 update语句，大部分业务场景，也是不需要判断返回值的，只关注语句有没有产生异常就好了，少部分需要关注返回值确定有影响记录的行数，这时候才加 if 判断

不如先讨论一下什么情况下既创建失败，又不触发 Exception。。。

为啥你们都把 DB::beginTransaction(); 写在 try catch 之外呢 我都是写在里面的