質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.90%

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

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 1,316

YukaSaku

score 43

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

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

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

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

<?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");
}
?>
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • takaboo

    2017/05/06 11:45

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

    キャンセル

  • YukaSaku

    2017/05/09 02:12

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

    キャンセル

回答 2

+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 = "/^.+@.+\\..+$/";であって、文字数のチェックは入っていませんので、それが原因では?

また、バリデーションエラーの時に入力が消えるのは、

    echo $message;
    @extract($clean_data);
    include("form.inc");

この部分でextractしている、$clean_dataっていうのが無いせいじゃないでしょうか?
$good_dataの間違い?

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/05/09 02:09

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

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

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

    キャンセル

check解決した方法

0

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.90%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る