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

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

質問する

ただいまの
回答率

90.86%

  • PHP

    18160questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

会員登録システムの実装について

解決済

回答 1

投稿 ・編集

  • 評価
  • クリップ 0
  • VIEW 568
退会済みユーザー

退会済みユーザー

前提・実現をしたいシステム

PHPを勉強している者です。
http://noumenon-th.net/programming/2016/02/26/registration/
http://noumenon-th.net/programming/2016/03/03/login/
を全て参考にさせてもらい、会員登録・ログインシステムを実装することができました。
参考にさせていただいたのですが、自身では解決できない問題がありましたので、質問させていただきます。

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

http://noumenon-th.net/programming/2016/02/27/registration2/
の「registration_mail_check.php（メール確認・送信）」のコードについてなのですが、
そのままの状態だと、登録されているメールアドレスを弾くことができず、多重登録することができてしまうようです。
（元のコード（以下）の場合だとその処理の部分がコメントアウトされ、不明）

これを同じメールアドレスで多重登録ができないようにしたいです。
当方解決しようと試みましたが、まだまだ駆け出しの身のためどこをどう改良して良いのかわかりません。
申し訳ないのですが、サンプルコードとご説明をよろしくお願いします。

問題のコード抜き出し

}else{
    //POSTされたデータを変数に入れる
    $mail = isset($_POST['mail']) ? $_POST['mail'] : NULL;

    //メール入力判定
    if (empty($mail)){
        $errors['mail'] = "メールが入力されていません。";
    }else{
        if(!preg_match("/^([a-zA-Z0-9])+([a-zA-Z0-9\._-])*@([a-zA-Z0-9_-])+([a-zA-Z0-9\._-]+)+$/", $mail)){
            $errors['mail_check'] = "メールアドレスの形式が正しくありません。";
        }

        /*
        ここで本登録用のmemberテーブルにすでに登録されているmailかどうかをチェックする。
        $errors['member_check'] = "このメールアドレスはすでに利用されております。";
        */
    }
}

補足情報

PHP Version 5.6.25

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

more_vert2 件の質問への追記・修正依頼

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

  • A.Ichi

    A.Ichi

    2016/09/17 00:04

    if ($mail == '') {は、if (empty($mail)){ がよろしいかと、上でNULL入れてますし

    キャンセル

  • 退会済みユーザー

    退会済みユーザー

    2016/09/17 00:06

    なるほど...。貴重なご意見ありがとうございます。 現物のコード・こちらのページともに変更させていただきました。 ぜひ多重登録の件もご回答、よろしくお願いします。

    キャンセル

回答 1 件

link

checkベストアンサー

+1

ログイン処理のところで以下のような記述があるかと思いますが、これを判定処理のところに入れるといいでしょう。 

//アカウントで検索
        $statement = $dbh->prepare("SELECT * FROM member WHERE account=(:account) AND flag =1");
        $statement->bindValue(':account', $account, PDO::PARAM_STR);
        $statement->execute();

        //アカウントが一致
        if($row = $statement->fetch()){

できれば上記のヒントのみでトライしてもらった方が勉強になるかと思います。

どうしても上手くいかない場合は以下で試してみてください。 

}else{
    //POSTされたデータを変数に入れる
    $mail = isset($_POST['mail']) ? $_POST['mail'] : NULL;

    //メール入力判定
    if (empty($mail)){
        $errors['mail'] = "メールが入力されていません。";
    }else{
        if(!preg_match("/^([a-zA-Z0-9])+([a-zA-Z0-9\._-])*@([a-zA-Z0-9_-])+([a-zA-Z0-9\._-]+)+$/", $mail)){
            $errors['mail_check'] = "メールアドレスの形式が正しくありません。";
        }else{
            // ここで本登録用のmemberテーブルにすでに登録されているmailかどうかをチェックする。
            $statement = $dbh->prepare("SELECT * FROM member WHERE mail=(:mail)");
            $statement->bindValue(':mail', $mail, PDO::PARAM_STR);
            $statement->execute();
            if($row = $statement->fetch()){
                $errors['member_check'] = "このメールアドレスはすでに利用されております。";
            }
        }
    }
}

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/09/17 01:19

    頂いたヒント・回答を元に、試行錯誤して完成することができました!
    ありがとうございました。

    キャンセル

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

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

関連した質問

  • 受付中

    PHPのメール送信について

    こんにちは <form method="post" action="email.php">    <input type="text" name="email">    <inpu

  • 解決済

    phpのテンプレート「Twig」の使い方についてお聞きしたいです！

    お世話になっております。kaoriと申します。 現在お問合せフォームをPHPとTwigを使って作成中なのですが詰まってしまいました。 【最終的にやりたいこと】 入力フ

  • 解決済

    PHP 会員登録画面について

    メールアドレスとパスワードが正しければ会員登録できるプログラムを作っていますがうまくいきません。vardumpで$errorの中身を確かめるとどうやっても何も入らないので、配列の部

  • 解決済

    会員制サイト：メールアドレスでログイン

    前提・実現したいこと Qiita:PHPとMySQLで新規登録とログインを実装する(PDO使用)を参考に、PHPで会員制サイトを作っています。 このサイトで作れるものでは、ID

  • 解決済

    Undefined indexエラーを消したい。

    前提・実現したいこと phpでお問い合わせホームを作っています。 Notice: Undefined indexのエラーを出ないようにしないです。 発生している問題・エラ

  • 解決済

    ユーザーの入力した文字列が日付として有効かどうかをチェックする方法

    PHPのプログラミングについての質問です。 日付を入力するフォームがあり、送信された文字列が日付として有効かどうかをチェックしています。 年月日を別々に入力させてcheck

  • 解決済

    【PHP】ログイン機能の実装

    PHP初心者です。 PHPの勉強がてら、会員制のページを作成してます。 新規会員登録機能は実装できていますが、 ログイン機能の実装がうまくいきません。 やりたいことを実

  • 解決済

    配列が空か否かの真偽判定

    お世話になります。 配列が空か否かの判定について通常emptyを使用すると思いますが 以下のような多次元配列で最後の配列の要素が全体として存在するか否かを判定するsmart

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

  • PHP

    18160questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

PHP求人の業界動向 >

SNSアカウントでログイン

  • Facebookでログイン
  • Twitterでログイン
  • Googleでログイン
  • Githubでログイン
  • Hatenaでログイン

teratailアカウントでログイン

思考するエンジニアのためのQ&Aサイト「teratail」について詳しく知る