年代のプルダウンを選択すると該当の年代の人がDBから検索できるようにしたい。

前提

実現したいこと

PHPで、年代のプルダウンを選択すると該当の年代の人がDBから検索できるようにしたい。

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

エラーメッセージ

Uncaught Error: Call to a member function fetchAll() on bool in D:\XAMPP\htdocs\pulldown\test.php:12 Stack trace: #0 {main} thrown in D:\XAMPP\htdocs\pulldown\test.php on line 12

該当のソースコード

HTML
1<!DOCTYPE html>
2<html lang="ja">
3<head>
4    <meta charset="UTF-8">
5    <meta http-equiv="X-UA-Compatible" content="IE=edge">
6    <meta name="viewport" content="width=device-width, initial-scale=1.0">
7    <title>プルダウン</title>
8</head>
9<body>
10    
11</body>
12</html>
13<form method='POST' action='test.php'>
14<select name='age'>
15<option value='10代～20代'>10代～20代</option>
16<option value='30代～50代'>30代～50代</option>
17<option value='30代～50代'>60代以上</option>
18</select>
19<input type='submit' value='送信' />
20</form>
21</body>
22</html>

PHP
1<?php
2class DB
3{
4    //MySQLとやり取りするクラス
5    private $USER = "root";
6    private $PW = '';
7    private $dns = "mysql:dbname=test;host=localhost;charset=utf8";
8
9    private function Connectdb()
10    {
11        //PDOのインスタンスを生成（接続を担当する）関数
12        try {
13            $pdo = new PDO($this->dns, $this->USER, $this->PW);
14            return $pdo;
15        } catch (Exception $e) {
16            return false;
17        }
18    }
19
20    public function executeSQL($sql, $array = null)
21    {
22        //SQLを実行する関数
23        try {
24            if (!$pdo = $this->Connectdb()) return false;
25            $stmt = $pdo->prepare($sql);
26            $stmt->execute($array);
27            return $stmt; //戻り値はPDOStatementのインスタンス
28        } catch (Exception $e) {
29            return false;
30        }
31    }
32}
33
34

PHP
1<?php
2//入力値の取得
3$age=$_POST["age"];
4
5//データベース接続
6if(isset($age)){
7require_once('db.php');
8	$db = new DB();
9	$sql = 'SELECT * FROM test WHERE age_data=?';
10    $data[] = $age;
11    $res = $db->executeSQL($sql);
12    if ($recs = $res->fetchAll(PDO::FETCH_ASSOC)) {
13        print '<ul>';
14
15        foreach ($recs as $rec) {
16            $ban = $rec['ban'];
17            $name = $rec['name'];
18            $age_data = $rec['age_data'];
19           ?>
20            <li><?= $ban ?>
21                <?= $name ?>
22                <?= $age_data ?>
23            </li>
24        <?php
25        }
26    }else {
27        print '結果が存在しません。';
28    }
29}
30?>
31

試したこと

Call to a member function fetchAll() on boolで調べたがわからなかった。

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

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

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

回答1件

ベストアンサー

$res = $db->executeSQL($sql);

ここで$dataを渡してないのでSQL文字列にある?パラメータに割り当てる値がなくてfalseが返ってきてるのでは。
せっかくExceptionをcatchしているのにfalseを返すだけで、どのようなException（特にエラーメッセージ）があるかをどこにも出力してないのと、falseが返ってきたときの実装の考慮がないことが原因と思います。

投稿2022/12/05 08:23

m.ts10806

総合スコア80888

officeyuyu3

2022/12/05 08:26

間抜けな質問で大変失礼しました。ありがとうございます。

m.ts10806

2022/12/05 08:42

引数に初期値を与えてると見えにくくなるのは確かですが、実装を見ると「age_data=?」とあるので、条件がある前提の実装になっていますが、初期値nullを与えているので、呼び元から引数無しで呼び出した場合にnullの場合に何もセットされない事態が起きるのではと。なので、追加で対応すべき方向性としては２つ（どちらか） ①条件がない場合のSQL文字列組み立ても考慮した実装にする ②必ず条件がある前提の実装にするのどちらかに軌道修正すべきと思います。あとエラーについてはオブジェクト指向についての基本をPHPマニュアルから学ばれると良いと思います。まあ「検索」とのことなので、未入力時の場合の検索がをするかしないかで決められると良いですね。

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