入力データを保持したままエラー表示したい

Question

PHPの登録フォームを作っています。
ユーザーが入力ミスをした時にデータをキープしたままエラーメッセージが表示されるようにしたいです。

現在のプログラムですと、３項目のうち、１つや２つだけブランクにした場合、他の入力データをキープしたままvalidation エラーが表示されます。

しかし、validation errorにひっかかるデータを入力した場合、全て入力データが消えてしまいます。

またusername とemailのvalidationは、
"/^[A-Za-z0-9' -]{1,15}$/"
とセットしているのですが、１５文字以上入力できてしまいます。
もちろんDBの方は１５文字でセットしてあるので、１６文字以降はただカットされてます。
１５文字以上入力された場合、エラーを出すにはどうしたらいいでしょうか。

<?php
/*  Program name: register.php
 *  Description:  Program displays the blank form and checks
 *  all the form fields for blank fields.
 */
ini_set("display_errors","on");
error_reporting(E_ALL | E_STRICT);
ini_set("include_path","./includes");
include("dbinfo.inc");

if(isset($_POST['submitted']) and $_POST['submitted'] == "yes")
{
  foreach($_POST as $field => $value)
  {
    if(empty($value))
    {
      if($field != "middle_name")
      {
         $blank_data[] = $field;
      }
    }
    else
    {
      $good_data[$field] = strip_tags(trim($value));
    }
  }
  if(@sizeof($blank_data) > 0)
  {
    $message = "<p style='color: red; margin-bottom: 0;
                 font-weight: bold'>
                 You didn't fill in one or more required fields.
                 You must enter:
                 <ul style='color: red; margin-top: 0;
                 list-style: none' >";
 /* display list of missing information */
    foreach($blank_data as $value)
    {
      $message .= "<li>$value</li>";
    }
    $message .= "</ul>";
    echo $message;
    extract($good_data);
    include("form.inc");
    exit();
  }
  foreach($_POST as $field => $value)
  {
    if(!empty($value))
    {
		//validations
      $name_patt = "/^[A-Za-z0-9' -]{1,15}$/";
	  $email_patt = "/^.+@.+\\..+$/";
      $passwd_patt = "/^[A-Za-z0-9' -]{1,15}$/";
		
     if(preg_match("/username/i",$field))
      {
        if(!preg_match($name_patt,$value))
        {
          $error_array[] = "$value is not a valid Username";
        }
      }
      if(preg_match("/email/i",$field))
      {
        if(!preg_match($email_patt,$value))
        {
          $error_array[] = "$value is not a valid Email";
        }
      }

    if(preg_match("/password/i",$field))
      {
        if(!preg_match($passwd_patt,$value))
        {
          $error_array[] = "$value is not a valid password";
        }
      }
    }
  }
  if(@sizeof($error_array) > 0)
  {
    $message = "<ul style='color: red; list-style: none' >";
    foreach($error_array as $value)
    {
      $message .= "<li>$value</li>";
    }
    $message .= "</ul>";
    echo $message;
    @extract($clean_data);
    include("form.inc");
    exit();
  }
  else
  {
    $cxn = mysqli_connect($host,$user,$passwd,$dbname)
             or die("Couldn't connect to server");
    //foreach((array)$clean_data as $field => $value)
    $clean_data = array();
    foreach($_POST as $field => $value)
    {
      $clean_data[$field] = mysqli_real_escape_string($cxn,$value);
    }
	 $date = date("Y-m-d H:i:s");
     //echo date("Y/m/d H:i:s",$date);
    //$sql = "INSERT INTO CustomerPhone (last_name,first_name,middle_name,phone)
    //VALUES ('$clean_data[last_name]','$clean_data[first_name]',
    //       '$clean_data[middle_name]','$clean_data[phone]')";
	  ?>
	  <form method="post">
            <input type="hidden" name="username" value="<?php echo htmlspecialchars($_POST['username']); ?>">
            <input type="hidden" name="email" value="<?php echo htmlspecialchars($_POST['email']); ?>">
            <input type="hidden" name="password" value="<?php echo htmlspecialchars($_POST['password']); ?>">
            <input type="hidden" name="date" value="<?php echo $date; ?>">
       </form>
               
   <?php
	
 /* loop that displays the form */
    $sql = "INSERT INTO login (username,email,password,date) 
	VALUES ('".$clean_data['username']."','".$clean_data['email']."',
	'".$clean_data['password']."','$date')";
//echo $clean_data['username'];
    $result = mysqli_query($cxn,$sql)
            or die("Couldn't execute query");
	  $signUpMessage = 'Registered. Your Username is '.$clean_data['username'].' Password is'.$clean_data['password'].' ';  
	  echo  $signUpMessage;
    include("stored.inc");
  }
}
else
{
  include("form.inc");
}
?>

Comments

[takaboo]

"/^[A-Za-z0-9' -]{1,15}$/"はemailではなくpasswordでは？とりあえず質問が二つあるので分けて質問した方が良いですよ。あとテストができないのでform.incもさらしてみては？

[YukaSaku]

コメントありがとうございます。よくわからないまま解決したので、次回からそうします。

Answer 1

どうにもバグが取れないときは、物事を単純化しましょう。

$pass = "abcdefgh90123456";
var_dump(!preg_match("/^[A-Za-z0-9' -]{1,15}$/",$pass));
//bool(true)
$pass = "abcdefgh90123";
var_dump(!preg_match("/^[A-Za-z0-9' -]{1,15}$/",$pass));
//bool(false)
```ということで、正規表現は間違っていません。15文字を超えると`true`が返ります。
原因は他のところにあります。

emailのバリデーションは`$email_patt = "/^.+@.+\\..+$/";`であって、文字数のチェックは入っていませんので、それが原因では？

また、バリデーションエラーの時に入力が消えるのは、
```php
    echo $message;
    @extract($clean_data);
    include("form.inc");
```この部分で`extract`している、`$clean_data`っていうのが無いせいじゃないでしょうか？
`$good_data`の間違い？

Comments

[YukaSaku]

コメントありがとうございます！本当に感謝します。
私もここ、good_dataかと思い、入れ替えてみたんです。そしたら新たなPHPのwarning msgが出てきて、clean_dataに戻した次第です。
やりたかったことは、インプットデータを入力間違えした後も間違えたところだけを直せばいいように、そのままフォームに残したままにしたいということだったのですが、Validationのところはいじってなくて、他を色々といじくっているうちに、なぜか１５文字を超えるとエラーが表示されるようになりました。なぜだか本当によくわかりません。

ネットでサンプルプログラムを調べているともっとシンプルなフォームプログラムが沢山見つけられるので私が学校で習っているものは、なんて難しくややこしいプログラムなんだろうと思ってしまいます。

ありがとうございました。

Answer 2

やりたかったことは、インプットデータを入力間違えした後も間違えたところだけを直せばいいように、そのままフォームに残したままにしたいということが主だったのですが、Validationのところはいじってなくて、他を色々といじくっているうちに、なぜか１５文字を超えるとエラーが表示されるようになりました。なぜだか本当によくわかりません。

ご協力ありがとうございました。