关闭准备好的语句

Question

我知道结束准备好的语句是a suggested thing to do。

但我有一个像这样的 PHP 脚本：

$sql = "SELECT * FROM `mytable` WHERE ...";
$stmt = $dbh->stmt_init();

if($stmt->prepare($sql))
{
    $stmt->bind_param("s", $user);

    if($stmt->execute())
    {
        $result = $stmt->get_result();
        $stmt->close();
    } else
        header("Location: .?error=unknown");
} else
    header("Location: .?error=conn");

如果一切正常，该语句将关闭，但在执行过程中发生故障时，它不会关闭。

我应该写吗

else {
    $stmt->close();
    header("Location: .?error=unknown");
}

和

else {
    $stmt->close();
    header("Location: .?error=conn");
}

或者，既然发生了错误，我不应该担心关闭语句？

或者我什至可以写：

$sql = "SELECT * FROM `mytable` WHERE ...";
$stmt = $dbh->stmt_init();

if($stmt->prepare($sql))
{
    $stmt->bind_param("s", $user);

    if($stmt->execute())
    {
        $result = $stmt->get_result();
    } else
        header("Location: .?error=unknown");
} else
    header("Location: .?error=conn");

/*some other code*/

$stmt->close; //close every statement at the very end of the script

还是在我使用完准备好的语句后立即关闭它们以避免任何类型的错误更好？

Answer 1

你的代码的问题不是close()的使用，而是报错，没有开启。您必须启用mysqli error reporting，然后才能简化您的代码。

理想情况下，您的数据库操作应该封装在一个函数或方法中，这样您就不必担心关闭任何内容。它将自动为您关闭。

$sql = "SELECT * FROM `mytable` WHERE ...";
$stmt = $dbh->prepare($sql);
$stmt->bind_param("s", $user);
$stmt->execute();
$result = $stmt->get_result();
$stmt->close();

如果你要将它包装在一个函数中，那么就不需要close()。

function getResults(\mysqli $dbh, string $sql, string $types, array $params): array {
    $stmt = $dbh->prepare($sql);
    $stmt->bind_param($types, ...$params);
    $stmt->execute();
    return $stmt->get_result()->fetch_all(MYSQLI_ASSOC);
}

然后，如果您有异常，您应该创建一个通用错误处理程序，当错误发生时，它将通过生成 HTTP 500 响应代码将用户重定向到 500 页面，并将记录所有异常详细信息到服务器上的一个文件中。

Answer 2

不要关闭它。

尽管其他答案（写于 10 年前）说了什么，但通常您不会关闭准备好的陈述。只是没有必要。当前作用域关闭时会自动关闭。这意味着即使在脚本执行期间您也不关心关闭语句，更不用说完全完成 PHP 脚本 - 在这种情况下，整个数据库连接将自动关闭，它会释放所有相关的资源也是如此。

顺便说一句，运行简单查询所需的代码几乎是所需的五倍。以下是它应该如何完成，整洁和简洁：

$sql = "SELECT * FROM `mytable` WHERE ...";
$stmt->prepare($sql);
$stmt->bind_param("s", $user);
$stmt->execute();
$result = $stmt->get_result();

问题是，您的数据库代码永远不应该报告它的错误。其他地方的代码应该有一个不同的部分，可以处理所有错误。

不，在应用程序错误的情况下执行 HTTP 重定向不是要走的路。您的应用程序应该只返回一个正确的 HTTP 代码 (5xx) 来告诉客户端有问题。