MYSQL Call to a member function prepare() on nullを解決したい

Question

実現したいこと

prepare()でCall to a member function prepare() on nullが出てしまっているので、そちらを解決して、削除し処理を完成させたいです。

現状

現在MYSQL練習として簡単なtodoリストを作成しているのですが、todoを完了した時に、そのデータを削除するための処理でこのエラーが出てしまい、エラー文で検索をかけたりしたのですが、いまいち解決にたどり着けないので質問させていただいた次第です。

該当のソースコード

//index.phpです
require_once "DBconnect.php";
try {
    $pdo->beginTransaction();
    $sql = "INSERT INTO todo (todo) VALUES (:todo)";
    $stmh = $pdo->prepare($sql);
    $stmh->bindValue(':todo', $_POST['todo'], PDO::PARAM_STR);
    $stmh->execute();
    $pdo->commit();
} catch (PDOException $Exception) {
    $pdo->rollBack();
    print 'エラー：' .$Exception->getMessage();
}
?>
<div class="usermessage">
    <table border="1">
        <h1>やること</h1>
        <?php
    require_once "DBconnect.php";
    $sql = "SELECT * FROM todo WHERE 1";
    $stmh = $pdo->prepare($sql);
    $stmh->execute();

        while(1) 
        {
            $row = $stmh->fetch(PDO::FETCH_ASSOC); //１レコード取り出し
            if($row == false) //データがもうない場合
            {
                break; //ループから脱出
            }
            ?>
                <h3><?=htmlspecialchars($row['todo'], ENT_QUOTES)?></h3> <!-- データベースのデータを表示  -->
                <a class="btn btn-outline-primary" href=index.php?id=". $row['id'] ." > 完了 </a>
            <?php
        } //whileの閉じタグ
    $pdo = null;
    ?>
</div>

<?php
//削除処理
try {

    $stmh = $pdo->prepare('DELETE FROM todo WHERE id = :id'); //この文でエラーが出ています
    $stmh->execute(array(':id' => $_POST['id']));
} catch(PDOException $Exception) {
    echo 'エラーが発生しました。:' . $Exception->getMessage();
}

//DBconnect.phpです
$user = "root";
$password = "root";
$host = "localhost";
$dbname = "todolist";
$dbtype = "mysql";
        
$dsn = sprintf(
    "%s:host=%s;dbname=%s;charset=utf8",
    $dbtype,
    $host,
    $dbname,
);
        
try {
    $pdo = new PDO($dsn, $user, $password);
    $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
    $pdo->setAttribute(PDO::ATTR_EMULATE_PREPARES, false);
} catch (PDOException $Exception) {
    die('エラー：' .$Exception->getMessage());
}

エラーが発生している、
$stmh = $pdo->prepare('DELETE FROM todo WHERE id = :id');の$pdoが悪いのはわかっているのですが、僕が確認したところだと問題はなかったです。

以上です。
わかる方がいらしたらご回答よろしくお願いします。

Answer 1

ロジックを整理した方が良いです。

パッと見問題点

• 否応なしに登録処理が実行されようとしている
• 否応なしに削除処理が実行されようとしている
• 1ファイルコードで関数もないのに何度もDBconnect.phpを読み込んでいる
• 削除後に何もしてないので、「削除した後の一覧」が取得できない
• POST送信されなくても$_POSTが参照される

---

要は「適切に接続し、適切に閉じる」ことができれば、今回の問題は起きないと言えます。
なんとなく接続し、なんとなく閉じているために起きています。
きちんとメリハリをつけて、意図や意味を説明できるロジックを心がけましょう。

---

蛇足：
あと、DBconnect.phpのコードが提示されてないので確認してください。
PDOちゃんとエラーを出力されるような設定しているか。
PHPでデータベースに接続するときのまとめ#接続後にオプションを指定
きちんと設定されてないと、catchで捕捉してくれません。

Comments

[kotouharuto]

ご回答ありがとうございます。
確かにコードを見直してみると、自分でもおかしいと思う点がありました。
あやふやで理解しているところはきちんと理解する気をつけます。

失礼いたしました、ただいまDBconnect.php公開いたしましたのでご確認ください。

[m.ts10806]

