PDOで '？' にbindされない

Question

前提・実現したいこと

検索窓からMySQLDB内のgift_tbから、複数のカラムをconcat関数で結合し、LIKE検索してデータを取り出したいのですが上手くいきません。

var_dump($sql) ： " SELECT * FROM gift_tb WHERE CONCAT('gift_name','detail','reason') LIKE ? ORDER BY postdate desc limit 15 "
var_dump($arrVal) :  array(1) { [0]=> string(13) "'%検索ワード%'" } 

となっていて、？に%検索ワード%がバインドされません。
ただ、public function select($table, $column ='', $where ='', $arrVal =[])は他の場面では機能しています。
値をバインドする部分がよくわかっていません。
ご教授願います。

※検索文字列に"%"が含まれる時ののエスケープ処理はとりあえず飛ばしています。

該当のソースコード

top.php

if(isset($_GET['search']) === true){
    $search_word = htmlspecialchars($_GET['search']);
    // var_dump($search_word);
    $dataArr = $gift->getSearchGift($search_word);
    var_dump($dataArr);
    if($dataArr == false){
        $errMsg = '" ' . $search_word . ' "'. ' を含む 対象は見つかりませんでした。';
    }
}

Gift.class.php

public function getSearchGift($search_word)
    {
        $table = 'gift_tb';
        $col = '';
        $where = " CONCAT('gift_name','detail','reason') LIKE ?";
        $arrVal = ($search_word !== '') ? ["'%" . $search_word . "%'"] : [];
        // var_dump($arrVal);
        $res = $this->db->select($table, $col, $where, $arrVal);
        var_dump($res);
        return ($res !== false && count($res)  !==  0) ? $res : false;
    }

PDO.class.php

public function select($table, $column ='', $where ='', $arrVal =[])
    {
        $sql = $this->getSql('select' , $table, $where, $column);
        
        $this->sqlLogInfo($sql, $arrVal);
        var_dump($sql);
        var_dump($arrVal);
        $stmt = $this->dbh->prepare($sql);
        $res = $stmt->execute($arrVal);

        if($res === false){
            $this->catchError($stmt->errorInfo());
        }
        var_dump($res);
        $data = [];
        while ($result = $stmt->fetch(\PDO::FETCH_ASSOC)){
            array_push($data,$result);
        }
        return $data;

    }

private function getSql($type, $table, $where = '', $column = '')
    {
        switch ($type){
            case 'select':
                $columnkey = ($column !== '') ? $column : "*";
                break;

            case 'count':
                $columnkey = 'COUNT(*) AS  NUM ';
                break;
            
            default:
                break;
        }

        $whereSQL =  ($where !== '') ? ' WHERE ' . $where : '';
        $other = $this->groupby . " " . $this->order . " " . $this->limit . " " . $this->offset;

        //SQL文の作成
        $sql = " SELECT " . $columnkey . " FROM " . $table . $whereSQL . $other;
        var_dump($sql);
        return $sql;
    }

   public function setOrder($order = '', $limit= '')
    {
        if ($order !== ''){
            $limit2 = ($limit !== '') ? $limit: 'desc limit 20';
            $this->order = ' ORDER BY ' . $order . ' ' . $limit2;
        }
    }

試したこと

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

ここにより詳細な情報を記載してください。

Comments

[m.ts10806]

個人的な要望で申し訳ないのですが、同じファイル内やクラス内の記述であればコードブロックをまとめていただけますか？別ブロックにまとめられているとぶつ切りになっていて関連性や連動性が見えづらくなりますので。

[arahito]

複数ファイルにまたがっていて、関連性が逆にわかりづらくなるかと思いこのようにしていました。
以降ファイルごとに記述するようにします。

[m.ts10806]

複数ファイルなのであればファイル名がコードブロックの前などにあるとわかりやすくなります。
質問は編集できますので適宜調整いただければと

[m.ts10806]

解決した後で申し訳ないですが、
ファイル名らしきものを書かれた場所は本来「言語名」を書く場所なのでこんな感じにしてもらえたらと
※次回からで結構です。
xxx.php
```php

または
```php
//xxx.php

[arahito]

言語名を入力する箇所にドットが使えなくて表記に困っていました。
こういう部分を解説されることがないのでとても助かります。
ありがとうございます。

[m.ts10806]

まあcodeボタン押したときに「ここに言語名を入力」と出るので「そこで察してね、他は工夫してね」ということかもしれませんね。
以前は対応している言語名以外は未入力扱いになってたので、汎用的になりすぎた弊害と言えるかもしれません。

Answer 1

必要な変数の値を調べるところまで行き着いているのであとはよく見るだけです。

var_dump($arrVal) :  array(1) { [0]=> string(13) "'%検索ワード%'" } 

ということなので、?には「%検索ワード%」ではなく「'%検索ワード%'」がバインドされてしまっています。これでは期待した検索は行われないでしょう。

つまり、getSearchGiftで"'%" . $search_word . "%'"としてますが「'」が余計です。

あとこれは直接今回の質問とは関係ないですが、以下のコードだと$search_wordが空文字列の時にプレースホルダの数と$arrValの要素の数が合わなくなると思います。

        $where = " CONCAT('gift_name','detail','reason') LIKE ?";
        $arrVal = ($search_word !== '') ? ["'%" . $search_word . "%'"] : [];

Comments

[arahito]

他のブログやサイトを見ていて、どのページにもにもSELECT * FROM テーブル名 WHERE カラム名　LIKE '%検索文字%'と書いてあったので、テーブル名やカラム名にはシングルクオーテーションは不要で%検索文字%には必要なものだと思っていました。ありがとうございます！
また、プレースホルダの数と$arrValの要素数の問題もご指摘ありがとうございます。
空で検索した時は条件分岐して解決しました。

Answer 2

いくつか問題点があります

<?PHP
$search=filter_input(INPUT_GET,"search");
if(!is_null($search) and $search!==""){
  getSearchGift($search);
}

function getSearchGift($search_word){
  $table = 'gift_tb';
  $where = "and CONCAT(`gift_name`,`detail`,`reason`) LIKE ?";
  $sql="select * from {$table} where 1 {$where}";
  $data=["%{$search_word}%"];
  print "{$sql};<br>\n";
  print_r($data);
}
?>

<form>
<input type="text" name="search">
<input type="submit" value="send">
</form>

元ソース

$search_word = htmlspecialchars($_GET['search']);

検索文字をhtmlspecialcharsしてはいけません。変換されるとヒットしません
もちろん検索後の文字をhtmlspecialcharsするのは忘れずに

$where = " CONCAT('gift_name','detail','reason') LIKE ?";

カラム名をシングルクォーテーションでくくってはいけません
文字列として判断されます

既出

["'%" . $search_word . "%'"]

検索文字をシングルクォーテーションでくくってはいけません
シングルクォーテーション付きの文字を検索しようとします