質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

89.10%

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

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 117

kotouharuto

score 23

実現したいこと

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); //1レコード取り出し
            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が悪いのはわかっているのですが、僕が確認したところだと問題はなかったです。

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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

+3

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の名前を加えて検索してみて、デバッグ環境を整えることを最優先にすることをお勧めします。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/07/01 21:10

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

    キャンセル

  • 2020/07/01 21:38

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

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

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

    キャンセル

  • 2020/07/02 07:02

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

    キャンセル

+3

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

パッと見問題点

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

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


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

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/07/02 07:05

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

    キャンセル

  • 2020/07/02 07:51

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

    キャンセル

  • 2020/07/02 14:43

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 89.10%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる