会員登録で新規登録者だけを登録させたい

Question

会員登録で新規登録者だけを登録させたいです。
事前にemailが登録されてるかでcountをとり、count数が0なら新規登録、1ならなにもしないという処理にしようとしています。
しかし、カウント０でもデータベース上には登録されていなく、うまく処理せれてないようです。
書き方などおかしいところがあれば指摘して欲しいですl。

//データベース接続
	$conn = dbConnect();

	$sql = "SELECT COUNT(*) AS count FROM user_date WHERE email ='$encrypt'";
	$stmt = $pdo->query($sql);
	$result = $stmt->fetchAll(PDO::FETCH_ASSOC);
	foreach ($result as $row) {
		$count = $row['count'];
	}

	echo $count;

	if($count === 0){
		$sql = "INSERT INTO user_date (name, password, email) VALUES (?, ?, ?)";
		$stmt = $pdo->prepare($sql);
		$stmt->bindValue(1, $name, PDO::PARAM_STR);
		$stmt->bindValue(2, $hash, PDO::PARAM_STR);
		$stmt->bindValue(3, $encrypt, PDO::PARAM_STR);
		$stmt->execute();
		$pdo = null;
		echo "会員登録が完了しました。<br>";
		echo "<a href='login.html'>ログイン画面</a>";
	}

	if($count === 1){
		echo "すでに登録済みです";
		echo "<a href='login.html'>ログイン画面</a>";
	}
		$pdo = null;

Answer 1

提示されたコード自体あまり適切でないのでフローには触れません
今回の原因は、
if($count === 0){が意図した挙動になっていないからです。
echo $count;をvar_dump($count);に変更して確認してみてください。
string になっていると思います。

Comments

[退会済みユーザー]

確認したところ文字になっていました。intに直したところうまくいきました！色々おかしいみたいですのできちんと勉強し直そうと思います。

[退会済みユーザー]

原因の原因に関して触れていませんでした＾＾；
PDO::ATTR_EMULATE_PREPARES が ftrue だと、select の結果が string で返ります。

この辺を読むといろいろ書いてあります。
https://qiita.com/mpyw/items/b00b72c5c95aac573b71#select-%E3%81%AE%E7%B5%90%E6%9E%9C%E3%82%92%E5%8F%96%E5%BE%97%E3%81%99%E3%82%8B

Answer 2

データベース、SQLの観点から

user_dateテーブルの構造が不明なので、回答は一般的な解決策になります。

  if($count === 0){
        $sql = "INSERT INTO user_date (name, password, email) VALUES (?, ?, ?)";

まずemailには

 $stmt->bindValue(3, $encrypt, PDO::PARAM_STR);

ではなく

 $stmt->bindValue(3, $email, PDO::PARAM_STR);

ではないでしょうか。

次に

password　にhashを代入していますのが気になります。
テーブルのpasswordの型とPHP側の変数の型は一致していますか？

あと参考になりそうな記事をどうぞ
ttps://stackoverflow.com/questions/28281114/sql-password-field-does-not-work-when-i-include-encryption

Comments

[退会済みユーザー]

データベースにハッシュ化したパスワード、暗号化したメールアドレスをいれたかったので、、、書き方がおかしいかもですね。勉強します。

[kawakawa2018]

＞データベースにハッシュ化したパスワード、暗号化したメールアドレスをいれたかった

変数名を工夫されると、私のような誤解は発生しないと思います。

[退会済みユーザー]

変数名をを工夫したり、コメントを書くなど以後読み手を考えて質問しようと思います。

Answer 3

emailにunique属性をつけて

  $sql = "INSERT IGNORE INTO user_date (name, password, email) VALUES (?, ?, ?) ";

で処理すればよいでしょう。
その上で、rowcountで更新の有無を確認して下さい

Comments

[退会済みユーザー]

なんとかできました。回答ありがとうございます。