Q&A
＞表示項目が余計に一つ出ている
どの表示項目でしょう。そしてどう出したいのでしょう
phpで、入力画面を作ったのですが、[戻る]という表示項目が余計に一つ出ているバグが起きています。
表示結果
修正方法を教えてください。
php
1<html> 2 <head> 3 <meta http-equiv="content-type" charset="utf-8"> 4 </head> 5 6 7 <body> 8 9 <?php 10 $nickname=$_POST["nickname"]; 11 $email=$_POST["email"]; 12 $goiken=$_POST["goiken"]; 13 14 if($nickname=="") 15 { 16 print"ニックネームが入力されていません。<br>"; 17 } 18 else 19 { 20 print"ようこそ"; 21 print $nickname; 22 print"様"; 23 print"<br>"; 24 } 25 26 if($email=="") 27 { 28 print"メールアドレスが入力されていません。<br>"; 29 } 30 else 31 { 32 print"メールアドレス:"; 33 print$email; 34 print"<br>"; 35 } 36 37 if($goiken=="") 38{ 39 print"ご意見が入力されていません。<br>"; 40} 41else 42{ 43 print"ご意見『"; 44 print $goiken; 45 print"』<br>"; 46} 47 48if( $nickname=''||$email==''||$goiken=='') 49{ 50 print'<form>'; 51 print'<input type="button" onclick="history.back()" value="戻る">'; 52 print'</form>'; 53} 54 print'<form method="post" action="thanks.php">'; 55 print'<input type="button" onclick="history.back()" value="戻る">'; 56 print'<input type="submit" value="OK">'; 57 print"</form>"; 58?> 59 60 </body> 61</html>
あとこの作りだと「OK」押したところでボタン以外何も送信されませんよ。
＞どの表示項目でしょう。そしてどう出したいのでしょう
「戻る」というボタンが余分に二個表示されています。
＞あとこの作りだと「OK」押したところでボタン以外何も送信されませんよ。
ご指摘ありがとうございます。しかし、このプログラムは作成途中の段階で、現在はこの表示が出たため、質問をさせてもらうことにさせていただきました。
了解です。
あとタグはphpとHTML以外は不要かなと（むしろタイトルを正とするならphpだけでいいような）思いますので、整理をお願いできたらと。
回答2件
0
ベストアンサー
php
1if( $nickname=''||$email==''||$goiken=='') // １つ比較式ではなく代入が混じってます 2{ 3 print'<form>'; 4 print'<input type="button" onclick="history.back()" value="戻る">'; 5 print'</form>'; 6} //閉じてるからその後はそのまま出力されるよね 7 print'<form method="post" action="thanks.php">'; 8 print'<input type="button" onclick="history.back()" value="戻る">'; 9 print'<input type="submit" value="OK">'; 10 print"</form>";
あとは「フラグ制御」としたいのであればボタン出力の分岐に同じ式を入れるのではなく、
デフォルトfalseとしたフラグをそれぞれの入力チェックの時にtrueにしてそのフラグだけ見れば良いです。そのほうがスッキリします。
php
1$error = false; 2if($a==''){ 3 $error = true; 4}else{ 5} 6 7if($b==''){ 8 $error = true; 9}else{ 10} 11 12if($error){ 13 //入力エラーあるよ 14}else{ 15 //入力ーないよ 16}
投稿2019/08/20 03:23
編集2019/08/20 03:43総合スコア80850
試したのですが、うまくいきませんでした。
表示結果の画像を搭載させていただきましたので、是非確認してください。
何を試してどうなったのかが分かりません。
直接のアドバイスとしては「//閉じてるからその後はそのまま出力されるよね」のところです。
質問者さんは「trueのときはaだけ出したい。falseのときはbだけ出したい」と言っていても下記のように書いているのは分かりますか？
if(true){
echo "a";
}
echo "b";
この書き方だとbはどんなときも出ます。
ちなみに私の回答のコードはあくまで質問者さんのコードそのままで指摘のコメントを書いただけのもので、「これを使えば想定通り動くよ」というものではありません。
「どこが悪いのか分からない」ように見えたので、指摘の結果から、「どうすればいいか」を考えてもらいたかったのです。
>何を試してどうなったのかが分かりません。
試した結果の表示結果は、質問記事に指摘された後に、画像を貼らせていただきましたので、参考になれば幸いです。
＞質問者さんは「trueのときはaだけ出したい」と言っていても下記のように書いているのは分かりますか？
if(true){
echo "a";
}
echo "b";
なるほど。条件をクリアした時のみ表示したいのに、コードではその条件でなくても表示されるコードを書いているということでよろしかったですか？
>画像を貼らせていただきましたので、参考になれば幸いです。
いえ、コードです。私の回答からコードを修正した結果でしたら、そのコードも提示してください。
>なるほど。条件をクリアした時のみ表示したいのに、コードではその条件でなくても表示されるコードを書いているということでよろしかったですか？
はい。分岐関係ないので何があっても通ります。
解決しました。
ありがとうございます。
フォローをさせていただきました。
今後とも、ご迷惑をおかけしますが、よろしくお願いします。
「本来のフラグの使いかた」も簡易コードで追記してますので参考にしてください
ご丁寧に分かりやすくありがとうございます
0
まず変数への投入はfilter_inputで
PHP
1$nickname=filter_input(INPUT_POST,"nickname"); 2$email=filter_input(INPUT_POST,"email"); 3$goiken=filter_input(INPUT_POST,"goiken");
if( $nickname=''||$email==''||$goiken=='')
文字列比較がおかしい（==もしくは===とすべきところを=になっています)
またpostで受け取ったデータを出力するときは必ず
htmlspecialcharsを利用しなくてはいけません
投稿2019/08/20 03:28
総合スコア114775
ご指摘ありがとうございます。
指摘をしてくださった点を、書き換えてみたのですが、表示結果は変わりませんでした。表示結果のスクリーンショットを掲載させていただきましたので、是非ご確認ください。
あ、重要なところ指摘忘れてました
ifにたいしてelesが抜けています
これが、原因だったんですね。
解決しました。ありがとうございます。
横やり失礼します。
eles ではなく else ですね。
あなたの回答
tips
プレビュー
