メールアドレスの重複登録防止機能の実装

前提・実現したいこと

PHPとMySQLを使ってサインアップするシステムを作っています。
メールアドレスの重複登録防止機能の実装。

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

メールアドレスを使用し重複登録の防止機能を実装していますが上手くいきません。
通常にサインアップする際は登録も出来ます。データベースにも情報が保管されてますが、
この状態でもう一度同じメールアドレスを使用してサインアップするとエラー出ずに重複登録できる状態です。

該当のソースコード

ソースコード

<?php

session_start();
require('./dbconnect.php');

if (empty($error)) {
    $member=$db->prepare('SELECT COUNT (*) AS cnt FROM members WHERE email=?');
    $member->execute(array($_POST['email']));
    $record=$member->fetch();
    if ($record['cnt'] > 0) {
        $error['email'] = 'duplicate';
    }
}

  if(empty($error)){
    $_SESSION['join'] = $_POST;
    header('Location: check.php');
    exit();
  }
}
?>

~dbconnect.php~

<?php

try {
$db = new PDO('mysql:host=hostname;dbname=dbname;charset=utf8','username','pwd');
}catch(PDOException $e){
 exit('データベース接続失敗。'.$e->getMessage());
}

 ?>

テーブル定義
テーブル：member
カラム：
Id int PK設定
name varchar
email varchar
psw varchar
created datetime
modified timestamp

試したこと

$member=$db->prepare("select * from members where email = :email limit 1");
とか試しましたが改善なしです。
お力添え頂ければ幸いです。

補足情報（FW/ツールのバージョンなど）

PHP
Mysql(5.7)

ここにより詳細な情報を記載してください。

sola-msr

2019/05/15 17:04

いろいろと指摘する箇所はあるのですが、とりあえずメールアドレスの登録はサインアップ時ではなくてですか？

Orlofsky

2019/05/15 21:22

コードは https://teratail.com/help#about-markdown の [コードを入力] に修正してください。

JakeN

2019/05/15 22:20

ご返信ありがとうございます！間違えました。サインアップで間違えないです。

行動規範の内容に同意します

回答3件

まず「重複を許さない」のであればその項目をテーブル定義時にPKにするのが先決と思います。
重複チェックはその次です。

投稿2019/05/15 21:45

m.ts10806

総合スコア80850

JakeN

2019/05/15 22:30 編集

ご返信ありがとうございます。テーブル定義していてPKもしていますのでphpからmysqlのテーブルにデータ登録される状態ではあるんですが重複していてても弾いてくれない状態です。説明不足で申し訳ございません。

m.ts10806

2019/05/15 22:36

メールアドレスPKにしてたらINSERTできないですよ。SQLで重複エラーが返されます。設定見直してみては。いずれにしてもテーブル定義を質問本文に追記してください

退会済みユーザー

2019/05/15 22:51

プライマリじゃなくて、ユニークじゃない？

m.ts10806

2019/05/15 23:00

あ、確かに。 idカラムがあってAI指定ならPKしててもIDが被らないからメールアドレス同じでも登録できますね。ユニーク必要ですね。

JakeN

2019/05/16 04:32

ご回答ありがとうございます。テーブル定義追記しましたので確認頂ければ幸いです。

m.ts10806

2019/05/16 04:43

>テーブル定義していてPKもしていますので私の回答部分があまり伝わってないように思いますが、メールアドレスを重複不可にしたいのでしたらメールアドレスにキーを設定しましょう、という意味です。提示内容ですとId にしかキーが設定されていないので、そこが問題です。te2jiさんの補足通り、ユニークキーを設定してください。

行動規範の内容に同意します