PHP MYSQLを使用し複数条件で検索したい

Question

現在２つの条件でMYSQLからデータを引っ張ってきたいのですが
下記のコードではnameだけのデータでしか検索・表示しかできておりません。

SQLを見てみますと
SELECT * FROM p1 WHERE 1=1 AND card_name LIKE :name

//p_no　と　nameを　前のページからformを使用
$p_no = filter_input(INPUT_GET, 'p_no');
$name = filter_input(INPUT_GET, 'name');

if (is_string($name)) {
  //MySQLデータベースは省略
  try {
    $pdo = new PDO($dsn, $user, $password);
    $pdo->setAttribute(PDO::ATTR_EMULATE_PREPARES, false);
    $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
    // SQL文を作る
    $sql = 'SELECT * FROM p1 WHERE 1=1';
    //検索条件
    if($p_no) $sql .= ' AND pp_no = :p_no';
    //名前の検索条件（あいまい検索）
    if($name) $sql .= ' AND card_name LIKE :name';
    $sth = $pdo -> prepare($sql);
    //プリペアドステートメントを実行する(インジェクション対策）
    if($area) $sth -> bindValue(':p_no', $p_no, PDO::PARAM_STR);
    if($name) $sth -> bindValue(':name', '%'.$name.'%', PDO::PARAM_STR);
    // $sth -> execute($sql);
    $sth -> execute();
    $result = $sth->fetchAll(PDO::FETCH_ASSOC);

　　echo $sql;

     if(count($result)>0){
      echo "名前に「{$name}」が含まれているレコード";
      // テーブルは省略

    } else {
      echo "名前に「{$name}」は見つかりませんでした。";
    }
  } catch (Exception $e) {
    echo '<span class="error">エラーがありました。</span><br>';
    echo $e->getMessage();
  }
}

原因は
$sth -> execute();
となっているのはわかるのですが、

$sth -> execute($sql);　としますと、（サンプルコードはexecute($sql)となっています）
エラー表示
Warning: PDOStatement::execute() expects parameter 1 to be array, string given in
となります。

参考にさせていただいたサンプルコードがあり、
それを自身の変数に変えるぐらいの知識しか無いため、

executeは引数に配列を期待しているがstringを渡しているための警告です。

という事がわかってもサンプルコードでは動いているはずなので、
どのPODをどのように変更してよいのかわかりません。

よろしくお願いします。

Comments

[zushi0905]

サンプルコードを提示できるのであれば、もっと詳しく回答できるかもしれないので提示して欲しいです！

[ooyuki]

https://kinocolog.com/where11/
上記のブログのサンプルコードを参考にさせていただきました。

よろしくお願いします。

Answer 1

if($area) $sth -> bindValue(':p_no', $p_no, PDO::PARAM_STR);

if($p_no) $sth -> bindValue(':p_no', $p_no, PDO::PARAM_STR);

ですね

} catch (Exception $e) {

ここはこうです

} catch(PDOException $e){

Comments

[ooyuki]

ご指摘ありがとうございます。
今までnoの検索が出来なかったのですが、できるようになりました。

$sth -> execute($sql);
でエラーが発生していましたので
その前だけ見て考えておりましたが、関係のない部分が原因だったとは・・・

本当にありがとうございます。

Answer 2

質問のコードではexecuteに渡している値がSQL文になっています。そもそも、executeに引数で渡すものはprepareで読み込んだSQL文のプリペアードステートメントのプレースホルダーに渡す値です。

今回の場合はbindValueで丁寧にパラメータを指定してあげてるので、executeの引数無しでいいとは思いますが、もしexecuteの引数でパラメータを渡すなら、配列を生成して渡してあげる必要があります。

Comments

[zushi0905]

bind関数でパラメータの数があっていれば、executeの引数は無視して実行できるんですね。勉強になりました。
サンプルコードの方はPDO::execとPDOStatement:executeを混同してる感じですね。
executeの引数はなしにしといた方がいいかとは思います。