Q&A
CREATE TABLE もコードブロックに入れたほうがよいです。
PHPとMySQLをもちいて入力→確認→完了→テーブルの順に表示されるデータベースをつくっています。
入力から確認まではPOSTとsessionを用いて問題なくデータが引き継がれているのですが、確認画面からPOSTで送られたデータが完了画面でMySQLに保存されていないです。
以下が該当箇所のコードです。
個人的には、完了画面の//データの追加部分が怪しいと思っているのですが具体的にどこを修正すればよいかわかりません。
色々な本やサイトを参考にしているのでつぎはぎで汚いプログラムです。
初歩的なミスであったら申し訳ございません。
ご教授いただけたらうれしいです。
よろしくお願いいたします。
回答1件
0
ベストアンサー
INSET
そんなものはありません。
「INSERT」です。
INSERT部分もtry-catch囲ってないとエラーが捕捉出来ませんので
DB関連の処理はまとめてtry-catchしておきましょう。
php
1try { 2 $dbh =new PDO($dsn, $user, $pass); 3 //データ追加 4 $sql="INSERT INTO user_id (age,name,number) VALUES(".$age.",".$name.",".$number.")"; 5 $stmt=$dbh ->prepare($sql); 6 $stmt->execute(); 7}catch(PDOException $e){ 8 echo $e -> getMessage(); 9 exit(); 10}
あと、折角prepareしているのですから、bindValue使ってきちんとSQLインジェクション対策してください。
php
1try { 2 $dbh =new PDO($dsn, $user, $pass); 3 //データ追加 4 $sql="INSERT INTO user_id (age,name,number) VALUES(:age,:name,:number)"; 5 $stmt=$dbh ->prepare($sql); 6 $stmt->bindValue(':age', $age, PDO::PARAM_INT); 7 $stmt->bindValue(':name', $name, PDO::PARAM_STR); 8 $stmt->bindValue(':number', $number, PDO::PARAM_INT); 9 $stmt->execute(); 10}catch(PDOException $e){ 11 echo $e -> getMessage(); 12 exit(); 13}
蛇足：
テーブル名：user_id って本当ですかね？通常はカラム名で使われるものですけど。。。
<html lang="en">
日本語で記述しているのにenって・・。
投稿2019/02/07 07:43
編集2019/02/07 07:52
総合スコア80850
INSERT INFOも
↓
INSERT INTOかも。。
おお・・・
スペルミスからlangまで、ほんと無茶苦茶なプログラムで申し訳ないです。
まだMySQLに保存されないのでもう少しエラーを探してみます。
ありがとうございます
execute()のところまで囲えばエラーも拾えると思うのでそこを確認してください。
user_idってテーブル名本当ですよね？テーブル名から内容は想像できません。user_idsとかでid管理しているテーブルなら分かるんですけどそうじゃないようですし。
もし完全に解決を望むのでしたらテーブル定義も含めて質問にご提示ください。
テーブルの画像追加させていただきました。
恐らく僕が参考サイトと同じように変な名前でテーブル作成してしまったのだと思います。
ご迷惑をおかけいたします、よろしくお願いいたします。
追記ありがとうございます。
ただ、そこから「構造」に入ってもらわないとわからないですね。
「エクスポート」で出力したcreate table文を提示された方がこちらも再現確認しやすくなります。
が、照合順序がこれだと全角文字が入ったときに文字化けします。
追加させていただきました。もし、間違っていたら再度追加するのでお手数をおかけしますがコメントよろしくお願いいたします。
「再現確認のため」なので画像ではなくテキストで貼り付けて欲しいです・・。
まあでも
$_SESSION[age]
↑これだと「ageという定数に入っている情報のキー」になるので
そういう定数を設定していないのであれば
$_SESSION["age"]
ですよね。
申し訳ございません。
修正します
あと、session_start();はブラウザに何かを出力する前にコールするようにしましょう。
ありがとうございます。解決しました。
エラーの部分が間違っていました。
エラーの部分とは？？
解決は良いのですが結局どのように解決されたかはなるべく具体的に記載してください。でないと、回答した者としては意図が伝わってるかとかじゅうぶん理解されたのかとか色々と不安にもなりますので
プログラムはあっていたのですが、文字の入力の部分がしっかり設定できていなかったです。
SQL側が文字入力のデータ型がINTになっていたのと、照合属性を正しく記入できていなかったです。
あとは、エラー処理に関する（ $dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);）を記入が漏れていました。
以上店を修正すると正しく動きました。
最後まで雑くて申し訳ないです。
了解です。解決されたようで何よりです。
ご迷惑おかけしました。
ありがとうございます。
あなたの回答
tips
プレビュー
