Q&A
ユーザ定義関数についてどの程度理解されているのでしょうか。かなり謎の作りをされているのでもう少し簡単にひとつひとつ確かめながら作っていかれた方がいいように思います
エラーメッセージの内容としては定義されていないのが原因のようなので、
最初の条件式の中に定義されているのが原因と予想しております。
PHP
1if (!empty($_POST) && empty($_SESSION['input_data'])) {
コード全体はこちらです。
実装方法がわからないので教えていただきたいです。
PHP
1<!DOCTYPE html> 2<html lang="ja"> 3<head> 4<meta charset="UTF-8"> 5<style>.error {color: #FF0000;}</style> 6</head> 7<body> 8<?php 9session_start(); 10 11/*エスケープ処理*/ 12function h($input_value) { 13 return htmlspecialchars($input_value, ENT_QUOTES, 'UTF-8'); 14} 15if (!empty($_POST) && empty($_SESSION['input_data'])) { 16 /* バリデーションチェック */ 17 function valid_name($input_value) { 18 if ($_POST["first_name"] === '' || $_POST["last_name"] === '') { 19 return $err_msg['full_name'] = '*名前を入力してください'; 20 } 21 } 22 function valid_email($input_value) { 23 if ($_POST["email"] === '' || $_POST["email_confirmation"] === '') { 24 return $err_msg['email'] = '*メールアドレスを入力してください'; 25 } elseif (!filter_var($_POST["email"], $pattern['email']) || !filter_var($_POST["email_confirmation"], FILTER_VALIDATE_EMAIL)) { 26 return $err_msg['email'] = 'メールアドレスの形式が不明です'; 27 } elseif ($_POST["email"] !== $_POST["email_confirmation"]) { 28 return $err_msg['email'] = '*メールアドレスが一致していません'; 29 } 30 } 31 /* END. バリデーションチェック */ 32 /*エラーがない場合は確認画面に推移させる*/ 33 if (!isset($err_msg)) { 34 $_SESSION['input_data'] = $_POST; 35 header('Location:./confirm.php'); 36 exit(); 37 } 38} elseif (!empty($_SESSION['input_data'])) { 39 $_POST = $_SESSION['input_data']; 40} 41session_destroy(); 42?> 43<div class="wrapper"> 44 <form method="post"action="input.php"> 45 <table> 46 <tr> 47 <td><p>名前</p></td> 48 <td> 49 <input type="text" name="first_name" value="<?php echo isset($_POST['first_name']) ? h($_POST['first_name']) : '';?>" /> 50 <input type="text" name="last_name" value="<?php echo isset($_POST['last_name']) ? h($_POST['last_name']) : '';?>" /> 51 <span class="error"><?php echo valid_name($_POST); ?></span> 52 </td> 53 </tr> 54 <tr> 55 <td><p>メールアドレス</p></td> 56 <td> 57 <input type="text" name="email" value="<?php echo isset($_POST['email']) ? h($_POST['email']) : '';?>" size="50" placeholder="hogehoge@example.com" /> 58 <span class="error"><?php echo valid_email($_POST); ?></span> 59 </td> 60 </tr> 61 <tr> 62 <td><p>メールアドレス(確認用)</p></td> 63 <td> 64 <input type="text" name="email_confirmation" value="<?php echo isset($_POST['email_confirmation']) ? h($_POST['email_confirmation']) : '';?>" size="50" /> 65 <span class="error"><?php echo valid_email($_POST); ?></span> 66 </td> 67 </tr> 68 <td></td> 69 <td><input type="submit" value="送信する" /></td> 70 </table> 71 </form> 72 </div> 73</body> 74</html> 75
関数として定義する前
PHP
1if (!empty($_POST) && empty($_SESSION['input_data'])) { 2 /* バリデーションチェック */ 3 if ($_POST["first_name"] === '' || $_POST["last_name"] === '') { 4 return $err_msg['full_name'] = '*名前を入力してください'; 5 } 6 if ($_POST["email"] === '' || $_POST["email_confirmation"] === '') { 7 return $err_msg['email'] = '*メールアドレスを入力してください'; 8 } elseif (!filter_var($_POST["email"], $pattern['email']) || !filter_var($_POST["email_confirmation"], FILTER_VALIDATE_EMAIL)) { 9 return $err_msg['email'] = 'メールアドレスの形式が不明です'; 10 } elseif ($_POST["email"] !== $_POST["email_confirmation"]) { 11 return $err_msg['email'] = '*メールアドレスが一致していません'; 12 } 13 /* END. バリデーションチェック */ 14 /*エラーがない場合は確認画面に推移させる*/ 15 if (!isset($err_msg)) { 16 $_SESSION['input_data'] = $_POST; 17 header('Location:./confirm.php'); 18 exit(); 19 } 20} elseif (!empty($_SESSION['input_data'])) { 21 $_POST = $_SESSION['input_data']; 22}
ご回答ありがとうございます。どの辺が謎なのかわからなかったので再度学び直したいと思います。
ifの中に入れるところが。定義自体を制御したいなら別ですが呼び出されない限りはなにもしないので分岐の中に入れるところが謎です
どのような意図があって定義自体を分岐の中に入れたのかお聞きしたいです。そうしなければ都合が悪いことがあるのでしたらその辺りも質問に追記してください
回答3件
0
if文の中で function valid_name() を宣言しているので、このif文を通らなければ valid_name() は当然、未定義となります。
投稿2018/09/02 05:40
総合スコア821
ご回答ありがとうございます、if文を経過させてvalid_name()で入力チェックさせてエラーの場合表示させる方法が分からなかったので質問させていただきました、よろしければヒントいただけませんでしょうか？
function valid_name() は、関数 valid_name() の動作を定義しているところであって、これは外から valid_name() で呼ばれて初めて動作します。通常、function の定義は if() など条件の中に入れるものではありません。具体的には、if() の囲みの外に出してください。
0
標題についてはtakepan1973さんの回答で解決しますが、幾つか別の問題があります。
$err_msgは常に未定義になります。例え回答の通り関数定義部分を外に出したとしても常にheaderでリダイレクトされ続けることになるはずです。
なぜかというと関数にてreturnをしていますが、下記のようになっています。
php
1 return $err_msg['email'] = '*メールアドレスを入力してください';
変数には「スコープ」というもの（変数が有効な範囲）があり、関数内で定義された変数は関数の中だけしか生きません。
下記のようにミニマムコードで試してみると良いです。
php
1<?php 2function test(){ 3 $test = 2; 4 return $test; 5} 6 7$test2 = test(); 8 9if(isset($test)){ 10 echo $test; 11}else{ 12 echo "なし"; 13}
「なし」と出力されると思います。
functionの直後に書こうとどこに書こうと$testはtest()の中で定義されたもので、returnしたとしてもスコープはtest()の中以外にはありません。
つまり、実際はメッセージが出力されるから気にしないかもしれませんが、そのままechoするのであれば、returnの部分は下記と変わりません。
php
1return '*メールアドレスを入力してください';
では if (!isset($err_msg)) {を有効にするのはどうしたらいいか。
結局エラーメッセージを出力するためだけなのであれば、呼び出す場所を変えて、確実に通るようにすれば良いです。
項目毎に関数作ると項目が増えれば関数が増えるので一つにします。
本当はルール作って設定とループなど使って一気にやるのがいいのですが、その辺りはおいおい考えてもらうとして。
例：
php
1function valid() { 2 $err = []; 3 if ($_POST["name"] === '' || $_POST["name"] === '') { 4 $err["name"][] = '*名前を入力してください'; 5 } 6 return $err; 7} 8 9 10 11$err_msg = valid(); 12if (!isset($err_msg)) { 13 14/////省略 15 16 17//エラーメッセージ表示させるところ 18$name_err = ""; 19if(count($err_msg["name"])>0){ 20 $name_err = implode("/",$err_msg["name"]); 21} 22 23/////省略 24<span class="error"><?php echo $name_err ; ?></span>
作ってて気づきましたけど、$input_valueってなにも使ってませんね。
$_POSTは調べればわかりますが「スーパーグローバル変数」で、実はこの変数はスコープ関係なくどこから参照できるものです。
引数で渡す意味がありません。
またemailチェックの中でも未定義関数があります。
かなり厳しいかもしれませんが、作りがかなりおかしいです。
文法的なところもそうですし、未定義の変数がいきなりできていたり、
もちろん今回のifの中でfunctionを定義していたりスーパーグローバル変数を引数で渡したり、
引数で渡した変数を使っていなかったり。
セッションの使い方もちょっと微妙に思います。
そういった意味でコメントで「謎の作り」と表現しました。
今回私のほうでミニマムコードを提示しましたが、そのようなミニマムコードで最低限の動作を確認して、
機能を１つ１つ理解されていったほうがいいように思います。
今の作りと理解度では前に進んだようで何歩も戻るようなことを繰り返すことになります。
投稿2018/09/02 06:47
編集2018/09/02 06:48総合スコア80850
0
表示する所で
<span class="error"><?php echo valid_name($_POST); ?></span>
を
<span class="error"><?php if(function_exists('valid_name')) { echo valid_name($_POST);} ?></span>
投稿2018/09/02 06:10
総合スコア321
あなたの回答
tips
プレビュー