いえ、DBconnect.phpはあくまで「ちゃんと設定されているかの確認」だけしてもらえればよかったです。
で、最初のinsertでエラー出てないということになりますかね。
となると、DBのテーブル定義も見直す必要があると思います。
todoカラムがnull OKになってませんか？

[kotouharuto]

そうだったのですね。
はい、todoカラムの方確認しましたらnull OKになっていました。こちらはNOTに指定しまって大丈夫ですか？

[m.ts10806]

ケースバイケースですが、「必須項目」ならNOT NULLに指定して、本来は登録前にバリデーションもした方が良いです。
ケースバイケースと言うか、「設計次第」ですね。
コード先行だと途中からどうこうが難しくなってロジックが意味をなさなくなるので、要件をしっかりかためて、設計してから取り掛かるべきと思います。
それがどんな簡易なアプリケーションでも。

[kotouharuto]

確かにそうですね。
一から作るときはきちんと設計した上で取り掛かろうと思います。
この度は非常に参考になるアドバイスをありがとうございました！

[m.ts10806]

目の前の問題解決だけを優先すると、結局今回のようにロジックのまずさから出るエラーに悩むことになりますよ。整理するなら、早い方が良いです。
「ここまで作ったからもったいない」は通じません。あくまでプログラムは書いた通りに動くので。
スクラップ＆ビルドも1つの技術的な選択肢ですよ。

[kotouharuto]

はい、宣言元からしっかりと確認してみます。

[m.ts10806]

あれ、伝わってなかった。

[kotouharuto]

いえ、十分伝わっています。
伝わったのですが、どうかえせばいいかわからなくなったので、文がおかしくなってしまいました。
失礼しました。

[m.ts10806]

では、あまりむずかしい言葉や表現を使わなくて良いので、自身が受け取ったものをそのまま表現してください。

[kotouharuto]

はい、了解しました。
ありがとうございます。

Answer 2

Call to a member function prepare() on null
を素直に読むと、null上でprepare()が実行されているという事です。

$stmh = $pdo->prepare('DELETE FROM todo WHERE id = :id'); //この文でエラーが出ています
と言うのがわかっているのであれば、$pdoに何かを代入しているところに問題があるのでしょう。

で、$pdoでコード内を検索してみると

$pdo = null;
で$pdoがnullになっていますから、問題は多分ここでしょうね。
コメントアウトすれば動くと思いますよ。

お勧めというかデバッグ方法

PHP xdebug に使用しているIDEの名前を加えて検索してみて、デバッグ環境を整えることを最優先にすることをお勧めします。

Comments

[kotouharuto]

ご回答ありがとうございました。
自分の記入漏れで申し訳ないのですが、$pdoでエラーが起きているというとこはわかっていて、$pdoの宣言元を確認したのですが、特に問題が確認できないので質問しました。
必要そうなので、そちらのコードの方も記載します、失礼いたしました。

わざわざありがとうございます！
そちらのことについても調べさせていただきます！

[tanat]

回答の
$pdo = null;
が原因なので、これをコメントアウトすればいけると思いますよ

[kotouharuto]

ありがとうございます。
そちらの方コメントアウトしたら、Call to a member function prepare() on nullは消えたのでひとまずベストアンサーにさせていただきます！

[tanat]

コメントとソースの印象からだと、
- コード1行1行の意味を把握しきれていない
- デバッグの仕方を知らない
- そのため、原因確認時に思い込みが入ってしまったり、勘に頼っている
のであと一歩のところで原因にたどり着けない様に見受けられました。（責めている訳ではありません）

例えば今回のケースだと、
- xdebugを使ってブレークポイントを全行に仕掛けて$pdoがどのタイミングでnullになってしまっているかを確認するとか
- var_dump(is_null($pdo))の様な検証コードを仕掛けてどのタイミングでnullになっているか確認する
という方法を知っていれば
「$pdoでエラーが起きているというとこはわかっていて、$pdoの宣言元を確認したのですが、特に問題が確認できないので質問しました。」
という結論にはならなかったと思います。

ですので、xdebugを導入するとともに、
`PHP デバッグ 方法`などで、デバッグの仕方を学ばれるのが近道かと思いますよ。

[kotouharuto]

ありがとうございます。
確かにそうでしたね、僕の方でも作ることを第一に考えてしまい確認作業を省いてしまっていました。
デバック方法を学んで出直してきます。