お問い合せフォーム　バリデーション　PHP

前提

PHPのバリデーションを使用すること

実現したいこと

入力欄が未入力の時はエラーメッセージを表示させる。
(〇〇は必須項目です。と表示)

発生している問題・エラーメッセージ

エラーメッセージが表示されない

該当のソースコード

<?php
//使用する変数を初期化
$name = '';
$hurigana = '';
$tel = '';
$email = '';
$contact = '';

//エラー内容
$errors = [];

//送信データをチェック
if (isset($_POST)) {
  //氏名
  if (empty($_POST['name'])) {
    $errors[] = '氏名は必須項目です。';
  }
  //フリガナ
  if (empty($_POST['hurigana'])) {
    $errors[] = 'フリガナは必須項目です。';
  }
  if (empty($_POST['email'])) {
    $errors[] = 'メールアドレスは必須項目です。';
  }
  if (empty($_POST['contact'])) {
    $errors[] = 'お問い合わせ内容を入力してください';
  }
}
?>
<!DOCTYPE html>
<html lang="ja">
<head>
    <meta charset="UTF-8">
    <meta http-equiv="X-UA-Compatible" content="IE=edge">
    <meta name="viewport" content="width=device-width, initial-scale=1">
    <link rel="stylesheet" href="contact.css">
    <script src="https://cdn.jsdelivr.net/npm/jquery@3.4.1/dist/jquery.min.js"></script>
<title>Document</title>
</head>
<body>
<?php include ('header.php');?>
<!---!/*お問い合わせフォーム*--->
<form action="comfirm.php" method="post">
<div class="Form">
 <div class="title">お問い合わせ</div>
 <p class="under"></p>
 <p class="sub">下記の項目を入力し、送信ボタンを押してください。</p>
  <div class="Form-Item">
    <p class="Form-Item-Label"><input type="submit" class="Form-Item-Label-Required" value="必須">氏名</p>
    <input type="text" name="name" class="Form-Item-Input" placeholder="" requied>
  </div>
  <div class="Form-Item">
    <p class="Form-Item-Label"><input type="submit" class="Form-Item-Label-Required" value="必須">フリガナ</p>
    <input type="text" name="hurigana" class="Form-Item-Input" placeholder="" required>
  </div>
  <div class="Form-Item">
    <p class="Form-Item-Label">電話番号</p>
    <input type="text" name="tel" class="Form-Item-Input" placeholder="" required>
  </div>
  <div class="Form-Item">
    <p class="Form-Item-Label"><input type="submit" class="Form-Item-Label-Required" value="必須">メールアドレス</p>
    <input type="email" name="email" class="Form-Item-Input" placeholder="" required>
  </div>
  <div class="Form-Item">
    <p class="Form-Item-Label isMsg"><input type="submit" class="Form-Item-Label-Required" value="必須">お問い合わせ内容</p>
    <textarea name="contact" class="Form-Item-Input-main" placeholder="" required></textarea>
  </div>
  <input type="submit" class="Form-Btn" value="送信">
</div>
<?php include ('footer.php');?>
</form>
</body>
</html>

アドバイスいただけると嬉しいです。

yuma.inaura

2023/01/16 15:14

$errors を表示してる箇所がどこにもないですね

行動規範の内容に同意します

回答3件

PHP
1$errors

投稿2023/01/16 16:16

退会済みユーザー

総合スコア0

ベストアンサー

汎用性を考えるとこんな感じでは？

PHP
1<?php
2$errflg=$_SERVER["REQUEST_METHOD"]=="POST"?false:true;
3$errors = [];
4$vals=[];
5$validate=[
6  ["param"=>"name"     ,"required"=>"氏名は必須項目です。",],
7  ["param"=>"hurigana" ,"required"=>"フリガナは必須項目です。",],
8  ["param"=>"tel"],
9  ["param"=>"email"    ,"required"=>"メールアドレスは必須項目です。",],
10  ["param"=>"contact"  ,"required"=>"お問い合わせ内容を入力してください",],
11  ];
12foreach($validate as $data){
13  $val=filter_input(INPUT_POST,$data["param"]);
14  $vals[$data["param"]]=htmlspecialchars($val);
15  $errors[$data["param"]]="";
16  if(isset($data["required"]) and $val===""){
17    $errflg=true;
18    $errors[$data["param"]]=$data["required"];
19  }
20}
21?>
22<style>
23.error{color:red}
24</style>
25<?PHP
26if($errflg){
27?>
28<form method="post">
29<dl>
30<dt>氏名</dt><dd><input name="name" value="<?=$vals["name"]?>"><span class="error"><?=$errors["name"]?></span></dd>
31<dt>フリガナ</dt><dd><input name="hurigana" value="<?=$vals["hurigana"]?>"><span class="error"><?=$errors["hurigana"]?></span></dd>
32<dt>電話番号</dt><dd><input name="tel" value="<?=$vals["tel"]?>"><span class="error"><?=$errors["tel"]?></span></dd>
33<dt>メールアドレス</dt><dd><input name="email" value="<?=$vals["email"]?>"><span class="error"><?=$errors["email"]?></span></dd>
34<dt>お問い合わせ内容</dt><dd><textarea name="contact"><?=$vals["contact"]?></textarea><span class="error"><?=$errors["contact"]?></span></dd>
35<input type="submit" value="送信">
36</form>
37<?PHP
38}else{
39?>
40OK
41<?PHP
42}
43?>

投稿2023/01/17 01:36

yambejp

総合スコア117678

変数使用箇所を通知・ハイライトしてくれるプログラミング特化のエディタや統合環境（IDE）を利用されてはどうでしょうか。

以下はeclipseのエディタ部分のスクリーンショット

現状だと値の定義だけで使用（出力）されていない変数が多いように思います。

この手のエディタでは使用箇所から定義されている箇所に飛べたり（関数も）
構文エラー程度なら実行前に警告してくれるのでかなり有用と思いますし、
デバッグ実行もそんなに難儀ではないです。

投稿2023/01/16 23:33

m.ts10806

総合スコア80888

m.ts10806

2023/01/17 00:56

プログラミングに特化したエディタ（というか統合環境で無料で利用できる）で質問タグにもあるeclipseがスパムだったらteratail自体を否定することになるんですが、そこまで考えてないでしょうね。誰とは言わないけど。ちゃんと理由を書こう。スパム要素が全く見当たりません。

退会済みユーザー

2023/01/17 01:52

私ではない。冤罪。

m.ts10806

2023/01/17 07:17 編集

なら、出てこないでください。回答編集したわけでもないのにわざわざコメントに反応してるのがもう、自分がやったって言ってるようなものだし。少しは自覚してんのかね。まあでもあなたはサービス全てへの邪魔しかしてないので。あなたが持ってる他のアカウントのどれでもないという保証はないし前科もたっぷりあるのをお忘れなく。

行動規範の内容に同意します

あなたの回答