Q&A
有り難うございます！
前提・実現したいこと
簡易的な掲示板を作っているのですが、名前とコメントが未入力の際に別ファイルに遷移させたいのですが、元のページを開いた瞬間にheader関数が動いて、入力画面が表示される前に遷移してしまいます。
該当のソースコード
<!DOCTYPE html> <html> <head lang="ja"> <meta charset="utf-8"> <title>坂本掲示板</title> </head> <body> <h3>坂本掲示板</h3> <form method="post" action="db_keijiban.php"> <input type="text" name="name" placeholder="Name"> <br> <textarea name="come"></textarea> <input type="submit" value="送る"> </form> <?php $sentname=$_POST['name']; $sentcome=$_POST['come']; try{ $dbh= new PDO('mysql:dbname=mydb;host=localhost;charset=utf8','root','root'); }catch(PDOException $e){ echo 'DBerror:'.$e->getMessage(); } if(!empty($sentname) && !empty($sentcome)){ $dbh->query("INSERT INTO keigiban(name,comment,time) VALUES('$sentname','$sentcome',NOW())"); }else{ header("location:erro.php"); } ?> <span> <?php $stmt=$dbh->query("SELECT * FROM keigiban"); while ($row = $stmt->fetch()) { if(empty($row)){ echo "さあ始めましょう"; }else{ printf("%s ; %s ; %s ; %s<br />\n", $row['id'], $row['name'], $row['comment'], $row['time']); } }; ?> </span> <div> <p>消したい番号とパスワードを入力してください</p> <form action="db_keijiban.php" method="post"> <input type="text" name="delpass" placeholder="パスワード"> <input type="text" name="del_id" placeholder="番号"> <input type="submit" value="パスワード入力"> </form> </div> <div> <span> <?php $del_id=$_POST['del_id']; $delpass='ryosuke0513'; if($delpass==$_POST['delpass']){ $sql="DELETE FROM keijiban WHERE id={$del_id}"; }else{ echo "パスワードが違います"; } ?> </span> </div> </body> </html>
補足情報（FW/ツールのバージョンなど）
php 7.3.8
回答3件
0
【PHP: header - Manual】
https://www.php.net/manual/ja/function.header.php
覚えておいて頂きたいのは、header() 関数は、 通常の HTML タグまたは PHP からの出力にかかわらず、すべての実際の 出力の前にコールする必要があることです。 頻出するエラーとして、include または require 関数、他のファイルをアクセスする関数に 空白または空行があり、header() の前に出力が 行われてしまうというものがあります。同じ問題は、単一の PHP/HTML ファイルを使用している場合でも存在します。
投稿2020/01/23 08:03
総合スコア69398
0
ベストアンサー
そもそもそこでheaderは文法的にアウトです。
覚えておいて頂きたいのは、header() 関数は、 通常の HTML タグまたは PHP からの出力にかかわらず、すべての実際の 出力の前にコールする必要があることです。
でなくても、出力を伴わないロジックはなるべく出力前に集めましょう。
「エラー」というのはバリデーションの結果、引っ掛かったということかと思いますが、別に他のページに飛ばす必要はないのでは？
単にエラーだけ表示させたページを用意したいのならそのページをincludeしてエラーメッセージだけ変数で渡して、表示させるのでも充分と思います。
いずれにしても今のコードだと$_POSTのindexのNoticeも出るでしょうし、REQUEST_METHODを見てPOSTかどうかを判断して、POSTに関する処理を入れるのは必要と思います(アクセスしたとたんのリダイレクトも解消されると思います)
投稿2020/01/23 08:10
総合スコア80850
解決は良いのですが、結局どのように解決されたのでしょうか。
https://teratail.com/help/question-tips#questionTips4-2
ご指摘有り難うございます。
別ファイルにて中身を検証し、それをincludeしてif文の中に入れようと考えています！
んー・・ちょっと変な処理になりそうな気がしますけど。
あくまで「ロジックとビューは分離させよう」という考え方です。
https://qiita.com/tadsan/items/fb496e450fc27c8c4494
>テンプレートとロジックは分けろ
phpの表示されない部分とhtmlを分けて書くということですね。参考になります。有り難うございます。
そうですね。1ファイルに記述するにしても出力する変数の記述をのぞいて「処理」部分はhtmlの記述と混ぜてはいけません。
読みにくいですしメンテナンスもしづらくなります。問題切り分けも困難にします。
0
以下参考になるかもしれません
投稿2020/01/23 08:11
総合スコア876
あなたの回答
tips
プレビュー
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。