Q&A
デバッグはされましたか？
せめてどの分岐を通ってるかくらいは調べましょう。
実現したい実装：「カートに追加」ボタンを押すと、「客のuser_id・客が購入した商品情報（item_id）・購入する個数（amount）・日付（createdate）・更新日（updatedate）」がカート用のテーブルに（beertabi_cart）に追加される。
発生している不具合：エラーは表示されないが、ボタンを押しても、上記のデータがテーブルに反映されない。
分からないところ：なぜ反映されないのか、何が間違えて正しく機能していないのか分からない。
※セッションは最近学習し始めました、まだ理解が未熟のためお手柔らかに教えて頂けると幸いです。
<?php session_start(); // セッション開始 $host = ''; $username = ''; $password = ''; $dbname = ''; $charset = 'utf8'; // MySQL用のDSN文字列 $dsn = 'mysql:dbname=' . $dbname . ';host=' . $host . ';charset=' . $charset; // 完成MSG $complete = []; //エラー配列 $error = []; //空配列 $data = []; try { // データベースに接続 $dbh = new PDO($dsn, $username, $password, array(PDO::MYSQL_ATTR_INIT_COMMAND => 'SET NAMES utf8mb4')); $dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); $dbh->setAttribute(PDO::ATTR_EMULATE_PREPARES, false); // item_id (カートに追加されたもの)が存在するか確認 if(isset($_POST['item_id']) === TRUE && isset($_SESSION['user_id']) === TRUE){ // 変数に入れる $item_id = $_POST['item_id']; $user_id = $_SESSION['user_id']; $sql = 'SELECT * FROM beertabi_cart INNER JOIN beertabi_user ON beertabi_cart.user_id = beertabi_user.user_id WHERE beertabi_user.user_id = ? AND beertabi_cart.item_id = ?'; $stmt = $dbh->prepare($sql); $stmt->bindValue(1, $user_id, PDO::PARAM_INT); $stmt->bindValue(2, $item_id, PDO::PARAM_INT); $stmt->execute(); if($stmt->fetch() === FALSE){ // カートに中身がない場合：一度目の購入 // 注目：'amount'に1を入れる-----一回めの購入（カウント） $sql = 'INSERT INTO beertabi_cart (user_id, item_id, amount, createdate, updatedate) VALUES (?, ?, 1, NOW(), NOW())'; } else { // カートに中身が既にある場合：一度目以降の購入 // 注目：'amount'に+1する $sql = 'UPDATE beertabi_cart SET amount = amount +1, updatedate = NOW() WHERE user_id = ? AND item_id = ?'; } $stmt = $dbh->prepare($sql); $stmt->bindValue(1, $user_id, PDO::PARAM_INT); $stmt->bindValue(2, $item_id, PDO::PARAM_INT); $stmt->execute(); } //ステータス：２（公開）のデータを取得 $sql = 'SELECT * FROM items JOIN items_stock ON items.item_id = items_stock.item_id WHERE items.status = 2 AND items_stock.stock >= 1 ORDER BY items.createdate DESC'; // SQL文実行準備 $stmt = $dbh->prepare($sql); // 実行 $stmt->execute(); // 選択したレコード取得 $data = $stmt->fetchAll(); } catch (PDOException $e){ $error[] = 'データベース接続に失敗しました。理由：'.$e->getMessage(); } ?> <!DOCTYPE html> <html> <head> <!--required-meta--> <meta charset="utf-8"> <meta name="viewport" content="width=device-width, initial-scale=1.0"> <!--optional-meta--> <meta name="description" content="ビア旅の商品管理画面です"> <!--Bootstrap--> <link rel="stylesheet" href="https://stackpath.bootstrapcdn.com/bootstrap/4.2.1/css/bootstrap.min.css" integrity="sha384-GJzZqFGwb1QTTN6wy59ffF1BuGJpLSa9DkKMp0DgiMDm4iYMj70gZWKYbI706tWS" crossorigin="anonymous"> <!--link--> <link rel="stylesheet" href="css/style.css"> <!--favicon--> <link rel="shortcut icon" href="img/favicon.png"> <link rel="icon" type="image/png" sizes="16x16" href="img/favicon.png"> <link rel="manifest" href="img/favicon.png"> <!--fontawsome--> <link href="https://use.fontawesome.com/releases/v5.6.1/css/all.css" rel="stylesheet"> <!--google_fonts--> <link href="https://fonts.googleapis.com/css2?family=M+PLUS+Rounded+1c&display=swap" rel="stylesheet"> <title>ビア旅｜お客様ページ</title> </head> <body> <header class="fixed-top"> <div id="header_inside" class="d-flex justify-content-between flex-wrap py-2"> <!--ロゴ--> <img class="img-fluid col-5 col-xl-2 col-lg-3 col-md-4 pl-0" src="img/logo.png"> <!--ユーザーネームとカート--> <div class="user_button_box p-lg-3 p-2"> <!--ユーザー名表示--> <p>こんにちは、<br class="fold_rwd"><?php if(isset($_SESSION['user_name'])){ print $_SESSION['user_name']; } ?>さん！</p> <!--カートボタン--> <a class="px-2" id="cart_btn" href="cart.php"><i class="fas fa-cart-arrow-down fa-2x"></i></a> <!--ログアウト--> <form method="post" action="logout.php"> <button id="logout_button" type="submit">ログアウト</button> </form> </div> </div> </header> <div id="wrapper" class="container-fluid"> <h5 class="font-weight-bold py-lg-3 pt-4 py-xs-3 pb-3">本日のおススメビール</h5> <!--3つ最新ビールを表示--> <div class="reccomend_box text-center mx-auto d-flex flex-md-row flex-column"> <?php $i = 0; ?> <?php foreach ($data as $value) { ?> <!--レコメンドビールのカード--> <?php if($i >= 3){ break; } ?> <div class="reccomend bg-white p-3 m-2"> <div class="beer_info d-flex flex-column"> <h5>商品名：<?php print htmlspecialchars($value['name'],ENT_QUOTES); ?></h5> <h5>産地：<?php print $value['area']; ?></h5> <img class="img-fluid" src="img_box/<?php print $value['img']; ?>"></img> </div> <div class="d-flex flex-row justify-content-between"> <p>￥<?php print $value['price']; ?></p> <!--購入ボタン--> <?php if(isset($_SESSION['user_name'])){ ?> <form method="post"> <button id="button" type="submit" name="item_id" value="<?php print $value['item_id']; ?>">カートに追加</button> </form> <?php } ?> </div> </div> <?php $i++; } ?> </div> <div class="py-5 my-4"></div> <div class="my-3 return_to_search"> <a id="button" href="search_beer.php">商品一覧・検索ページへ</a> </div> </div> <!--jQuery for Bootstrap--> <script src="https://code.jquery.com/jquery-3.3.1.slim.min.js" integrity="sha384-q8i/X+965DzO0rT7abK41JStQIAqVgRVzpbzo5smXKp4YfRvH+8abtTE1Pi6jizo" crossorigin="anonymous"></script> <script src="https://cdnjs.cloudflare.com/ajax/libs/popper.js/1.14.6/umd/popper.min.js" integrity="sha384-wHAiFfRlMFy6i5SRaxvfOCifBUQy1xHdJ/yoi7FRNXMRBu5WHdZYu1hA6ZOblgut" crossorigin="anonymous"></script> <script src="https://stackpath.bootstrapcdn.com/bootstrap/4.2.1/js/bootstrap.min.js" integrity="sha384-B0UglyR+jN6CkvvICOB2joaf5I4l3gm9GU6Hc1og6Ls7i6U/mkkaduKaBhlAXv9k" crossorigin="anonymous"></script> </body> </html>
デバック自分なりにしましたが（正直PHPでのデバックの方法があまり把握できていない）、エラーが出てくれず不具合だけ発生し困っているためこちらに投稿いたしました。PHPもまだ苦手意識を持っているため、「データの流れ」「分岐？」を理解しきれていない面があります、しかし、第三者さまからご指摘頂くことで初めて気づく面もあるかと思いこちらにそのまま投稿しました。
エラーは出す必要ないです。
「想定のタイミングで想定の場所を通っているか」の確認です。
echo __LINE__."<br />\n";
↑これを分岐と言う分岐に入れるだけでもいいですよ。通った行数が出力されます。
なるほどですね。分岐をたどる方法を教えて頂きありがとうございます！
早速今から、教えて頂いた方法で分岐をたどってみようと思います。
可能なら、その時点での変数が想定通りの値かどうか、var_dump()も併用してください。
デバッグとはあくまでロジックを追って問題解決の糸口とする作業であって、エラーを起こすことではないです（あえてエラーにするという手法もありますが、それはあくまで分かっている前提）
そうですよね。デバック＝エラーを起こすではないのは理解していたのですが、
「if文を用いて条件と合わなければ、作成したエラー用の配列にいれて、問題を表示」
この方法しか恥ずかしながら使用したことがありませんでした。var_dump()の使い方も学習した上で、もう一度デバックをしてみます。
回答1件
0
自己解決
原因：ログインのページ（login.php）にて、
$result = $stmt->fetchAll(); // セッション変数からuser_id取得 if (isset($_SESSION['user_id'])) { $_SESSION['user_id'] = $result['user_id']; } if(count($result) === 1){ $_SESSION['user_name'] = $_POST['user_name']; header('Location: ./user_page.php'); exit; }
・ログインしていないと、処理されない状況になっていた
・count()===1, だと安全性が低かった
・fetchAll()で取ってきたデータはあくまでも配列で返ってくる＝
一つしかデータが返ってこないとしても[0]をつける必要性がでる、認識ができていなかった
解決方法：上記を下記のように変更することで解決しました
$result = $stmt->fetchAll(); if(count($result) > 0){ $_SESSION['user_id'] = $result[0]['user_id']; $_SESSION['user_name'] = $_POST['user_name']; header('Location: ./user_page.php'); exit; }
問題を発見した方法：
search_beer.php(投稿したページ)のtry下に、下記を入れることで
$user_id = null であることを確認
if($_SERVER['REQUEST_METHOD']===TRUE){ var_dump($_POST,$_SESSION); exit; }
投稿2021/07/20 10:44
総合スコア3
あなたの回答
tips
プレビュー
