PDO 准备和执行查询总是返回错误

Question

我有一个函数，

这总是返回给我一个错误，即使在我过去查询时在 phpmyadmin 中有一个结果。

我想我的查询有问题，但我不知道是什么。我以前做我的代码做mysql_real_escape_string，然后我转向PDO，他们告诉我应该为转义GET vars做准备，所以我试着去做。

以下是我的查询

public static function getDetailService($param) {
    global $bdd;
    $detail = $bdd->prepare('SELECT
            spb_services.spb_services__name,
            spb_services.spb_services__description,
            spb_services.spb_services__banner,
            spb_services.spb_services__tabs,
            spb_services.spb_services__category
        FROM spb_services
        WHERE spb_services.spb_services__name LIKE :service');

    $detail->bindValue(':service', $_GET[$param], PDO::PARAM_STR);
    $resultat = $detail->fetchAll(PDO::FETCH_ASSOC);

    //var_dump($_GET[$param]);

    $detail->debugDumpParams();
    $lignes = $detail->fetchColumn();

    //var_dump($lignes);
    $detail = $detail->fetchAll(PDO::FETCH_ASSOC);
    $retour = ($lignes > 0) ? array('status' => 'ok') : array('status' => 'error');
    var_dump($retour);
}

当我调用函数时：$service = nosServices::getDetailService('service');

var_dump($_GET[$param]) 的 Var 转储返回给我预期的内容（来自 url）

然后我做了$detail->debugDumpParams();

我在我的本地主机 phpmyadmin 中通过了查询，它返回给我预期的结果，但在使用 PDO 时没有。

我猜有些小问题是错的，但我不知道是什么。

这不会返回错误，但总是错误，好像没有num_rows_result

总结一下麻烦，GET返回了预期的结果，但是当我们去查询时，它没有返回任何结果（除了在我的phpmyadmin复制并粘贴查询）

我们将不胜感激任何帮助

编辑：其他用户按预期完成的修改

Answer 1

参数绑定需要修正。

您在为参数提供值时缺少:。

更正的代码：

...
WHERE spb_services.spb_services__name LIKE  :service');
        $detail->bindValue(':service', $_GET[$param], PDO::PARAM_STR);
...

Answer 2

只有代码中的问题忘记在获取数据之前执行查询它会是

 $detail->bindValue(':service', $_GET[$param], PDO::PARAM_STR);
 $detail->execute();// execute it first
 $resultat = $detail->fetchAll(PDO::FETCH_ASSOC);
 print_r($resultat);

Answer 3

您的代码中发生了许多奇怪的事情。

我已经评论了我认为需要改变的地方

public static function getDetailService($param) {
    global $bdd;    // bad practice, see later suggestion
    $detail = $bdd->prepare('SELECT
                     spb_services.spb_services__name,
                     spb_services.spb_services__description,
                     spb_services.spb_services__banner,
                     spb_services.spb_services__tabs,
                     spb_services.spb_services__category
                FROM spb_services
                WHERE spb_services.spb_services__name LIKE :service');

    // $GET? I assume you want to use the `$param you pass as a param to this function
    //$detail->bindValue(':service', $_GET[$param], PDO::PARAM_STR);

    // a LIKE normally requires a string like '%something%'
    // or 'something%'
    // DO we assume you passed $param with the wildcards already applied?
    $detail->bindValue(':service', $param, PDO::PARAM_STR);

    // now the prepared query must be executed
    $detail->execute();

    // fetchAll returns ALL the result set into an array
    $resultat = $detail->fetchAll(PDO::FETCH_ASSOC);

    // as you are using a LIKE we have to assume there will be more 
    // than one row returned. 

    // fetchColumn makes no sense here
    //$lignes = $detail->fetchColumn();

    // You already did a fetchAll so this makes no sence
    //$detail = $detail->fetchAll(PDO::FETCH_ASSOC);

    // as all you appear to be doing is testing if one or more rows are returned
    // then all you need to do is coumt the occurances in the $resultat array

    $retour = (count($resultat) > 0) ? array('status' => 'ok') : array('status' => 'error');

    // Now you need to return something
    return $retour;

}

在类方法中使用global 也是不好的做法，因为它会破坏封装，最好将类似的东西作为参数传递。

EG这个

public static function getDetailService($param) {
    global $bdd;

变成这样

public static function getDetailService($bdd, $param) {
    // global $bdd;   <-- no longer needed

或者，如果整个班级都需要它，则将其设为班级属性！

如果我是对的，并且您想从这种方法中知道的只是是否存在某些东西，SELECT COUNT(id) as cnt 将是一种更有效的方法，但让我们再等一天，因为它也会改变您的获取方式结果并编写此代码的其余部分

最后注意：在任何可能出错的 PDO 语句之后，您都没有进行错误检查。几乎所有这些 PDO 语句都返回一个状态，如果为 false，您应该向自己显示 PDO 生成的错误消息，请参阅ErrorInfo

Answer 4

你忘了做 $detail->execute();处理 SQL 请求

$detail->bindValue(':service', $_GET[$param], PDO::PARAM_STR);
if( ! $detail->execute()) {
   die('Invalid Mysql Query!');
}
$resultat = $detail->fetchAll(PDO::FETCH_ASSOC);

希望有帮助:)