Q&A
どのあたりが、「どうも上手くいきません」なのか、もう少し記載してもらえると、いいですね！
PHP超初心者です。
PHPにてお問い合わせフォームを作成しています。
フォームに空白があった場合にエラーメッセージにて入力を促す指示を出したいのですがどうも上手くいきません。
フォームを送信すると$_POSTに各データがセットされているかの確認、データがセットされていなければ、エラーメッセージの表示、セットされていれば confirm.phpへ遷移という流れを書いたつもりです。
どなたかご教授賜りたいです。
＊具体的な問題点
現状のコードで実行し、フォームに内容を入力して送信てみると、フォームの入力内容が消え、次の画面（confirm.php）へ遷移せず、エラーメッセージもでないといった状況です。
当初はエラーメッセージ機能は実装していませんでした。（PHP記述をせず、HTMLのみ）
その場合だと、confirm.phpへ遷移することが出来たため、PHPの記述が間違っているのだろうと考えています。
コード <? //送信ボタンが押されたら// if (isset($_POST ["register"])) { //エラーメッセージを格納する配列を作成 $error_message = array (); if ($_POST ["onamae"]!=="") { //データがセットされていたら各変数にPOSTのデータを格納 $onamae = htmlspecialchars($_POST["onamae"],ENT_QUOTES); //各データがなかったらエラーメッセージを配列に格納 }else{ $error_message[] = "お名前を入力して下さい。<br>"; } if ($_POST["mailadress"]!=="") { $mailadress = htmlspecialchars($_POST["mailadress"],ENT_QUOTES); }else{ $error_message[] = "メールアドレスを入力してください。<br>"; } if ($_POST["category"]!=="") { $mailadress = htmlspecialchars($_POST["category"],ENT_QUOTES); }else{ $error_message[] = "お問い合わせの種類を選択してください。<br>"; } if ($_POST["interest[]"]!=="") { $mailadress = htmlspecialchars($_POST["interest"],ENT_QUOTES); }else{ $error_message[] = "興味のある分野を１つ以上選択してください。<br>"; } if ($_POST["body"]!=="") { $mailadress = htmlspecialchars($_POST["body"],ENT_QUOTES); }else{ $error_message[] = "メッセージを入力してください。<br>"; } //エラーが無い時 if (!count($error_message)){ //確認ページヘ header("Location:confirm.php"); exit; } } if (count($error_message)) { foreach ($error_message as $message){ print ($message); } } ?> <!DOCTYPE HTML> <html> <head> <title>お問い合わせフォーム</title> <meta charset="utf-8"> </head> <body> <form name="form1" method="post"action="confirm.php" > <H1>お問い合わせフォーム</H1> ・お名前<br> <input type="text" name="onamae" > <br> ・メールアドレス<br> <input type="text" name="mailadress"> <br> ・お問い合わせの種類<br> <select name="category" > <option value=""selected>選んで下さい</option> <option value="質問">質問</option> <option value="要望">要望</option> <option value="その他お問い合わせ">その他お問い合わせ</option> </select> <br> ・興味のある分野<br> <input type="checkbox" name="interest[]" value="HTML"> HTML<br> <input type="checkbox" name="interest[]" value="PHP"> PHP<br> <input type="checkbox" name="interest[]" value="MySQL"> MySQL<br> <input type="checkbox" name="interest[]" value="Linux"> Linux<br> <input type="checkbox" name="interest[]" value="マネジメント"> マネジメント<br> <input type="checkbox" name="interest[]" value="その他"> その他<br> ・メッセージ<br> <textarea name="body" cols="30" rows="5"></textarea> <br> <input type="submit" name="register value="送信"> </form> </body> </html>
回答4件
0
ベストアンサー
インデントを整理するとわかると思いますが、コードの次の部分が働きません
PHP
1if (count($error_message)) { 2 foreach ($error_message as $message){ 3 print ($message); 4 } 5}
以下、整理したコード
PHP
1//送信ボタンが押されたら// 2if (isset($_POST ["register"])) { 3 //エラーメッセージを格納する配列を作成 4 $error_message = array (); 5 if ($_POST ["onamae"]!=="") { 6 //データがセットされていたら各変数にPOSTのデータを格納 7 $onamae = htmlspecialchars($_POST["onamae"],ENT_QUOTES); 8 //各データがなかったらエラーメッセージを配列に格納 9 }else{ 10 $error_message[] = "お名前を入力して下さい。<br>"; 11 } 12 13 if ($_POST["mailadress"]!=="") { 14 $mailadress = htmlspecialchars($_POST["mailadress"],ENT_QUOTES); 15 }else{ 16 $error_message[] = "メールアドレスを入力してください。<br>"; 17 } 18 19 if ($_POST["category"]!=="") { 20 $mailadress = htmlspecialchars($_POST["category"],ENT_QUOTES); 21 }else{ 22 $error_message[] = "お問い合わせの種類を選択してください。<br>"; 23 } 24 25 if ($_POST["interest[]"]!=="") { 26 $mailadress = htmlspecialchars($_POST["interest"],ENT_QUOTES); 27 }else{ 28 $error_message[] = "興味のある分野を１つ以上選択してください。<br>"; 29 } 30 if ($_POST["body"]!=="") { 31 $mailadress = htmlspecialchars($_POST["body"],ENT_QUOTES); 32 }else{ 33 $error_message[] = "メッセージを入力してください。<br>"; 34 } 35 //エラーが無い時 36 if (!count($error_message)){ 37 //確認ページヘ 38 header("Location:confirm.php"); 39 exit; 40 } 41} 42if (count($error_message)) { 43 foreach ($error_message as $message){ 44 print ($message); 45 } 46}
最初に指摘した部分がif (isset($_POST ["register"])) {}の外にあるため、エラーがあっても表示できません。
追記
他の方が指摘されていたように、エラーメッセージの表示場所に問題があるので、その点も踏まえて次のようにするとどうでしょうか。
PHP
1<? 2//送信ボタンが押されたら// 3$error_message = array (); //<--ここ$error_messageを定義 4if (isset($_POST ["register"])) { 5 //エラーメッセージを格納する配列を作成 6 if ($_POST ["onamae"]!=="") { 7 //データがセットされていたら各変数にPOSTのデータを格納 8 $onamae = htmlspecialchars($_POST["onamae"],ENT_QUOTES); 9 //各データがなかったらエラーメッセージを配列に格納 10 }else{ 11 $error_message[] = "お名前を入力して下さい。<br>"; 12 } 13 14 if ($_POST["mailadress"]!=="") { 15 $mailadress = htmlspecialchars($_POST["mailadress"],ENT_QUOTES); 16 }else{ 17 $error_message[] = "メールアドレスを入力してください。<br>"; 18 } 19 20 if ($_POST["category"]!=="") { 21 $mailadress = htmlspecialchars($_POST["category"],ENT_QUOTES); 22 }else{ 23 $error_message[] = "お問い合わせの種類を選択してください。<br>"; 24 } 25 26 if ($_POST["interest[]"]!=="") { 27 $mailadress = htmlspecialchars($_POST["interest"],ENT_QUOTES); 28 }else{ 29 $error_message[] = "興味のある分野を１つ以上選択してください。<br>"; 30 } 31 if ($_POST["body"]!=="") { 32 $mailadress = htmlspecialchars($_POST["body"],ENT_QUOTES); 33 }else{ 34 $error_message[] = "メッセージを入力してください。<br>"; 35 } 36 //エラーが無い時 37 if (!count($error_message)){ 38 //確認ページヘ 39 header("Location:confirm.php"); 40 exit; 41 } 42} 43?> 44<!DOCTYPE HTML> 45<html> 46<head> 47 <title>お問い合わせフォーム</title> 48 <meta charset="utf-8"> 49</head> 50<body> 51<?php 52//ここに記述することでhtml内で表示させる 53if (count($error_message)) { 54 foreach ($error_message as $message){ 55 print ($message); 56 } 57} 58?> 59 <form name="form1" method="post"action="confirm.php" > 60 61 <H1>お問い合わせフォーム</H1> 62 ・お名前<br> 63 <input type="text" name="onamae" > 64 <br> 65 66 ・メールアドレス<br> 67 <input type="text" name="mailadress"> 68 <br> 69 70 ・お問い合わせの種類<br> 71 <select name="category" > 72 <option value=""selected>選んで下さい</option> 73 <option value="質問">質問</option> 74 <option value="要望">要望</option> 75 <option value="その他お問い合わせ">その他お問い合わせ</option> 76 </select> 77 <br> 78 79 ・興味のある分野<br> 80 <input type="checkbox" name="interest[]" value="HTML"> 81 HTML<br> 82 <input type="checkbox" name="interest[]" value="PHP"> 83 PHP<br> 84 <input type="checkbox" name="interest[]" value="MySQL"> 85 MySQL<br> 86 <input type="checkbox" name="interest[]" value="Linux"> 87 Linux<br> 88 <input type="checkbox" name="interest[]" value="マネジメント"> 89 マネジメント<br> 90 <input type="checkbox" name="interest[]" value="その他"> 91 その他<br> 92 93 ・メッセージ<br> 94 <textarea name="body" cols="30" rows="5"></textarea> 95 <br> 96 97 <input type="submit" name="register value="送信"> 98 99 </form> 100</body> 101</html>
投稿2016/04/05 23:51
編集2016/04/05 23:58
総合スコア1698
ご回答ありがとうございます。
エラーメッセージの表示場所の指摘の意味がようやく分かりました。。ありがとうございます。
修正していただいたコードを使い実行してみましたが、次の画面に遷移させることができましたが、空白のままでも遷移できてしまうので、PHPのロジックが間違っているのではと考えております。自分なりにもう一度考えなおしてみます。
とても分かりやすく、また修正していただき、ありがとうございました。
0
おっと、、、ざっと見、ですが、'}'１つ多くありません？？？
php
1 if (!count($error_message)){ 2 //確認ページヘ 3 header("Location:confirm.php"); 4 exit; 5 } // <<<<これいらない？ 6 }
っと、インデントから判断しましたが、そうすると、最後に'}'が足らないのかな？？？
投稿2016/04/05 23:34
編集2016/04/05 23:36総合スコア1283
0
php
1header("Location:confirm.php");
が実行されていないようですので、おそらく、PHP側の何らかのエラーでしょう。
そこで、
php
1<?php 2//エラー出力強制 3ini_set('display_errors',1); 4//すべてのエラー表示 5error_reporting(E_ALL);
を先頭に記載して、PHP側のエラーを表示させると、もう少し何かわかるかもしれません。
投稿2016/04/05 23:30
総合スコア1283
上記を試してみましたが、エラーが表示されず、＊具体的な問題点に挙げた現象が再び起きてしまいました・・・。
「フォームの入力内容が消え」との事？なので、HTMLは表示されているのでしょうか？真っ白画面ではない（真っ白画面だとPHPのエラーですが、HTMLがでているなら、PHPのシンタックスは正常）？
ということは、処理場
```
//エラーが無い時
if (!count($error_message)){
//確認ページヘ
header("Location:confirm.php");
exit;
}
}
```
も、
```
if (count($error_message)) {
foreach ($error_message as $message){
print ($message);
}
}
```
も、通っていないということですね、、、
ということは、ロジック的な誤りかと。
再度、エラーチェック部分を１つ１つ追いかけてみたらよいのではと思います。
HTMLは表示されています。
私もロジックが間違っているのではないかと思うに至りましたので、ご指摘の通りエラーチェック部分を追いかけてみようと思います。
何度もご回答ありがとうございます。
0
ざっと気になるとこは…
・エラーメッセージをhtmlの前に出してる
・confirm.phpへ入力内容を引き継いでない
で、もちっと「こうなるつもりなんだけど、こうなっちゃってる」
みたいな話を書いてもらえると具体的な指定がされると思うます。
投稿2016/04/05 23:17
総合スコア7458
confim.phpへ入力内容を引き継いでいないというのは、formタグにaction="confirm.php"の記述がないということでしょうか？
あ、今気づきましたが…
エラーが無い場合に confirm.php へ遷移するつもりのようですが、
action="confirm.php" なので submitしたら confirm.php が
実行されますね。つまり、ここで提示された処理は実行されないのでは?
php で遷移するのと、action で遷移するのではどちを想定されてるのか
分かりませんが、それによってデータ引き継ぎ云々が変わってきます。
あなたの回答
tips
プレビュー
