将mysqli转PDO时遇错误：Call to a member function execute() on string

嘿，这个错误我太熟了——你这是把普通的SQL字符串当成PDO的语句对象来调用execute()方法了，这肯定行不通呀！PDO的操作流程和mysqli完全不一样，得严格走「建连接→准备语句→绑定参数→执行→取结果」的步骤，我帮你一步步把问题解决，顺便把价格排序的功能补全：

1. 先搞定PDO数据库连接（这是基础中的基础） #

首先你得确保search.php开头已经建立了有效的PDO连接，没有连接的话后续操作全白搭：

// 替换成你自己的数据库信息
$dsn = 'mysql:host=localhost;dbname=你的数据库名;charset=utf8mb4';
$db_user = '你的用户名';
$db_pass = '你的密码';

try {
    $pdo = new PDO($dsn, $db_user, $db_pass);
    $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); // 开启异常模式，方便调试出错
} catch (PDOException $e) {
    die('数据库连不上啦：' . $e->getMessage());
}

2. 重构SQL构建逻辑（解决语法+安全问题） #

你原来的代码有两个坑：一是WHERE前面少了空格，会导致SQL语法错误；二是直接把变量拼进SQL有注入风险。用PDO就别手动拼变量了，用参数绑定更安全：

$sql = "SELECT * FROM products WHERE deleted = 0";
$bind_params = [];

// 处理分类筛选
$cat_id = $_POST['cat'] ?? '';
if (!empty($cat_id)) {
    $sql .= " AND categories = :cat_id"; // 注意前面的空格！不然会拼出无效SQL
    $bind_params[':cat_id'] = sanitize($cat_id); // 如果你习惯用sanitize也可以，PDO参数绑定本身已经防注入
}

// 补全你没写完的价格升降序逻辑
$price_sort = $_POST['price_sort'] ?? 'ASC'; // 假设你的单选按钮name是price_sort，值是ASC/DESC
// 验证排序方向，防止恶意输入瞎搞
if (!in_array(strtoupper($price_sort), ['ASC', 'DESC'])) {
    $price_sort = 'ASC';
}
$sql .= " ORDER BY price {$price_sort}";

3. 正确执行PDO查询（核心修复点） #

现在你得把SQL交给PDO准备成可执行的语句对象，再调用execute()——这才是PDO的正确姿势，别再直接对字符串调用方法了：

try {
    $stmt = $pdo->prepare($sql);
    $stmt->execute($bind_params); // 把绑定的参数传进去
    $products = $stmt->fetchAll(PDO::FETCH_ASSOC); // 获取所有商品结果，关联数组格式
} catch (PDOException $e) {
    die('查询出错啦：' . $e->getMessage());
}

现在$stmt是正经的PDOStatement对象，调用execute()就不会报错了。

4. 给你补个前端表单参考（确保参数能传对） #

你的preview.php里的表单，价格排序部分大概要这样写，确保name属性和后端对应上：

<form method="POST" action="search.php">
    <!-- 分类单选按钮 -->
    <label>
        <input type="radio" name="cat" value="1" /> 数码产品
    </label>
    <label>
        <input type="radio" name="cat" value="2" /> 家居用品
    </label>

    <!-- 价格排序单选按钮 -->
    <label>
        <input type="radio" name="price_sort" value="ASC" checked /> 价格从低到高
    </label>
    <label>
        <input type="radio" name="price_sort" value="DESC" /> 价格从高到低
    </label>

    <button type="submit">开始筛选</button>
</form>

最后再划个重点 #

• 你之前的核心错误：没把SQL字符串转成PDOStatement对象，直接对字符串调用execute()
• 别忽略SQL拼接时的空格，不然会出现莫名其妙的语法错误
• 用PDO就尽量用参数绑定，比手动写sanitize靠谱多了，还能防SQL注入

内容的提问来源于stack exchange，提问作者Lortnok