PHPの自動販売機システム

Question

前提・実現したいこと

PHPで自動販売機を制作しています。ユーザーがドリンクを購入した結果情報を表示する「購入結果ページ」（ result.php ）で、isset関数を使ってif文を修正したいです。現在エラーが3か所あります。

流れは以下のようになります。
「管理ページ(tool.php)」で、 販売する商品の追加をする。
「購入ページ(index.php)」で、購入する商品を選択し、金額を入力して購入する。
「購入結果ページ(result.php)」で、購入した結果を表示する。

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

Notice: Undefined variable: img in /home/ec2-user/environment/htdocs/php/21/result.php on line 120 Call Stack #TimeMemoryFunctionLocation 10.0003245528{main}( ).../result.php:0 ">
Notice: Undefined variable: drink_name in /home/ec2-user/environment/htdocs/php/21/result.php on line 121
Notice: Undefined variable: change in /home/ec2-user/environment/htdocs/php/21/result.php on line 125

該当のソースコード

<?php 
    $host = 'localhost'; //ホスト名
    $username = ''; //ユーザー名
    $password = ''; //パスワード
    $dbname = ''; //データベース名
    $link = mysqli_connect($host, $username, $password, $dbname);
    $err_msg = [];
    $drink_id = '';
// (isset関数を使う)
    if ($link) {
        // 文字化け防止
        mysqli_set_charset($link, 'utf8');    

        //POST情報が入っていたら   
        if ($_SERVER['REQUEST_METHOD'] === 'POST') {        
            // drink_idを取得
            $drink_id = $_POST['drink_id'];
            // 商品情報を取得
            $sql = "SELECT drink_master.drink_id, img, drink_name, price, status, stock
                    FROM drink_master
                    JOIN stock
                    ON drink_master.drink_id = stock.drink_id
                    WHERE drink_master.drink_id = '$drink_id'";

                //クエリ実行 
                if ($result = mysqli_query($link, $sql)) {
                    // 情報を各変数へ代入
                    $row = mysqli_fetch_array($result);
                    $price = (int)$row['price'];
                    $img = $row['img'];
                    $drink_name = $row['drink_name'];
                    $stock_number = (int)$row['stock'];
                    $open_status = (int)$row['status'];
                    $money = $_POST['money'];
                    
                    // エラーチェック
                    // 商品未選択
                    if (count($drink_id) === 0) {
                        $err_msg[] = '商品を選択してください';
                    }
                    // 金額入ってない
                    if ($money === '') {
                       $err_msg[] = 'お金が投入されていません';

                    //  金額が足りない
                    }
                    else if ($money < $price) {
                        $err_msg[] = 'お金が足りません';
                    // 整数以外が入力
                    }
                    else if(preg_match('/^[0-9]+$/', $money) !== 1) {
                        $err_msg[] = '投入金額は0以上の整数を入力してください';
                    }
                    // 購入ページを開いた後に在庫0になった
                    if ($stock_number <= 0) {
                        $err_msg[] = '在庫が切れました';
                    }
                    // 購入ページを開いた後にステータス0になった
                    if ($open_status === 0) {
                        $err_msg[] = '公開出来なくなりました・・・・';
                    }

                    // エラーがなければ
                    if (count($err_msg) === 0) {
                        // オートコミット
                        mysqli_autocommit($link, false);
                        // 残金計算
                        $change = $money - $price;
                        // 在庫1本減らす
                        $sql = "UPDATE stock
                                SET stock = stock - 1
                                WHERE drink_id = '$drink_id'";
                        // クエリの実行
                        if (mysqli_query($link, $sql) === TRUE) {
                            // 購入履歴追記
                            $sql = "INSERT INTO drink_history
                                    (drink_id, purchase_date)
                                    VALUES ($drink_id, now())";
                            // クエリが実行出来なかったら
                            if (mysqli_query($link, $sql) !== TRUE) {
                            // エラー
                            $err_msg[] = 'drink_history: UPDATEエラー：' . $sql;
                            }
                        // エラー
                        } 
                        else {
                            $err_msg[] = 'UPDATE drink_stock: UPDATEエラー：' . $sql;
                        }

                        // トランザクション成否
                        if (count($err_msg) === 0) {
                            mysqli_commit($link);
                        }
                        else {
                            mysqli_rollback($link);
                        }
                    }
                }
        }
    }
    else {
        $err_msg[] = 'connectエラー' . $link;
    }
?>

<!DOCTYPE html>
<html lang="ja">
    <head>
        <meta charset="UTF-8">
        <title>自動販売機 購入結果</title>
    </head>
    <body>
        <h1>購入結果</h1>
        <?php foreach ($err_msg as $value) { ?>
        <?php echo $value . "<br>";  ?>
        <?php } ?>
        <?php if (count($err_msg) === 0) { ?>
        <p>がしゃん！
           <img src="./drink_picture/<?php echo $img; ?>"><br> //1個目のエラー
           <?php echo $drink_name; ?>　//2個目のエラー
           が買えました！
        </p>
        <p>おつりは
           <?php echo $change; ?> //3個目のエラー
           円です！
        </p>
        <?php } ?>
        <a href="index.php">戻る</a>
    </body>
</html>

試したこと

ここに問題に対して試したことを記載してください。

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

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

Comments

[m.ts10806]

エラーとNoticeは厳密には違うものです

Answer 1

$imgはDBから取り出したデータとなっています
当該行が複数あっても上書きされて最後のデータしか取れないようになっています。
それとは別にmysqlの実行がされないときに$imgは定義されないので
未定義の注意がでるわけです。
noticeをはずすだけならソースの先頭の方で$img="";
のように空文字で初期化しておけばよいでしょう