速记 PDO 查询

Question

目前要使用 PDO 执行查询，我使用以下代码行：

$sql = "SELECT * FROM myTable WHERE id = :id";
$stmt = $conn->prepare($sql);
$stmt->bindParam(':id', $id);
$stmt->execute();
$result = $stmt->fetchAll(PDO::FETCH_ASSOC);

经过一番研究，我发现了一种执行相同命令的更短方法：

$stmt_test = $conn->prepare("SELECT * FROM status WHERE status_id = ?");
$stmt_test->execute([$id])->fetchAll(PDO::FETCH_ASSOC);
$result = $stmt_test->fetchAll(PDO::FETCH_ASSOC);

从那里我认为我可以使用以下代码使其更短：

$stmt_test = $conn->prepare("SELECT * FROM status WHERE status_id = ?");
$result = $stmt_test->execute([$id])->fetchAll(PDO::FETCH_ASSOC);

但我收到以下错误：

致命错误：在非对象上调用成员函数 fetchAll() /home/.../index.php 第 20 行

问题：为什么会出现此错误？据我了解，$stmt_test->execute([$id]) 应该首先执行，然后其结果将执行->fetchAll(PDO::FETCH_ASSOC) 并从那里将数组返回到$result，但是由于发生了错误，所以我的逻辑一定有缺陷。我究竟做错了什么？另外，有谁知道执行上一个查询的更好的速记方法吗？

Answer 1

因此，您已经得到了“为什么我收到此错误”问题的答案，但没有得到“速记 PDO 查询”的答案。

为此，我们需要一些叫做“编程”的东西。

关于编程的一个有趣的事情是，我们不局限于现有的工具，就像其他职业一样。通过编程，我们总是可以创建自己的工具，然后开始使用它而不是一整套旧工具。

而面向对象编程尤其擅长于此，因为我们可以获取现有对象并添加一些功能，其余部分保持不变。

例如，假设我们想要一种在 PDO 中运行准备好的查询的简写方式。我们所需要的只是用一种新的速记方法扩展 PDO 对象。最难的部分是为新方法命名。

剩下的很简单：你只需要几行代码

class MyPDO extends PDO
{
    public function run($sql, $bind = NULL)
    {
        $stmt = $this->prepare($sql);
        $stmt->execute($bind);
        return $stmt;
    }
}

这是您需要的所有代码。您可以将其存储在存储数据库凭据的同一文件中。请注意，此添加不会以任何方式影响您现有的代码 - 它保持完全相同，您可以像往常一样继续使用所有现有的 PDO 功能。

现在你只需在 PDO 构造函数中更改 2 个字母，将其称为

$conn = new MyPDO(...the rest is exactly the same...);

您可以立即开始使用您闪亮的新工具：

$sql = "SELECT * FROM myTable WHERE id = :id";
$result = $conn->run($sql, ['id' => $id])->fetchAll(PDO::FETCH_ASSOC);

或者，给它一点优化，

$result = $conn->run("SELECT * FROM myTable WHERE id = ?", [$id])->fetchAll();

因为您始终可以一次性设置默认提取模式，并且对于单个变量，命名占位符没有用处。与公认的答案相比，这使得这段代码成为真正的简写，

$stmt_test = $conn->prepare("SELECT * FROM status WHERE status_id = ?");
$stmt_test->execute([$id]);
$result = $stmt_test->fetchAll(PDO::FETCH_ASSOC);

甚至是迄今为止你得到的最佳答案，

$result = $conn->prepare("SELECT * FROM status WHERE status_id = ?");
$result->execute([$id]);

更不用说后者并不总是可用的，因为它只适合获取一个数组。使用 real 速记时，任何结果格式都是可能的：

$result = $conn->run($sql, [$id])->fetchAll(); // array
$result = $conn->run($sql, [$id])->fetch(); // single row
$result = $conn->run($sql, [$id])->fetchColumn(); // single value
$result = $conn->run($sql, [$id])->fetchAll(PDO::FETCH_*); // dozens of different formats

Answer 2

$stmt_test->execute([$id]) 返回一个 boolean 值。这意味着

$result = $stmt_test->execute([$id])->fetchAll(PDO::FETCH_ASSOC);

无效。相反，你应该这样做

$stmt_test->execute([$id]);
$result = $stmt_test->fetchAll(PDO::FETCH_ASSOC);

Answer 3

您遇到的错误来自 PDO 的设计方式。 PDOStatement::execute() 不返回语句，而是一个表示成功的布尔值。因此，您想要的快捷方式是不可能的。

见http://php.net/manual/en/pdostatement.execute.php中的函数定义

另外让我补充一点，forEach() 通常（并非总是）是一种代码异味，并且占用相对较多的内存，因为它必须将所有行存储为 PHP 值。

Answer 4

我相信 PDO 的 execute() 方法返回 true 或 false。正如错误已经告诉你的那样：fetchAll() 需要一个对象。使用三行代码将是最短的方法。

另一种选择是使用像propel 这样的 ORM，它运行起来非常流畅，可以为您节省大量时间。