Q&A
コード＆エラーはそれぞれコードブロックで囲んでいただけませんか？ ```（バッククオート3つ）で囲み、前後に改行をいれるか、コードを選択して「<code>」ボタンを押すとコードブロックになります。
phpでフォームを作りました。
form.php→sent.php→complete.phpの順番ですが、
特に問題なくメール飛んで、リロードもしないようにしましたが、
urlで直接sent.phpを入力してたら、エラーがいっぱい出ています。
調べたところissetで解決できそうなんですが、具体的にどうすればいいですか？よろしくお願いいたします。
###発生している問題・エラーメッセージ
Notice: Undefined index: name in C:\xampp\htdocs\form\sent.php on line 4
Notice: Undefined index: gender in C:\xampp\htdocs\form\sent.php on line 5
Notice: Undefined index: age in C:\xampp\htdocs\form\sent.php on line 6
Notice: Undefined index: body in C:\xampp\htdocs\form\sent.php on line 7
Notice: Undefined index: sports in C:\xampp\htdocs\form\sent.php on line 8
アンケート
名前
Notice: Undefined index: name in C:\xampp\htdocs\form\sent.php on line 33
名前が未入力です
性別
Notice: Undefined index: gender in C:\xampp\htdocs\form\sent.php on line 49
年齢
Notice: Undefined index: age in C:\xampp\htdocs\form\sent.php on line 55
好きなスポーツを選択して下さい
Warning: Invalid argument supplied for foreach() in C:\xampp\htdocs\form\sent.php on line 64
内容
Notice: Undefined index: body in C:\xampp\htdocs\form\sent.php on line 77
内容が未入力です
Notice: Undefined index: body in C:\xampp\htdocs\form\sent.php on line 80
Notice: Undefined index: name in C:\xampp\htdocs\form\sent.php on line
###該当のソースコード
<?php session_start(); $_SESSION["ticket"] = SHA1(uniqid().mt_rand()); $name= $_POST['name']; $gender=$_POST['gender']; $age=$_POST['age']; $body=$_POST['body']; $sports=$_POST['sports']; ?> <!DOCTYPE html> <html> <head> <meta charset="utf-8"> <title>php</title> <link rel="stylesheet" href="common/css/reset.css"> <link rel="stylesheet" href="common/css/sent.css"> </head> <body><?php echo htmlspecialchars($_POST['age']); ?><div id="anketo"> <h4>アンケート</h4> <div class="title"><p>名前</p></div> <?php if ($_POST['name']=="") { echo '<FONT COLOR="RED">名前が未入力です</FONT>'; }else{ echo htmlspecialchars($_POST['name']);} ?> <div id="sex"> <div class="seibetu"><p>性別</p></div> <?php echo htmlspecialchars($_POST['gender']); ?> </div> <div class="title"><p>年齢</p></div>
<div class="sports"> <p class="suki">好きなスポーツを選択して下さい</p> </div> <div id="sp"> <?php
foreach ($sports as $value) {
print("$value"."<br>");
}
?>
</div>
<div id="btn"> <p id="text">内容</p> <div class="naiyou"> <?php if($_POST['body']=="") { echo '<FONT COLOR="RED">内容が未入力です</FONT>'; } $text = $_POST['body']; echo nl2br($text); ?> </div> </div> <form action="complete.php" method="post"> <div id="botan"> <div class="soushin"> <!--送信ボタン--> <?php if($_POST['name']==""||$_POST['body']==""){ ' <input type="submit" name="submit" value="送信"/></div>';} else{ echo ' <input type="submit" name="submit" nvalue="送信"/></div>';} ?> <input type="hidden" name="ticket" value="<?php echo htmlspecialchars($_SESSION["ticket"], ENT_QUOTES);?>"> <input type="hidden" name="name" value="<?php echo $name; ?>"> <input type="hidden" name="gender" value="<?php echo $gender; ?>"> <input type="hidden" name="age" value="<?php echo $age; ?>"> <input type="hidden" name="sports" value="<?php foreach ($sports as $value) {
print ("$value"."\r\n");
}?>">
<input type="hidden" name="body" value="<?php echo $body; ?>">
</div> <!--戻るボタン--></div><div class="modoru"> <input type="button" class="submit" id="return" value="戻る" onClick="history.back()"> <input type="hidden" name="name" value="<?php echo $name; ?>"></div>
</div><!-- anketo --> </body> </html> ###試したこと</form>
###補足情報(言語/FW/ツール等のバージョンなど)
初心者なので、優しくお願いいたします！
回答4件
0
ベストアンサー
そこまでわかっているならあとすこしですね。
POSTされていないのに
$name= $_POST['name'];
があるからエラーは当然出ます。
form.phpからPOSTした場合のみ変数に格納するわけですから、
php
1if(isset($_POST['name'])){ 2// $name= $_POST['name']; 等 3}
とかにしてあげれば大丈夫です。
投稿2016/11/24 01:15
総合スコア686
回答ありがとうございます！
もうひとつなんですが、
チェックボックスの場合はどうするですか？
foreach文使ってるですが、どうissetしたらいいかわからないです。
<?php
foreach ($sports as $value) {
print("$value"."<br>");
}
?>
すみません、よろしくお願いいたします。
チェックボックスもテキストも同じですよ。
チェックボックスの値をpostする場合も同様で
<input type="checkbox" name="hoge" value="huga">
に対して
$obj = $_POST["hoge"]; // $hoge = "huga";
になるわけですから。
foreachで回す場合はnameの部分がhoge[]とかになっている場合ですね。
issetはあくまで「格納されているか(nullじゃないか)」を確認するだけです。
ありがとうございます。
なんか そのままissetしたら 解決できました！
foreachで回すのは
http://alphasis.info/2012/11/php-gyakubiki-form-post-multiple/
多分こういうことを想定していたのでしょうか。
配列で一旦受け取ってからisset(nullかどうか)ではなくis_array(配列かどうか)を確認してifで分岐させるのもありでしょうね。
参考ページありがとうございます。
<?php
if (isset($_POST['sports'])){
foreach ($sports as $value)
print("$value"."<br>");
}
?>
自分はただこうしてissetしたら 普通にエラーが消えました。
もともとforeachにprintがあるので、そのままいれてたら、消えましたｗ
ちなみになんですが
<?php
if($_POST['name']==""||$_POST['body']==""){
' <input type="submit" name="submit" value="送信"/></div>';}
else{
echo ' <input type="submit" name="submit" nvalue="送信"/></div>';}
?>
ここもエラがでてまして、ここはもともと入力画面から何も入力されてないときに確認画面で警告が出てくるよう書いたのですが、今確認画面sent.phpのurlを直接入力すると、やはりnoticeエラーがでています。
ここはどうすべきでしょうか
if($_POST['name']==""||$_POST['body']==""){
echo ' <input type="submit" name="submit" value="送信"/></div>';
}
else{
echo ' <input type="submit" name="submit" value="送信"/></div>';
}
echoが抜けてますよ。あとnvalueになっています。（これはエラーとは関係ないと思いますが）
昔issetという関数を知らない頃、
if($_POST['name']==""){
みたいな書き方してました。懐かしい・・
ありがとうございます！
0
ヒント
session_start();
$_SESSION["ticket"] = SHA1(uniqid().mt_rand());
if(isset($_POST['name']) && isset($_POST['gender']) && isset($_POST['age']) && isset($_POST['body']) && isset($_POST['sports']) ){
$name= $_POST['name'];
$gender=$_POST['gender'];
$age=$_POST['age'];
$body=$_POST['body'];
$sports=$_POST['sports'];
}
投稿2016/11/24 01:15
総合スコア615
0
form.phpからsent.phpへがフォームを使ったPOSTリクエストが本来求める流れだとすると
sent.phpへGETでそれもPOSTパラメータ無しでアクセスがあった場合は弾かないといけない（意図しない動作になるため）
エラーはPOSTパラメータがないにもかかわらず、POSTパラメータを取得しようとしているからです。
いろいろやり方はあると思いますが
if($_SERVER["REQUEST_METHOD"] != "POST"){ // エラーもしくはリダイレクトする }
の値を見てがPOSTでない場合はエラーにするとかform.phpへリダイレクトしてやれば良いと思います。
投稿2016/11/24 01:12
総合スコア1689
0
sent.phpやcomplete.phpに直接アクセスされた場合はどのような動作をさせたいのでしょうか？
form.phpへリダイレクトさせるならこれを参考にしてください。
【PHP】結果画面のURLを打ち込んで結果画面に直接行こうとするとリダイレクト処理をしたい
https://teratail.com/questions/54150
投稿2016/11/24 01:15
総合スコア7914
回答ありがとうございます。
結果画面じゃなくて、確認画面のurlを直接打ち込んだ場合、エラーが出ないように、普通に入力画面で入力しないまま確認画面に飛んだ感じと一緒ですかね、説明が下手でごめんなさい。
あなたの回答
tips
プレビュー
