当前位置：文江博客话题详情

thinkphp 新增为啥不用判断是否成功

发布于 2022-09-12 23:15:52 字数 209 浏览 20 评论 0

我看到挺多代码都是
$order = new Order();
$res = $order->saveAll($lst);

获取
Order::create($lst)

并没有有类似这样的
$res=Order::create($lst)
if(!$res->id) 提示错误

有没必要增加这个判断呢还是tp他会默认做了呢

分享到QQ

分享到微博

如果你对这篇内容有疑问，欢迎到本站社区发帖提问参与讨论，获取更多帮助，或者扫码二维码加入 Web 技术交流群。

发布评论

需要登录才能够评论，你可以免费注册一个本站的账号。

我只土不豪 2022-09-19 23:15:52

错误还是要进行处理的,只是你们开发比较乐观,觉得不会出错,并没有捕获错误

回复收藏 0

我最亲爱的 2022-09-19 23:15:52

先回答你第一个问题：有没必要增加这个判断呢？

这个问题，其实看一下源码就知道（TP版本5.0.22）

就你举例的这个create方法而言，主要代码是在

/thinkphp/library/think/Model.php
/thinkphp/library/think/db/Query.php
/thinkphp/library/think/db/Connection.php
三个文件中

最终的执行方法是Connection.php文件里的excute方法，代码如下：

    /**
     * 执行语句
     * @access public
     * @param  string        $sql sql指令
     * @param  array         $bind 参数绑定
     * @param  Query         $query 查询对象
     * @return int
     * @throws PDOException
     * @throws \Exception
     */
    public function execute($sql, $bind = [], Query $query = null)
    {
        $this->initConnect(true);
        if (!$this->linkID) {
            return false;
        }

        // 记录SQL语句
        $this->queryStr = $sql;
        if ($bind) {
            $this->bind = $bind;
        }

        Db::$executeTimes++;
        try {
            // 调试开始
            $this->debug(true);

            // 预处理
            $this->PDOStatement = $this->linkID->prepare($sql);

            // 是否为存储过程调用
            $procedure = in_array(strtolower(substr(trim($sql), 0, 4)), ['call', 'exec']);
            // 参数绑定
            if ($procedure) {
                $this->bindParam($bind);
            } else {
                $this->bindValue($bind);
            }
            // 执行语句
            $this->PDOStatement->execute();
            // 调试结束
            $this->debug(false, '', true);

            if ($query && !empty($this->config['deploy']) && !empty($this->config['read_master'])) {
                $query->readMaster();
            }

            $this->numRows = $this->PDOStatement->rowCount();
            return $this->numRows;
        } catch (\PDOException $e) {
            if ($this->isBreak($e)) {
                return $this->close()->execute($sql, $bind, $query);
            }
            throw new PDOException($e, $this->config, $this->getLastsql());
        } catch (\Throwable $e) {
            if ($this->isBreak($e)) {
                return $this->close()->execute($sql, $bind, $query);
            }
            throw $e;
        } catch (\Exception $e) {
            if ($this->isBreak($e)) {
                return $this->close()->execute($sql, $bind, $query);
            }
            throw $e;
        }
    }

可以看到在插入操作数据库代码的外面加了一层try catch的异常捕获，并且抛出了异常

由此，可以得到，你上面说的这种处理

$res=Order::create($lst)
if(!$res->id) 提示错误

其实也可以，但是非必要。加了是更严谨

一旦数据库操作报错，TP5源码中会直接抛出异常出来，你可以在模型操作外面加一层try catch捕获异常即可：

try {
    Order::create($lst);
} catch (\PDOException $e) {
    // TODO 这里处理执行失败之后的操作
} catch (\Exception $e) {
    // TODO 这里处理执行失败之后的操作
}

第二个问题：tp有没有做这种判断呢？

答案是有，可以看/thinkphp/library/think/db/Query.php文件的insert方法，代码片段：

// 执行操作 (这里的execute的最终调用方法就是Connection.php的execute方法)
$result = 0 === $sql ? 0 : $this->execute($sql, $bind, $this);
if ($result) {
    $sequence  = $sequence ?: (isset($options['sequence']) ? $options['sequence'] : null);
    $lastInsId = $this->getLastInsID($sequence);
    if ($lastInsId) {
        $pk = $this->getPk($options);
        if (is_string($pk)) {
            $data[$pk] = $lastInsId;
        }
    }
    $options['data'] = $data;
    $this->trigger('after_insert', $options);

    if ($getLastInsID) {
        return $lastInsId;
    }
}

可以看到它也有处理，通过getLastInsID()方法获取最后一次的插入id，会把id放在你上面$res = Order::create($lst)中的$res变量中，所以你在后面加一步!$res->id的判断也是可以的

你应该有答案了吧~

回复收藏 0