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

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

新規登録して質問してみよう
ただいま回答率
85.35%
HTML5

HTML5 (Hyper Text Markup Language、バージョン 5)は、マークアップ言語であるHTMLの第5版です。

phpMyAdmin

phpMyAdminはオープンソースで、PHPで書かれたウェブベースのMySQL管理ツールのことです。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

Q&A

1回答

1140閲覧

自動販売機の管理ページでエラーが表示されます

turtle410

総合スコア13

HTML5

HTML5 (Hyper Text Markup Language、バージョン 5)は、マークアップ言語であるHTMLの第5版です。

phpMyAdmin

phpMyAdminはオープンソースで、PHPで書かれたウェブベースのMySQL管理ツールのことです。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

0グッド

0クリップ

投稿2021/02/01 04:46

前提・実現したいこと

以下の要件を満たす販売商品の追加・変更を行う「管理ページ」を作成しています。
エラー箇所はHTML内の<?php foreach ($rows as $value) { ?>の部分です。
$rowsの代わりにどの変数を使えばいいかが分からなくなってしまいました。

・「ドリンク名」「値段」「在庫数」「公開ステータス」を入力し、商品を追加できる。
・商品を追加する場合、「商品画像」を指定してアップロードできる。
・追加した商品の一覧情報として、「商品画像」、「商品名」、「値段」、「在庫数」、「公開ステータス」のデータを一覧で表示する。
・商品一覧から指定ドリンクの在庫数を入力し、在庫数の変更ができる。
・商品一覧から指定ドリンクの公開ステータス「公開」あるいは「非公開」の変更ができる。
・商品の追加あるいは指定ドリンク情報(「在庫数」、「公開ステータス」)の変更が正常に完了した場合、完了のメッセージを表示する。
・商品を追加する場合、「商品名」「値段」、「在庫数」、「公開ステータス」「商品画像」のいずれかを指定していない場合、エラーメッセージを表示して、商品を追加できない。
・商品を追加する場合、「値段」、「在庫数」は、0以上の整数のみ可能とする。0以上の整数以外はエラーメッセージを表示して、商品を追加できない。
・商品を追加する場合、公開ステータスは「公開」あるいは「非公開」のみ可能とする。「公開」あるいは「非公開」以外はエラーメッセージを表示して、商品を追加できない。
・商品一覧から指定ドリンクの在庫数を変更する場合、0以上の整数のみ可能とする。0以上の整数以外はエラーメッセージを表示して、変更できない。

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

Notice: Undefined variable: rows in ~ Warning: Invalid argument supplied for foreach() in ~

該当のソースコード

PHP

1<?php //tool.phpは追加、変更、表示の役割をする 2$drink_master = []; 3$err_msg = []; // エラーメッセージの格納先 4$complete_msg = []; 5$drink_id = ''; //下からデータが飛んでくるので''で初期化する 6$stock = ''; 7$drink_name = ''; 8$price = ''; 9$status = ''; 10$new_img_filename = ''; // アップロードした新しい画像ファイル名 11$img_dir = './drink_picture/'; 12$now_date = date('Y-m-d H:i:s'); 13//改行して初期化の部分とデータベース接続の部分に分ける 14$host = 'localhost'; //ホスト名 15$username = '4455984'; //ユーザー名 16$password = '4455984'; //パスワード 17$dbname = '4455984'; //データベース名 18$charset = 'utf8'; 19$dsn = 'mysql:dbname='.$dbname.';host='.$host.';charset='.$charset; 20//date_default_timezone_set('Asia/Tokyo'); // タイムゾーン設定 21 22 23//アップロードできる「商品画像」のファイル形式は「JPEG」「PNG」のみ可能とする。(大文字小文字両方) 24//「JPEG」、「PNG」以外はエラーメッセージを表示して、商品を追加できない。 25// アップロード画像ファイルの保存 26if ($_SERVER['REQUEST_METHOD'] === 'POST') { 27 // HTTP POST でファイルがアップロードされたかどうかチェックする 28 //is_uploaded_file関数は、指定したファイルがアップロードされたファイルかどうかを確認する 29 if (is_uploaded_file($_FILES['new_img']['tmp_name']) === TRUE) { 30 //tmp_nameを指定することで一時的なファイルの名前が取得できる 31 // 画像の拡張子を取得する 32 $extension = pathinfo($_FILES['new_img']['name'], PATHINFO_EXTENSION); 33 // 指定の拡張子であるかどうかチェック 34 if ($extension === 'jpg' || $extension === 'jpeg') { 35 // 保存する新しいファイル名の生成(ユニークな値を設定する) 36 $new_img_filename = sha1(uniqid(mt_rand(), true)). '.' . $extension; 37 // 同名ファイルが存在するかどうかチェック 38 if (is_file($img_dir . $new_img_filename) !== TRUE) { 39 // アップロードされたファイルを指定ディレクトリに移動して保存 40 if (move_uploaded_file($_FILES['new_img']['tmp_name'], $img_dir . $new_img_filename) !== TRUE) { 41 $err_msg[] = 'ファイルアップロードに失敗しました'; 42 } 43 } else { 44 $err_msg[] = 'ファイルアップロードに失敗しました。再度お試しください。'; 45 } 46 } else { 47 $err_msg[] = 'ファイル形式が異なります。画像ファイルはJPEGのみ利用可能です。'; 48 } 49 } else { 50 $err_msg[] = 'ファイルを選択してください'; 51 } 52} 53 54try { 55 // データベースに接続する 56 $dbh = new PDO($dsn, $username, $password, array(PDO::MYSQL_ATTR_INIT_COMMAND => 'SET NAMES utf8mb4')); 57 $dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); 58 $dbh->setAttribute(PDO::ATTR_EMULATE_PREPARES, false); 59 echo 'データベースに接続しました'; 60 if ($_SERVER['REQUEST_METHOD'] === 'POST') { 61 if (isset($_POST['stock_button']) === TRUE) { 62 if (isset($_POST['drink_id']) === TRUE) { 63 $drink_id = trim($_POST['drink_id']); 64 } 65 if (isset($_POST['stock']) === TRUE) { 66 $stock = trim($_POST['stock']); 67 } 68 if ($drink_id === '') { 69 $err_msg[] = '商品が選択されていません'; 70 } else if (preg_match('/^[1-9][0-9]*$/', $drink_id) !== 1) { 71 $err_msg[] = '商品が正しくありません'; 72 } 73 if ($stock === '') { 74 $err_msg[] = '在庫数が入力されていません'; 75 } else if (preg_match('/^[0-9]+$/', $stock) !== 1) { 76 $err_msg[] = '在庫数は0以上の整数で入力してください'; 77 } 78 if (count($err_msg) === 0) { 79 try { 80 $sql = 'UPDATE drink_stock 81 SET stock = ?, 82 update_datetime = ? 83 WHERE drink_id = ?'; 84 $stmt = $dbh->prepare($sql); 85 // SQL文のプレースホルダに値をバインドする 86 $stmt->bindValue(1, $stock, PDO::PARAM_INT); //ここをPARAM_STRではなくてINTにかえる 87 $stmt->bindValue(2, $now_date, PDO::PARAM_STR); //日付は文字列なのでSTR 88 $stmt->bindValue(3, $drink_id, PDO::PARAM_INT); 89 // SQLを実行する 90 $stmt->execute(); 91 } catch (PDOException $e) { 92 $err_msg[] = 'データベース処理でエラーが発生しました。理由:' . $e->getMessage(); 93 } 94 } 95 } 96 } 97} catch (PDOException $e) { 98 99} 100 101 //「商品追加機能」「表示ステータス切り替え機能」の部分を追加する 102 //入力チェック、sql文、画像のチェックをサーバーにアップロードの順 103 //エラーがないならsql文を実行して 104 //画像をアップロードする 105 // if (isset($_POST['insert']) === TRUE) { 106 // } 107?> 108<!DOCTYPE html> 109<html lang="ja"> 110 <head> 111 <meta charset="UTF-8"> 112 <title>自動販売機の管理画面</title> 113 <style> 114 table,tr,th,td { 115 border:solid 1px; 116 } 117 </style> 118 </head> 119 <body> 120 <h1>自動販売機管理ツール</h1> 121<?php foreach ($err_msg as $value) { ?> 122 <p><?php echo $value;?></p> 123<?php }?> 124<?php foreach ($complete_msg as $value) { ?> 125 <p><?php echo $value;?></p> 126<?php }?> 127 <section> 128 <h2>新規商品追加</h2> 129 <form method="post" enctype="multipart/form-data"> 130 <p><label>名前<input type="text" name="drink_name"></label></p> 131 <p><label>値段<input type="text" name="price"></label></p> 132 <p><label>個数<input type="text" name="stock"></label></p> 133 <p><input type="file" name="new_img"></p> 134 <select name="new_status"><br> 135 <option value="0">非公開</option> 136 <option value="1">公開</option> 137 </select><br> 138 <input type="hidden" name="sql_kind" value="insert"> 139 <p><input type="submit" name="insert" value="■□■□■商品追加■□■□■"></p> 140 </form> 141 </section> 142 <section> 143 <h2>商品情報変更</h2> 144 <caption>商品一覧</caption> 145 <table> 146 <tr> 147 <th>商品画像</th> 148 <th>商品名</th> 149 <th>価格</th> 150 <th>在庫数</th> 151 <th>ステータス</th> 152 </tr> 153<?php foreach ($rows as $value) { ?> //エラーの箇所です 154 <tr class="status_false"> 155 <td><img src="<?php print htmlspecialchars($img_dir.$value['img'], ENT_QUOTES, 'UTF-8'); ?>" width=100 height=150></td> 156 <td><?php print htmlspecialchars($value['drink_name'], ENT_QUOTES, 'UTF-8'); ?></td> 157 <td><?php print htmlspecialchars($value['price'], ENT_QUOTES, 'UTF-8'); ?></td> 158 <td> <!--HTMLでは、=の前後をくっつける --> 159 <form method="post"> 160 <input type="hidden" name="drink_id" value="<?php print htmlspecialchars($value['drink_id'], ENT_QUOTES, 'UTF-8'); ?>"> 161 <input type="text" name="stock" value="<?php print htmlspecialchars($value['stock'], ENT_QUOTES, 'UTF-8'); ?>">162 <input type="submit" name="stock_button" value="変更"> 163 </form> 164 </td> 165 <td> 166 <form method="post"> 167 <input type="hidden" name="drink_id" value="<?php print htmlspecialchars($value['drink_id'], ENT_QUOTES, 'UTF-8'); ?>"> 168<?php if (htmlspecialchars($value['status'], ENT_QUOTES, 'UTF-8') === '0') { ?> 169 <input type="submit" name="status_button" value="非公開→公開"> 170 <input type="hidden" name="status" value='1' > 171 <style type="text/css"> 172 .status_false{ 173 background-color:#ccc; 174 } 175 </style> 176<?php } else {?> 177 <input type="submit" name="status_button" value="公開→非公開"> 178 <input type="hidden" name="status" value='0' > 179<?php }?> 180 </form> 181 </td> 182 </tr> 183<?php }?> 184 </table> 185 </section> 186 </body> 187</html> 188

試したこと

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

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

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

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

m.ts10806

2021/02/01 05:03

phpMyAdminは本件とはどう関係するのでしょうか。 見た感じ、ほぼ無関係です。 MySQLもしくはMariaDBなら関係しますが
退会済みユーザー

退会済みユーザー

2021/02/01 05:18 編集

他人のアドバイス、聞いていないのか届いていないのか、ほんと残念だよ。わからないことやしらないことやできないことは恥ずかしいことじゃないので、せっかく教える気でリアクションを待っていてもBAつけてコメントもなしにさっさと終了させちゃうと、もう何も言わなくてもわかってくれたのねって解釈してしまう。で、類似の質問がこうやって投稿されちゃうと、呆れるわけ。わかってないじゃん、と。
guest

回答1

0

$rowsの代わりにどの変数を使えばいいかが分からなくなってしまいました。

代わりも何も、そもそも定義されていない変数を使おうとしてしまっているという現実は認識されたほうが良いかと思います。

なので、「適切に定義された変数を利用してください」
というのが応えになります。

実装ざっと見た感じ、当該foreachのところに何を表示させたいのか不明です。
少なくともデータ取得のコードが書かれているようには見えません。
「そこに表示したい情報の取得処理を書いてください」と言ったほうがいいかな。

投稿2021/02/01 04:49

編集2021/02/01 04:53
m.ts10806

総合スコア80875

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

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

ただいまの回答率
85.35%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問