PHP：検索フォーム

Question

DBに登録している商品を検索かけて、表示させたいです。
商品IDや名前を検索かけると表示はされるのですが、適当な値をいれても、データを取り出せてしまいます。もしかしたら、どこかのコードでとりあえず出力するようになってしまっているのかと・・・原因がつかめないためお力を貸してほしいです。

　　$hoge1 = $_POST['number']??''; // 商品番号
    $hoge2 = $_POST['name']??''; //商品名
    $hoge3 = $_POST['date']??''; // 入荷日
    if(empty($hoge1) && empty($hoge2) && empty($hoge3)){ // 初期状態を空白
        $sql = $pdo->query('SELECT * FROM items where id=0');
    // 条件が一つでも入力されていれば検索をかけれるようにする
    }elseif($sql=$pdo->query('SELECT * FROM items WHERE 1=1'));{
    if($hoge1 ){'AND number=:number';
    }
    if($hoge2 ){'AND name=:name';
    }
    if($hoge3 ){'AND date=:date';
    }
    }
    $rows = $sql->fetchAll();
    

初期表示を空にするため

if(empty($hoge1) && empty($hoge2) && empty($hoge3)){ // 初期状態を空白
        $sql = $pdo->query('SELECT * FROM items where id=0');

検索条件は、一つでも当てはまっていれば表示する

// 条件が一つでも入力されていれば検索をかけれるようにする
    }elseif($sql=$pdo->query('SELECT * FROM items WHERE 1=1'));{
    if($hoge1 ){'AND number=:number';
    }
    if($hoge2 ){'AND name=:name';
    }
    if($hoge3 ){'AND date=:date';
    }
    }

テーブルのコード

<tbody>
                        
                        <?php foreach ($rows as $key => $row): ?>
                            <tr data-index="0">
                                <th scope="row">
                                    <!-- 1-2: #ラジオボタン -->
                                    <input class="form-check-input ml-1" type="radio" name="flexRadioDefault" id="flexRadioDefault1">
                                    <label class="form-check-label" for="flexRadioDefault1"></label>
                                </th>
                                <td><?= $row['number']?></td>
                                <td><?= $row['name']?></td>
                                <td><?= $row['date']?></td>
                            </tr>
                            <?php endforeach; ?>
                        </tbody>

<form method = "post" action = "hogehoge.php" name="test">
                        <div class="form-group">
                            <div class="row">
                                <div class="col-4">
                                    <label for="number" class="control-label">商品番号:</label>
                                    <input type="number" class="form-control" id="number" name="number" value="">
                                </div>
                                <div class="col-4">
                                    <label for = "name" class = "control-label">商品名:</label>
                                    <input type = "text" class = "form-control" id = "name" name = "name" value="">
                                </div>
                                <div class="col-4">
                                    <label for = "date" class = "control-label">入荷日:</label>
                                    <!-- input="date"でカレンダー表示 -->
                                    <input type="date" class="form-control" id="date" name="date" placeholder="年/月/日" value="">
                                </div>
                            </div>
                            <!-- 2-1:クリア処理　ボタン:「クリア」・「検索」-->
                            <div class="form-group text-right mt-3">
                                <button class="btn btn-warning btn-sm" type="reset" name="reset" value="clear">クリア</button>
                                <button class="btn btn-info btn-sm" type="serch">検索</button>
                            </div>
                        </div>
                    </form>

Comments

[退会済みユーザー]

