PHPの$stmt->execute()ができない。

Question

前提・実現したいこと

下記のモデルに検索ワード($_GET['search'])を送ると検索結果を表示する処理を作りたいのですが、
$articles->execute()の部分でtrueにならず次の処理へ進めません。
ここをtrueにしたいです。

※$_GET['search']が''の状態だと記事を取得することができます。

発生している問題・エラーメッセージ

$articles->execute()がfalseになってしまう。

該当のソースコード

  {
    if (isset($_GET['search']) && !empty($_GET['search'])) {
      $word = preg_replace('/(\s|　)+/', ',', $_GET['search']);
      if (strpos($word, ',') === 0) {
        $word = substr($word, 1, strlen($word) - 1);
      }
      $searchWord = explode(',', $word);
      $keywordCondition = [];
      for ($i = 0; $i < count($searchWord); $i++) {
        $keywordCondition[] = "title LIKE :title{$i}";
      }
      $title = implode(' AND ', $keywordCondition);
    } else {
      $title = '';
    }

    if ($_GET['category'] !== '') {
      $category = 'category = :category ';
    } else {
      $category = '';
    }

    $top = filter_input(INPUT_GET, 'top');
    switch ($top) {
      case 'all':
        $top = '';
        break;
      case '0':
        $top = "top = 0";
        break;
      case '1':
      case '2':
        $top = '(top = :top OR top = 3)';
        break;
      case '3':
        $top = "top = 3";
        break;
    }
    if ($title === '' && $top === '' && $category === '') {
      $where = '';
    } else {
      $where = 'WHERE ';
    }
    $and1 = '';
    $and2 = '';
    $and3 = '';
    if ($title !== '' && $category !== '' && $top !== '') {
      $and1 = ' AND ';
      $and2 = ' AND ';
    } else if ($title !== '' && $category !== '' && $top === '') {
      $and1 = ' AND ';
    } else if ($title === '' && $category !== '' && $top !== '') {
      $and2 = ' AND ';
    } else if ($title !== '' && $category === '' && $top !== '') {
      $and3 = ' AND ';
    }

    $sql = "SELECT * FROM articles  {$where} {$title} {$and1} {$and3} {$category} {$and2} {$top}  ORDER BY created DESC";
    $articles = $this->db->prepare($sql);
    $i = 0;
    if (isset($searchWord)) {
      foreach ($searchWord as $search) {
        $articles->bindValue(":title{$i}", '"%' . $search . '%"');
        $i++;
      }
    }
    if ($_GET['category'] !== '') {
      $articles->bindValue(':category', $_GET['category']);
    }
    if ($_GET['top'] !== '') {
      $articles->bindValue(':top', $_GET['top']);
    }

    if ($articles) {
      if ($articles->execute()) {
        while ($row = $articles->fetch()) {
          $rows[] = $row;
        }
        if (!isset($rows)) {
          $errors = '0件でした';
          return $errors;
        }
      }
      $this->db = null;
      return $rows;
    }
  }

試したこと

下記のbindValue部分で値がバインドされているか確認すると、trueとなった。
追記、trueではなく１が返ってきてました。

if (isset($searchWord)) {
      foreach ($searchWord as $search) {
        $articles->bindValue(":title{$i}", '"%' . $search . '%"');
        $i++;
      }
 }

補足情報（FW/ツールのバージョンなど）

heroku/7.42.13
PHP 7.1.33
macOS Mojave 10.14.6

Answer 1

この状態で他人がデバッグするのはかなり難しいので、
まずは

• PHPでデータベースに接続するときのまとめを参考にしてSQLエラーがあったら例外を吐くようにしてcatchして確認

• PDOStatement::debugDumpParams

を仕込んでSQLプリペアドコマンドを確認、

• 各所の変数をvar_dump()するなりxdebugでブレークポイントを仕掛けて想定通りになっているか確認。

• 怪しそうな変数を固定値に置き換えてテストしてみる

あたりでデバッグをしていって、その結果を整理して質問に追記されるとより適切な回答がえらるかと思います。（整理している間に解決することも多いです）

---

自分がデバッグするとしたら（例外は吐かせるようにしておくのは前提として）

$sql = "SELECT * FROM articles  {$where} {$title} {$and1} {$and3} {$category} {$and2} {$top}  ORDER BY created DESC";

が想定通りになっているかを

$sql = "SELECT * FROM articles  {$where} {$title} {$and1} {$and3} {$category} {$and2} {$top}  ORDER BY created DESC";
var_dump($sql);
die();

等で想定通りになっているか確認するところからスタートすると思います。
これが想定通りなら、
その出力された文字列を使って簡単なテスト用スクリプトを書いて、想定が正しいかをチェックします。

Comments

[suika11]

ありがとうございます。アドバイスを参考に再度デバックを行ってみます。

[tanat]

まずは
PHPでデータベースに接続するときのまとめ
の
接続後にオプションを指定
の
$pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
$pdo->setAttribute(PDO::ATTR_EMULATE_PREPARES, false);
が最も重要なところです。

質問のソース中に書くなら$pdoの部分を$this->dbに差し替えればとりあえずは動くと思います。（出来ればDB接続している部分に追加する方が良いですが）

[suika11]

ありがとうございます。オプションの設定も勉強になりました。

無事解決することができました。

SQLに検索ワードをバインドするときに、下記のようにダブルクォーテーションをバインドさせていたのですが、それを外したらうまく動作しました。
×$articles->bindValue(":title{$i}", '"%' . $search . '%"');
○$articles->bindValue(":title{$i}", '%' . $search . '%');

どこかでLIKE検索のさい値をバインドするときはダブルクォーテーションを含める、みたいな話を聞いたことがあったのですがそれが違ったようでした。
ありがとうございました。

Answer 2

$sqlで名前付きプレースホルダの書き方が間違ってます。
あと$whereが常に設定されないため、SQLのSyntaxエラーが発生していると思われます。
PDOStatement::bindValue

Comments

[suika11]

コメントありがとうございます。
名前付きプレースホルダを書き方は間違っているでしょうか？
検索ワードが複数あった場合に備えて、$keywordCondition[] = "title LIKE :title{$i}";としたのですが、
どう記述したらよいですか？

[退会済みユーザー]

LIKE '%' . :title{$i} . '%'
かな？

[suika11]

プレスホルダ作成の段階で%とつけたほうがいいということですか？
bindValueのバインド時につけてしまいましたが。

[退会済みユーザー]

おお、見落としてました。
どちらが正解か検証してみてください。

[suika11]

どうも調べたらバインドするときに%をつけるほうが良いとのことでした。