检查每个db查询的失败是否是一个很好的php编码习惯？

Question

我只编写了4个月的.and代码，我从未见过大型PHP脚本，也不知道它们是什么样子的。我在想，“我真的需要对每一个查询都这样做吗？”

 $result= $mysqli->query("select * from clients;");
 if(!result){
    //error.  log it and echo a proper message
 }else{
    //show the clients
 }

我是否需要对我在最终的生产代码中所做的每一个查询都这样做呢？因为我的php脚本现在充满了if(!$result) if(!$result) if(!$result) if(!$result) if(!$result)。这不需要更多的时间来解析，这意味着更多的响应延迟。

注意:在继续之前，我在php脚本的最上面测试数据库连接。

Answer 1

是的，你应该检查一下错误。首先，它有助于调试，而且由于您应该尽可能少地运行每个请求的查询，因此在加载速度上不会有任何明显的差异。所以没有理由，为什么你不应该这么做。

那么，为什么不创建自己的数据库类来自动创建呢？你只需要写一次。

class Database
{
    /* ... */
    private $connection = null;

    public function __construct($host, $user, $password, $database)
    {
        // connect
    }


    public function query($query)
    {
        $result = $this->mysqli->query($query);
        if(!result)
        {
            // error
        }
        else
        {
            return $result;
        }
    }

    public function __destruct()
    {
        // deconnect
    }
}

就这么说吧：

$db = new Database($host, $user, $password, $database);
$result = $db->query("select * from clients;");

Answer 2

是。去做吧。所有的数据库交互。

如果您关心检查每个数据库交互返回的开销.我倾向于担心您正在运行的数据库交互的数量。与运行不必要的查询或效率低下的查询，甚至是一个非常高效的查询的循环相比，可以用一个查询来执行.假的检查不会移动针头。

如果您想要担心代码路径的长度，请查看mysqli或PDO中的“幕后”发生了什么。或者看看其中一个ORM框架。那张假的支票连计量器上的针都不痒。

一定要考虑简化错误日志记录，这样您的代码就不会像处理错误那样混乱。我所说的精简，并不是说

   ... or die(mysqli_error($con));

Answer 3

您也可以这样做：

$select = $mysqli->query('SELECT * FROM users');
$delete = $mysqli->query('DELETE FROM customers');
$update = $mysqli->query('UPDATE table SET id={$id}');

if(!$select || !$delete || !$update) {
    // error messages
} else {
    // executions for each variable
}

但是，如果您希望为每个变量的错误处理提供不同的错误消息，那么您所做的是一个很好的实践。