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

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

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

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

Q&A

解決済

1回答

747閲覧

データーベース UPDATEについて

koba_kazu

総合スコア4

PHP

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

0グッド

0クリップ

投稿2020/07/08 07:27

php初心者です。

修正ページ作成中で、始めにコード・商品名・価格の3カラムだけで修正ページを作成した場合は上手くできてましたが、その後に画像も含めた修正ページを作成するとデータベースに反映されなくなりました。特にエラーも表示されずに何が原因かわからなくなりました。
『修正チェック画面』と『修正完了ページ』のソースコードを記載させていただきます。
わからなくて困っています。よろしくお願い致します。

コード 『修正チェック画面』 <!DOCTYPE html> <html lang="ja"> <head> <meta charset="UTF-8"> <title>gi</title> </head> <body> <?php $pro_code=$_POST['code']; $pro_name=$_POST['name']; $pro_price=$_POST['price']; $pro_gazou_name_old=$_POST['gazou_name_old']; $pro_gazou=$_FILES['gazou']; $pro_name=htmlspecialchars($pro_name,ENT_QUOTES,"UTF-8"); $pro_price=htmlspecialchars($pro_price,ENT_QUOTES,"UTF-8"); if($pro_name=='') { print '商品名が入力されていません。'; print '<br />'; } else { print '商品名:'; print $pro_name; print '<br />'; } if(preg_match('/[A-Za-z]/',$pro_price)) { print '価格が入力されていません。'; print '<br />'; } else { print '価格:'; print $pro_price; print '円 <br />'; } if($pro_gazou['size']>0) { if($pro_gazou['size']>1000000) { print '画像が大き過ぎます。'; } else { move_uploaded_file($pro_gazou['tmp_name'],'./gazou/'.$pro_gazou['name']); print '<img src="./gazou/'.$pro_gazou['name'].'">'; print '<br />'; } } if($pro_name==''||preg_match('/[A-Za-z]/',$pro_price)||$pro_gazou['size']>1000000) { print 'きちんと入力されていません。'; print '<form>'; print '<input type="submit" onclick="history.back()">'; print '</from>'; } else { print '<form method="post" action="pro_edit_done.php">'; print '<input type="hidden" name="name" value="'.$pro_name.'">'; print '<input type="hidden" name="price" value="'.$pro_price.'">'; print '<input type="hidden" name="code" value="'.$pro_code.'">'; print '<input type="hidden" name="gazou_name_old" value="'.$pro_gazou_name_old.'">'; print '<input type="hidden" name="gazou" value="'.$pro_gazou['name'].'">'; print '<input type="button" onclick="history.back()" value="戻る">'; print '<input type="submit" value="OK">'; print '</form>'; } ?> </body> </html>
コード 『修正完了ページ』 <!DOCTYPE html> <html lang="ja"> <head> <meta charset="UTF-8"> <title>ginpei</title> </head> <body> <?php try { $pro_name=$_POST['name']; $pro_price=$_POST['price']; $pro_code=$_POST['code']; $pro_gazou_name_old=$_POST['gazou_name_old']; $pro_gazou_name=$_POST['gazou']; $pro_name=htmlspecialchars($pro_name,ENT_QUOTES,"UTF-8"); $pro_price=htmlspecialchars($pro_price,ENT_QUOTES,"UTF-8"); $pro_code=htmlspecialchars($pro_code,ENT_QUOTES,"UTF-8"); $dsn='mysql:dbname=shop;host=localhost;charset=utf8'; $user='root'; $password=''; $dbh=new PDO($dsn,$user,$password); $dbh->setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION); $sql='UPDATE mst_product SET name=?,price=?,gazou=? WHERE code=?'; $stmt=$dbh->prepare($sql); $data[]=$pro_name; $data[]=$pro_price; $data[]=$pro_code; $data[]=$pro_gazou_name; $stmt->execute($data); if($pro_gazou_name_old!=$pro_gazou_name) { if($pro_gazou_name_old!='') { unlink("./gazou/$pro_gazou_name_old"); } } $dbh=null; } catch(Exception $e) { print 'ただいま障害により大変ご迷惑をおかけしております。'; exit(); } ?> <p>修正致しました。</p> <p><?php print $pro_name; ?></p> <p><?php print $pro_price.'円'; ?></p> <p><?php print $pro_code; ?></p> <p><?php print '<img src="./gazou/'.$pro_gazou_name.'">';?></p> <?php ?> </p> <form action="pro_list.php"> <input type="submit" value="一覧に戻る"> </form> </body> </html>

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

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

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

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

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

m.ts10806

2020/07/08 07:53

デバッグしてください。
m.ts10806

2020/07/08 07:57

あと、何を参考にしているか知らないですが、まずい実装がかなりあるので、その参考先はやめたほうがいいです。
koba_kazu

2020/07/08 08:38

ありがとうございます。 学習し直します。
m.ts10806

2020/07/08 09:15 編集

「初心者」であれば「し直す」にはならないと思います。 根本的に学習方法を見直してください。 目の前の問題だけ追っていてもアプリケーションは出来上がりません。
koba_kazu

2020/07/08 14:22

かしこまりました。 貴重なご意見ありがとうございます!
guest

回答1

0

ベストアンサー

$_FILES['gazou']と$_POST['gazou']の使い分けがおかしいのでは?

投稿2020/07/08 08:38

yambejp

総合スコア114810

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

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

koba_kazu

2020/07/08 08:40

ありがとうございます! 確認します!
yambejp

2020/07/08 08:47

修正チェックページから修正完了ページにhiddenデータをpostしてますが それではチェックとしては不十分です。 原則チェックページではすべてセッションに保持して、 完了ページはpostデータを信用してはいけません
koba_kazu

2020/07/08 14:38

なるほど! ありがとうございます! セッションについて調べて実装してみます。 ご回答ありがとうございます。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問