ログイン認証 メールアドレスの重複確認

Question

ログイン認証で、入力データにミスや空白があればエラーを返すようにはできるんですが、メールアドレスが重複していた場合にエラーを出すプログラムが上手く動作しません。

方法としては、入力したフォームデータをGETでゲットして、それをデータベースで検索して、もしデータベースにデータがあれば、エラーを返すようにしたいんです。

###Code
このコードでOverlapEmail関数に$_POST['email']でフォームで入力されたデータを渡します。もし、呼び出し先の関数でスローされたらエラーメッセージを返します。

//	データベースにメールアドレスがあればエラーを返す。
try {
	$userModel = new \MyApp\Model\User();
	$userModel->OverlapEmail(['email' => $_POST['email']]);
} catch (\MyApp\Exception\DuplicateEmail $e) {
	$this->setErrors('email', $e->getMessage());
}

上のコードで呼ばれる関数が以下です。
呼び出す祭に渡されたデータ（メールアドレス）を元にデータベースを探しもしあれば$resにtrueが返ってきて、見つからなかったら、falseが返ってくるはずです。
下のコードにかいてあるように、渡されたデータとデータベースの結果を一時表示してます。渡されるデータは入力されたメールアドレスがきちんと渡ってきてます。問題なのはその次のデータベースの結果です。データベースに存在するメールアドレスでも存在しないメールアドレスでも、trueが返ってきてしまいます。
どこが問題なのでしょう？

	// 重複チェック(メールアドレス)
	public function OverlapEmail($values){
		$stmt = $this->db->prepare("select anumber from users where email = :email");
		$res = $stmt->execute([':email' => $values['email']]);
		echo var_dump($values['email']);
		echo var_dump($res);

		if ($res === false) {
			throw new \MyApp\Exception\DuplicateEmail();
		}
	}

Answer 1

PDOStatement::executeは成功するとtrue、失敗するとfalseなので、
成功して、0行が返って来ているので、trueなんじゃないでしょうか？

追記

    // 重複チェック(メールアドレス)
    public function OverlapEmail($values){
        $stmt = $this->db->prepare("select anumber from users where email = :email");
        $stmt->execute([':email' => $values['email']]);
        $row = $stmt->fetch();

        //echo var_dump($values['email']);
        //echo var_dump($row);

        if (!$row) {
            throw new \MyApp\Exception\DuplicateEmail();
        }
    }
```こんな感じです。

Comments

[ryo-dev]

そうですか、true が帰ってくるんですか。
でも、存在しているメールアドレスでも、true が帰ってくるんで結局どこかがおかしいですね

[shi_ue]

executeはただクエリーを発行するだけです。
結果が欲しければ、fetchしましょう。一行もfetch出来なければ存在しませんし、fetch出来れば存在します。

[ryo-dev]

データを取得したいのではなく、存在するかしないかをほしいので　execute　で大丈夫です。ただ、存在してもしなくても、結果が変わらないので、困ってます。

[Z-TALBO]

executeだと、SELECTはできたよ。
で終わりになりますよね。
データがあろうがなかろうが、文があってれば、SELECTはできてるよ。までなので、結果は変わらないんじゃないですか？
だからfetchで実際にデータがあるのかないのかを出してみて判定するって感じじゃないですか？

[ryo-dev]

$res = $stmt->fetch([':email' => $values['email']]);
文をこう書き換えたのですが、今度はデータベースに存在するメールアドレスでもしないメールアドレスでも false が返ってきます。

[shi_ue]

executeは必要です。
その後データをfetchして初めて、データが存在するかどうかが分かります。
サンプルは公式サイトにもたくさん載っているので、そちらを参考にしてください。
http://php.net/manual/ja/pdostatement.fetch.php

[ryo-dev]

＞shi_ue　さん
$res = $stmt->execute([':email' => $values['email']]);
を実行している時点で、前のSQL文にデータを挿入すると同時に検索も行い、$res　に　true　が、なければ　false　が入っている状態ですよね？

[shi_ue]

そこが違うところです。SELECT文の実行が成功すれば true 失敗すれば false です。SQLの文法が間違っていなければ、必ず true になると考えてよいと思います。
その上で fetch してデータがあるかどうか確認する、という流れです。

[shi_ue]

質問者さんの提示したソースだと、「from の users が user になってた」とかだと、false が返ると思いますよ。PDOの設定でエラーを例外で返すようにしていると、その前に例外が出てしまいますけど。

[ryo-dev]

ありがとうございます。
fetch　をすれば、確認できました。