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

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

ただいまの
回答率

88.34%

重複データをはじいて、メッセージを表示したい

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 2,560

YukaSaku

score 43

フォーム登録画面で、もし同じユーザーネームが登録されたら、”○○ is already exist.” と表示させたいです。現在は、DBのユーザーネームをuniqueにして、ただ単にクエリーエラーにさせてるだけです。
それをきちんとプログラム内で、重複データ用のエラーを表示させたいです。

現在の動くプログラムはこんな感じです。

<?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))
    {
              $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>";
    //extract($good_data);
       echo $message;
   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}$/";

    $field = htmlspecialchars($field, ENT_QUOTES);
        $value = htmlspecialchars($value, ENT_QUOTES);

     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='$field' value="<?php echo htmlspecialchars($good_data['value']); ?>">
            <input type="hidden" name="date" value="<?php echo $date; ?>">
       </form>

   <?php
  //$status = "none";
 /* 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");
}
?>


そして、エラーメッセージを出すために$clean_dataをinsert into したあとに、この条件文を入れました。

// userテーブルからemail項目を条件に1件取得するSQL文 
$sql = "select * from login where username = :username"; 
// SQL発行準備 
$stmt = $cxn->prepare($sql); 
// SQL発行。条件のusernameは、引数$usernameの値。 
$stmt->execute(array(":username" => $username)); 
// loginテーブルから1件データ取得 
$login = $stmt->fetch(); 
// $usernameが存在していればtrue、そうでなければfalseを返す 
return $login ? true : false;
  if ($login == "true"){
          echo ''.$username.' is already exist.' ;
      }

そして、DBに既に存在するユーザーネームを打ち込むと、

Fatal error: Uncaught Error: Call to a member function execute() on boolean in /home/vol14_5/ihostfull.com/uoolo_19138640/htdocs/PHP/Ex5/E5/register.php:128 Stack trace: #0 {main} thrown in /home/vol14_5/ihostfull.com/uoolo_19138640/htdocs/PHP/Ex5/E5/register.php on line 128

と表示されます。

どのようにしたらいいでしょうか?

  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

checkベストアンサー

+3

現在の動くプログラム

では mysqliモジュールを使用していますが、

$clean_dataをinsert into したあとに

追加したプログラムは PDOモジュールを使用しているかのような文になってることが、
エラーの原因です。


それ以前に、今、試みている方法では正しく

重複データ用のエラーを表示

することはできません。

$clean_dataをinsert into したあと

select * from loginを実行すると必ず1件以上のデータが存在するため、
常にエラーメッセージが表示されることになるからです。

代わりに、

DBのユーザーネームをunique

のままにした上で、以下のようにすると良いでしょう。

    // IGNORE キーワードを追加
    $sql = "INSERT IGNORE INTO login (username,email,password,date) 
  VALUES ('".$clean_data['username']."','".$clean_data['email']."',
  '".$clean_data['password']."','$date')";

    $result = mysqli_query($cxn, $sql)
            or die("Couldn't execute query");

    $affectedRows = mysqli_affected_rows($cxn);
    if ($affectedRows === 1) {
        $signUpMessage = 'Registered. Your Username is '.$clean_data['username'].' Password is'.$clean_data['password'].' ';  
        echo  $signUpMessage;
    } else if ($affectedRows === 0) {
        echo $clean_data['username'] . ' is already exist.' ;
    } else {
        die('Something is wrong...');

    }


https://dev.mysql.com/doc/refman/5.6/ja/insert.html

IGNORE キーワードを使用した場合、(中略)重複キーエラーは生成しません。

http://php.net/manual/ja/mysqli.affected-rows.php

直前の MySQL の操作で変更された行の数を得る

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/05/12 03:00 編集

    とてもわかりやすい説明とサンプルをありがとうございます。解決しました。
    そうなんです、PDOとmysqliが頭の中でかなり混乱しています。
    ちゃんと分けて考えられるよう、勉強します。

    キャンセル

+1

エラーメッセージを読んでみてください。register.php の 128 行目で boolean の execute を呼んでいるが、そんなものはないと書いています。

PDO::prepare

返り値 ¶

もしデータベースサーバーが正常に文を準備する場合、 PDO::prepare() は PDOStatement オブジェクトを返します。 もしデータベースサーバーが文を準備できなかった場合、 PDO::prepare() は FALSE を返すか PDOException を発行します (エラー処理 の方法に依存します)。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/05/12 03:02

    ありがとうございます。PDO、もっと詳しく勉強します。

    キャンセル

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

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

関連した質問

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