php言語、エラー解決について

前提・実現したい事

php言語でカードゲームのデッキ登録サイトを作成しようと考えており、現在はカードの検索システムを組み立てようとしています。
検索機能を作成されているソースを元に作成しようとしたのですがエラーが発生し、中々次に進むことが出来ません。どなたか訂正箇所の方教えて頂けないでしょうか。

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

Warning:
1
2Warning: foreach() argument must be of type array|object, null given in C:\xampp\htdocs\test\card_search.php on line 27
3ID	カード名

該当のソースコード

<?php

  $dsn = "mysql:dbname=aaa;host=localhost;charset=utf8";
  $user = "root";
  $pass = "root";
  $options = [];
  $pdo = new PDO($dsn,$user,$pass,$options);
    if(@$_POST["id"] !="" OR @$_POST["カード名"] !=""){
      $stmt = $pdo->query("SELECT * FROM card WHERE ID='".$_POST["id"] ."' OR Name LIKE '%".$_POST["カード名"]."%')");
    }
?>

<!DOCTYPE html>
<html>
  <head>
  <title>Card Search</title>
  </head>
  <body>
  <h3 style="border-bottom: 1px solid #715c1f;border-left: 10px solid #715c1f;padding: 7px;">カード検索</h3>
    <form action="card_search.php" method="POST">
      ID:<input type="text" name="id"><br>
      Name:<input type="text" name="カード名"><br>
      <input type="submit" name="検索">
    </form>
      <table>
      <tr><th>ID</th><th>カード名</th></tr>
      <?php foreach ($stmt as $row): ?>
         <tr><td><?php echo $row[0]?></td><td><?php echo $row[1]?></td></tr>
      <?php endforeach; ?>
      </table>
  </body>
</html>

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

[id][カード名]　これがデータベースに登録している名前です。

行動規範の内容に同意します

回答2件

エラーの通り、
foreachにはarrayまたはobjectが与えられるべきところにnullがきた、ということです。

if(@$_POST["id"] !="" OR @$_POST["カード名"] !=""){
の式を満たさなければ$stmtという変数は定義されないので、それが原因かと。

冒頭で空配列として初期化しておけばとりあえず回避はできるでしょうけど、@でエラー出力を抑止したり、SQLインジェクション可能だったり、nameに全角が指定するような実装のほうが私は気になります。
データ取得できなくてもテーブル出ちゃいますしね。

投稿2021/02/01 23:45

m.ts10806

総合スコア80861

退会済みユーザー

2021/02/02 00:09

文字列の連結でSQL文編むなよなー、と。prepare→bindValue→executeしてほしいところ。かつ、try～catchも。

Sakuya1127

2021/02/02 00:47

回答ありがとうございます。参考にし、考えたいと思います。

yambejp

2021/02/02 00:51

ifの処理もおかしいですよ。合致しなかったときに$stmtが宣言されないのであとから参照は当然できません

m.ts10806

2021/02/02 01:10

「検索」なら、検索条件次第で動的にSQL組むようにすべきですね。今の実装だと両方入力しないと問い合わせないようになってますが、「検索したときだけ出す」にしても「初期アクセス」と「入力されない場合」にどうするかは設計で決めてください。

Sakuya1127

2021/02/02 01:47

丁寧にありがとうございます。初心者故にまだまだ理解が難しいのですが、頑張りたいと思います。

m.ts10806

2021/02/02 01:52

手書きレベルでいいので「なにをするか」「なぜするか」を書き出していくといいですよ。作る前に。簡易でも設計があるのとないのとでは雲泥の差が出ます。あと、エラーを読むこと。エラーはどうすればいいか教えてくれるアドバイザー。それに、どこかで必ず誰かがでくわしているので、ヒントすら見つからないというのはありえないものです。 https://qiita.com/cannorin/items/eb062aae88bfe2ad6fe5

行動規範の内容に同意します

流れ的にはこんな感じで

php
1$id=filter_input(INPUT_POST,'id');
2$card=filter_input(INPUT_POST,'card');
3$sql="SELECT * FROM card WHERE 1 "
4$data=[];
5if(empty($id) or empty($card)){
6  $sql.=" AND 0";
7}else{
8  $sql.=" AND ID=? AND Name LIKE ?";
9  $data[]=$id;
10  $data[]="%".$card."%";
11}?
12$stmt = $pdo->prepare($sql);
13$stmt->execute($data);
14$rows=$stmt->fetchAll(PDO::FETCH_ASSOC);
15print_r($rows);