Q&A
データベースへの接続はどうやって点検していますか？ いきなり込み入った事例を動かそうとせず、ミニマルな、データベース接続のみのコードを書いて動作確認をとってから動かしましょう。
前提・実現したいこと
「気がつけばプロ並みPHP改訂版」
という書籍でウェブサイトを制作してるのですが
下記のコードで
【pro_add_check.php】
で値を入力しても
【pro_add_done.php】
でｔｒｙ文が実行されず、catch文が実行されます。
何度も見返しうまく行かないので、サンプルコードをコピペしたのですが
catch文が実行されます。
稚拙な質問と思いますがアドバイスいただければ幸いです。
該当のソースコード
【pro_add.php】 <!DOCTYPE html> <html> <head> <meta charset="UTF-8"> <title>ろくまる農園</title> </head> <body> 商品追加<br /> <br /> <form method="post" action="pro_add_check.php"> 商品名を入力してください。<br /> <input type="text" name="name" style="width:200px"><br /> 価格を入力してください。<br /> <input type="text" name="price" style="width:50px"><br /> <br /> <input type="button" onclick="history.back()" value="戻る"> <input type="submit" value="ＯＫ"> </form> </body> </html>
【pro_add_check.php】 <!DOCTYPE html> <html> <head> <meta charset="UTF-8"> <title>ろくまる農園</title> </head> <body> <?php $pro_name=$_POST['name']; $pro_price=$_POST['price']; $pro_name=htmlspecialchars($pro_name,ENT_QUOTES,'UTF-8'); $pro_price=htmlspecialchars($pro_price,ENT_QUOTES,'UTF-8'); if($pro_name=='') { print '商品名が入力されていません。<br />'; } else { print '商品名:'; print $pro_name; print '<br />'; } if(preg_match('/\A[0-9]+\z/',$pro_price)==0) { print '価格をきちんと入力してください。<br />'; } else { print '価格:'; print $pro_price; print '円<br />'; } if($pro_name=='' || preg_match('/\A[0-9]+\z/',$pro_price)==0) { print '<form>'; print '<input type="button" onclick="history.back()" value="戻る">'; print '</form>'; } else { print '上記の商品を追加します。<br />'; print '<form method="post" action="pro_add_done.php">'; print '<input type="hidden" name="name" value="'.$pro_name.'">'; print '<input type="hidden" name="price" value="'.$pro_price.'">'; print '<br />'; print '<input type="button" onclick="history.back()" value="戻る">'; print '<input type="submit" value="ＯＫ">'; print '</form>'; } ?> </body> </html> コード
【pro_add_done.php】 <!DOCTYPE html> <html> <head> <meta charset="UTF-8"> <title>ろくまる農園</title> </head> <body> <?php try { $pro_name=$_POST['name']; $pro_price=$_POST['price']; $pro_name=htmlspecialchars($pro_name,ENT_QUOTES,'UTF-8'); $pro_price=htmlspecialchars($pro_price,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='INSERT INTO mst_product (name,price) VALUES (?,?)'; $stmt=$dbh->prepare($sql); $data[]=$pro_name; $data[]=$pro_price; $stmt->execute($data); $dbh=null; print $pro_name; print 'を追加しました。<br />'; } catch(Exception $e) { print'ただいま障害により大変ご迷惑をお掛けしております。'; exit(); } ?> <a href="pro_list.php">戻る</a> </body> </html>
試したこと
何度もコードの見直し、xamppの再起動等をして確認しました。
ここに問題に対して試したことを記載してください。
補足情報（FW/ツールのバージョンなど）
環境はxamppでエディターはVSCodeを使用しています。
catch文が実行されるってことはなんか例外でてるんじゃないですか？
回答3件
0
ミニマルなコードを先に試して動作確認できたら、
本来試したいコードを動かしてみてください。
ミニマルなコードの事例として、例えば
PHPでデータベースに接続するときのまとめ - Qiita
にある「基本コーディング」あたりは良い事例です。
データベース接続先を自分の試す環境に合わせて書き換えて、
XAMPPのapache httpdのerror_log(例えばC:\xampp\apache\logsにあります)もチェックしてみてください。
投稿2019/08/23 08:43
退会済みユーザー
総合スコア0
0
ベストアンサー
まずは下記を
php
1catch(Exception $e) 2{ 3 print'ただいま障害により大変ご迷惑をお掛けしております。'; 4 exit(); 5} 6
下記のように変更してエラー確認から。
PHP
1catch(PDOException $e) 2{ 3 die($e->getMessage()); 4} 5
学習期でExceptionを含めてエラー表示をしないのは問題解決を遅らせるだけです。
実務でも開発時は表示し、本番リリース時に外すということをやっています（または画面に表示しないまでもログに吐き出す）
で、何か出てきたらそこからが調査開始です。
エラーメッセージの読み方と対処, 検索や質問の原則
投稿2019/08/23 12:09
編集2019/08/23 12:19
総合スコア80850
返信が遅れ申し訳ございません。
恥ずかしながらエラーの出し方が分かりませんでした。
教えて頂き、解決の糸口をとらえることができました。
本当にありがとうございます！
0
環境をmanpに変え
パスワードの所を空文字ではなく
rootを入れたら
画面遷移できました。
$password='root';
真摯に返答いただいた方には感謝いたします(T_T)
投稿2019/08/25 10:58
総合スコア5
あなたの回答
tips
プレビュー
