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

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

ただいまの
回答率

87.93%

"Username already exists"と表示したい

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 938

score 44

登録画面で、クエリーエラーがあると、Couldn't execute queryと表示されるようになっています。これはきちんと機能しています。

が、ユーザーが入力をし、例えば同じユーザーネームがある場合に登録できない時など、Couldn't execute queryではなく、Username already existsと表示させたいです。

そのため、以下のコードを追加したところ、データベースに存在しないユーザーネームを登録しようとしてもはじかれるようになりました。

if (!empty($_POST['username'])  ) { // Confirm the username on the DB that user input 
            //$sql = "SELECT username FROM login";
            $sql = ("SELECT username FROM login WHERE username = '" . $_POST['username']);
           $rows = 1;
       }
  if ($rows == 1)
  {
      echo "Username already exists";
      include("form.inc");
      exit;
             } else {

全体のコードはこちらになります。

<?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");
$gen="";   

if(isset($_POST['submitted']) and $_POST['submitted'] == "yes")
{
    if(!isset($_POST['gen'])) 
    {
     $blank_data[] = $gen;
    }
  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 = "/^.+@.+\\..+$/";
      $phone_patt = "/^[0-9)(xX -]{7,20}$/";
      $passwd_patt = "/^[A-Za-z0-9' -]{1,60}$/";

    $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("/phone/i",$field))
      {
        if(!preg_match($phone_patt,$value))
        {
          $error_array[] = "$value is not a valid Phone";
        }
      }
      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";
        }
      }
      echo "<div class='field'>";  
    } 
   }

  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");
    $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");
    ?>

    <form method="post">
            <input type="hidden" name='$field' value="<?php echo htmlspecialchars($good_data['value']); ?>">
            <input type="hidden" name="gen" value="<?php echo $gen; ?>">
            <input type="hidden" name="date" value="<?php echo $date; ?>">
       </form>

   <?php
        if (!empty($_POST['username'])  ) { // Confirm the username on the DB that user input 
            //$sql = "SELECT username FROM login";
            $sql = ("SELECT username FROM login WHERE username = '" . $_POST['username']);
           $rows = 1;
       }
  if ($rows == 1)
  {
      echo "Username already exists";
      include("form.inc");
      exit;
             } else {


 /* loop that displays the form */
    $sql = "INSERT INTO login (username,phone,email,password,gen,date) 
  VALUES ('".$clean_data['username']."',
  '".$clean_data['phone']."',
  '".$clean_data['email']."',
  '".$clean_data['password']."',
  '".$clean_data['gen']."',
  '$date')";


    $result = mysqli_query($cxn,$sql)
            or die("Couldn't execute query");
    $signUpMessage = 
  'Your Username is '.$clean_data['username'].' <br>'.
  ' Phone is '.$clean_data['phone'].' <br>'.
  ' Email is '.$clean_data['email'].' <br>'.
 ' Gender is '.$clean_data['gen'].' <br>'.
  ' The registration date is '.$date.' <br>'.
  ' Keep your password for your security.';
    echo  $signUpMessage;
    include("stored.inc");
  }
  }
}
else
{
  include("form.inc");
}
?>

どのようにして表示させればいいでしょうか?
よろしくお願いします。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

checkベストアンサー

+1

INSERT IGNORE INTO で登録処理をはしらせ
rowCount()して、更新行が0件だったときに登録失敗させればよいでしょう

$username = filter_input(INPUT_POST,'username');
$pdo = new PDO($dsn, $user,$password);
if(!is_null($username)){
  $sql="INSERT IGNORE INTO login(username) values(?)";
  $stmt = $pdo->prepare($sql);
  $stmt->execute([$username]);
  if($stmt->rowCount()==0){
    echo "Username already exists";
  }
}

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/05/24 12:44 編集

    学校での課題でして、私もPDOを使って処理をしたいと思っているのですが、なぜかPDOを使うと先生に怒られます。なのでmysqliを使わないといけなかったのですが、方法が理解でき、解決できたのでこちらをベストアンサーにさせて頂こうと思います。ありがとうございます。

    キャンセル

  • 2017/05/24 12:46

    mysqliを利用するならaffected_rowsを参照下さい

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

    キャンセル

  • 2017/05/25 05:11

    このPHPマニュアルは、よくチェックするのですが、いまいちピンとこなくて、誰かに説明してもらわないと理解できないんです。でもこの件については、沢山例が載っていたので、参考に出来そうです。ありがとうございます。

    キャンセル

+1

SELECT文を実行している箇所がない、その上$rowsに1を入れているので常にif文の中に入っていますね。

また、SELECT文は構文エラーになると思います。ログ出力して確認しましょう。
さらにPOST値をそのままSQL文の組立に使うのは危険なのでやめたほうがいいですよ。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/05/24 02:24

    現在以下のように変えてみたら、Couldn't execute queryと表示されるようになり、この構文を入れてない状態に戻ってしまいました。

    if (!empty($good_data['username']) ) {
    $sql = "SELECT username FROM login WHERE username = '" . $good_data['username'];
    if ($result = $cxn->query($sql)) {
    $username_result = $good_data['username'];
    $rows =1;
    }
    if ($rows == 1)
    {
    echo "Username already exists";
    include("form.inc");
    exit;
    } else {

    select文を見直して、POSTをgood_dataに変えて、実行文を加えてみたのですが、$rowsはどのように対処したらいいのかわかりません。フラグを立てて、もしフラグが立ってればみたいな方法がいいと思ったのですが、どのようにしたらいいのでしょうか。他に方法があれば、その方法も教えていただければと思います。よろしくお願いします。

    キャンセル

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

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

関連した質問

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