「if(empty($hoge1 && empty($hoge2) && empty($hoge3))」 でカッコが足りてない。転記ミスはほかにないですか？ あと、ソースまるごと貼ってくれないと、第三者が再現できないので、エスパー的回答に終止するしかないんですけど。

Answer 1

if(empty($hoge1 && empty($hoge2) && empty($hoge3))

↓

if(empty($hoge1) && empty($hoge2) && empty($hoge3))

くらいはパット見でわかった。

---

    if(empty($hoge1 && empty($hoge2) && empty($hoge3)){ // 初期状態を空白
        $sql = $pdo->query('SELECT * FROM items where id=0');
    // 条件が一つでも入力されていれば検索をかけれるようにする
    }elseif($sql=$pdo->query('SELECT * FROM items WHERE 1=1'));{

elseifのところ、おかしいよな？

---

    if($hoge1 ){'AND number=:number';
    }
    if($hoge2 ){'AND name=:name';
    }
    if($hoge3 ){'AND date=:date';
    }

何がしたいの？ 文字列リテラルだけ置いても意味がない。

ダメ出しばかりしているのは、仕様や設計に関わる情報が質問文中にないために、
どう改善すればいいかを考える段階にないってことです。
思考整理中に試しに実行したコードのまま、動作確認すら取ってないのかなって思えるほど、
phpコードからやりたいことが読み取れないので。

Answer 2

}elseif($sql=$pdo->query('SELECT * FROM items WHERE 1=1'));{

ここ、最初のifを満たさなければ必ず実行されてます。

if($hoge1 ){'AND number=:number';

}
if($hoge2 ){'AND name=:name';
}
if($hoge3 ){'AND date=:date';

}

ここ、なんにもしてません。

SQLを動的に組み立てたいのでしょうけど、
これでは組み合わせの検索ができませんので、
配列に追加していってimplodeでandで結合すると良いです。

また、たとえ条件いれてもパラメータセットしててバインドしてないのでエラーになるのでは（ちゃんとtry-catchしてね）

いずれにしても下記のような記事は読んで、基本的な概念や組み方は覚えましょう。

PHPでデータベースに接続するときのまとめ

Answer 3

<?PHP
$number=filter_input(INPUT_POST,'number');
$name  =filter_input(INPUT_POST,'name');
$date  =filter_input(INPUT_POST,'date');
$data  =[];
$sql='SELECT * FROM items where 1 ';
if(count(array_filter([$number,$name,$date],function($x){
  return !empty($x);
}))==0){
  $sql.=" and 0";
}
foreach(["number","name","date"] as $val){
  if($$val){
    $sql.=" and {$val}=?";
    $data[]=$$val;
  }
}
print $sql.";<br>\n";
print_r($data);
?>
<hr>
<form method="post">
number:<input name="number" value=""><br>
name:<input name="name" value=""><br>
date:<input name="date" value=""><br>
<input type="submit" value="send">
</form>

※一部フローを調整

Comments

[m.ts10806]

array_filterって便利ですね。

[yambejp]

値が0だったときにemptyと判断するか・・・など課題はあります

[m.ts10806]

まあそこは仕様次第ですね。

[xxx0_0xxx]

大変ご丁寧な回答ありがとうございます。

@yambejp様のコードを基に、書いてみたものの、私の情報不備でエラーが出てしまいました。
$$valが見つからないというエラーが出るのですが、テーブルの取得が間違っているのでしょうか・・・

[m.ts10806]

>@m.ts10806様のコードを基に

宛先私ではないと思います。
yambejpさんも突貫で書かれたと思うので、変数名一部間違ってますね。

[yambejp]

可変変数が利用できないということでしょうか？

https://www.php.net/manual/ja/language.variables.variable.php

[m.ts10806]

失礼、最初と変数名連携してましたね。

[xxx0_0xxx]

「可変変数」というものがあるのですね！
変数を上書きするということなのでしょうか？
すみませんまだまだ勉強不足です＞＜

[yambejp]

変数を上書きしません
参考でつけているリンク先をよく読んでみてください

[m.ts10806]

「参考に組んだ」は理解できてないと使いこなせないと思うので実際に自身が組んだコードを提示されたほうが良いかと思いますし、yambejpさんの今回の回答のコードは、質問者さんにはかなり難しいと思います。

[yambejp]

TO:xxx0_0xxxさん
m.ts10806さんがおっしゃるように、回答者が独りよがりの回答をして
理解できないようなときには無理にBAをつける必要はないんですよ。

[xxx0_0xxx]

丁寧にコードを書いていただいたのでBAにしてしまったので！
でも解決済みになってしまうことを知らず、外させてもらいました＞＜
ごめんなさい混乱させてしまってm(__)m

[xxx0_0xxx]

m.ts10806様のおっしゃる通り、まだまだ私にはこのコードが理解できない状態です。
if(count(array_filter([$number,$name,$date],function($x){
return !empty($x);
}))==0){
$sql.=" and 0";
}
foreach(["number","name","date"] as $val){
if($$val){
$sql.=" and {$val}=?";
$data[]=$$val;
}
}

ここの部分でどういう動きになっているのか、よければご教授下さい。

[yambejp]

まず最初のifと、あとのforeachは独立してますので順に解説
array_filter([$number,$name,$date],function($x){
return !empty($x);
})
は、$numberと$nameと$dateのそれぞれから空でないものを抽出しています
それをcountしてるので空でない個数で、それが==0なのですから
ifの条件分は「すべてが空」という意味です。
その時、$sql文にand 0がつき、where 1 and 0となります
SQLの抽出条件としてはand 0は常に偽となるためSQLは必ず空を返します

foreachの方は、"number","name","date"という文字列をそれぞれ取り出して$valに入れて
ループ処理をします
仮に"number"を$valにわたすとif($$val)というのはif($number)と同等なので
$numberが空ではない場合という条件になります
SQL文には" and number=?"という文字が追加され、$dataに$numberを追加します
これをnameやdateについても続けます。

あとは私の回答をご自身の環境でコピペしてnumber、name、dateの入力欄に
値を入れたり、入れなかったりしてサブミットすると
どういう処理がされるかわかると思います。

なお最終的に$sqlはprepareを使って実行され、その値は$dataをexecuteするとよいでしょう

[xxx0_0xxx]

大変詳しいアドバイスありがとうございます！
おかげさまで、見事に実装することできました。
やはり、基礎からやり直してみたいと思います。
今後とも何卒ご教授いただければ幸いです！！