DBへ登録されているアドレスが自分以外のアドレスと重複した場合、登録できないを実現するには

Question

とある会員サイト制作において、ユーザー登録ページで他の人のアドレスと重複して登録できないように下記のようにして対応していましたが、
現在プロフィール更新ページを作成しており、
例えば、住所は変えるがアドレスは変えない場合、
下記のままでは、自分のアドレスが既に登録されていることになり、弾かれてしまいます。

「DBへ登録されているアドレスが自分以外のアドレスと重複した場合、登録できない」
と、文章にするのは簡単なのですが、どういった工夫を加えると良いでしょうか？

$sql = 'select count(*) as cnt from user_data where email=?';
	$stmt = $pdo->prepare($sql);

	$arrParam = array();
	$arrParam[] = $email;
	$stmt->execute($arrParam);

	$row = $stmt->fetch(PDO::FETCH_ASSOC);
	if (0 < $row['cnt']) {
		$this->error['email_registered'] = '<div class="errorMessage">' . "{$email}は既に登録されています" . '</div>';
	}

参考にですが

input1.php
input2.php
preview.php
thanks.php

と最後で一括更新させる構造になっています。

---

ログインしている人のidはまだ確立していませんが、仮の数字を入れてテストすることでこれでいけました。

public function isRegisterd(){
		if (is_null(filter_input_array(INPUT_POST))) {
			return $this;
		}

		$email = filter_input(INPUT_POST, 'email');

		$dsn = 'mysql:dbname=test;host=localhost;charset=utf8';
		$user = 'root';
		$password = '';
		$option = array(PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION);
		$pdo = new PDO($dsn, $user, $password, $option);

		$sql = 'SELECT count(*) AS cnt FROM user_data WHERE email=? AND id != ログインしてる人のid';
		$stmt = $pdo->prepare($sql);

		$arrParam = array();
		$arrParam[] = $email;
		$stmt->execute($arrParam);

		$row = $stmt->fetch(PDO::FETCH_ASSOC);
		if (0 < $row['cnt']) {
			$this->error['email_registered'] = '<div class="errorMessage">' . "{$email}は既に登録されています" . '</div>';
		}
		return $this;
	}

Comments

[azurite2016]

input1.php
input2.php
preview.php
thanks.php

と３遷移目で更新するのですが、thanks.phpで初めてerrorがでては遅いといいますか、それなら初めのinput1.phpで登録できないことを知らせたいのです。

↑の様な事を追記してください。
また。それぞれのPHPでどんな処理をしたいのかも記載してほしいです。

[earnest_gay]

すいません。１つ１つ片づけていきたいので、現時点の問題点は自分のアドレス以外登録できなくすることです。

Answer 1

なんの為にRDBを使っているのか思い出しましょう。

PHPではなく、DBのユニーク制約にひっかけて、エラーにするのが良いかと思います。

Comments

[earnest_gay]

それじゃあダメなんです...

input1.php
input2.php
preview.php
thanks.php

と３遷移目で更新するのですが、thanks.phpで初めてerrorがでては遅いといいますか、それなら初めのinput1.phpで登録できないことを知らせたいのです。

Answer 2

user_data に id的なカラムはないのでしょうか。
もしあるなら、それを除いて存在チェックを
かけるといいと思います。
なかったら、なんか個人を特定できるカラムで
同様なことをすればいいと思います。

Comments

[earnest_gay]

なるほど！

PHPであればif(!empty)などと逆を指定することは出来ますがSQLではそういったものはないのでしょうか？

WHERE id = !2のような...
これは試してもダメでした...

[pac894398]

where id != 2 でいけると思います。

[earnest_gay]

できました！
結構参考になりそうです！
ありがとうございます