メールアドレスの重複チェックについて

Question

下記コードで、アドレスの重複チェックを表示することができました。
DBでuserdataというテーブルにemailというカラムがあり、test@testという値が格納されてます。

<?php
	$dsn = 'mysql:dbname=test;host=localhost;charset=utf8';
	$user = 'root';
	$password = '';
	try{
		$dbh = new PDO($dsn, $user, $password);
		$dbh->query('SET NAMES sjis');
		$sql = 'select * from userdata';
		foreach ($dbh->query($sql) as $row) {
			if($row['email'] == $_POST['email1']){
				echo ($_POST['email1'] ."は既に登録されています");
			}
		}
	}catch (PDOException $e){
		print('Error:'.$e->getMessage());
		die();
	}
$dbh = null;
?>

<tr>
	<td>E-mail</td>
	<td colspan="2"><input type="email" name="email1" maxlength="50" size="46" placeholder="[e-mail]" ></td>
	<td></td>
</tr>

とりあえず正常に動作するかのテストなのでブラウザの一番上に表示されている状態ですが
本当は下記のところに表示させたいのです。

<tr>
	<td>E-mail</td>
	<td colspan="2"><input type="email" name="email1" maxlength="50" size="46" placeholder="[e-mail]" ></td>
	<td>ここにエラー時の表示をさせたい</td>
</tr>

ということで、変数に代入しました。
$error = ($_POST['email1'] ."は既に登録されています");

<tr>
    <td>E-mail</td>
    <td colspan="2"><input type="email" name="email1" maxlength="50" size="46" placeholder="[e-mail]" ></td>
    <td><?php $error ?></td>
</tr>

スコープの関係で呼び出せてないだろうなとは思いますが、
まだ勉強中の身で、この例だと$errorはif内でしか呼び出せなかったことを学んだ記憶があります。

アドレスの重複チェックを定義したものを呼び出すには、どうしたら良いのでしょうか？

最終的にはクラスを作って呼び出したいのです...

<?php
class mail{
	function collation(){
		$dsn = 'mysql:dbname=test;host=localhost;charset=utf8';
		$user = 'root';
		$password = '';
		try{
			$dbh = new PDO($dsn, $user, $password);
			$dbh->query('SET NAMES sjis');
			$sql = 'select * from userdata';
			foreach ($dbh->query($sql) as $row) {
				if($row['email'] == $_POST['email1']){
					$error = ($_POST['email1'] . "は既に登録されています");
				}
			}
		}catch (PDOException $e){
			print('Error:'.$e->getMessage());
			die();
		}
		$dbh = null;
	}
}
?>

<tr>
	<td>E-mail</td>
	<td colspan="2"><input type="email" name="email1" maxlength="50" size="46" placeholder="[e-mail]" ></td>
	<td><?php mail::collation(); ?></td>
</tr>

イメージは大体あってると思うのですが、記述をどこか間違えていることは自覚していますが、どうすればよいのかわからず...

Answer 1

<?php
//$dsn = 'mysql:dbname=test;host=localhost;charset=utf8';
$dsn = 'mysql:dbname=test;host=localhost;charset=sjis';
$user = 'root';
$password = '';

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

try {
// try ~ catch で例外を受けるためには必要
	$option = array(PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION);

//	$dbh = new PDO($dsn, $user, $password);
	$dbh = new PDO($dsn, $user, $password, $option);
//        $dbh->query('SET NAMES sjis'); // <- ダメ $dsn で設定する
// 	$sql = 'select * from userdata';
	$sql = 'select count(*) as cnt from userdata where email=?';
	$stmt = $dbh->prepare($sql);

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

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

// 全件取得して、PHPでループなんて無駄以外の何物でもない
//	foreach ($dbh->query($sql) as $row) {
//		if ($row['email'] == $_POST['email1']) {
//			echo ($_POST['email1'] . "は既に登録されています");
//		}
//	}
} catch (PDOException $e) {
	$error = $e->getMessage();
}
// $dbh = null; // <- 不要 どうしても必要なら、末尾で。
?>

<tr>
    <td>E-mail</td>
    <td colspan="2"><input type="email" name="email1" maxlength="50" size="46" placeholder="[e-mail]" ></td>
	<td>
		<?php
		if (isset($error)) {
			echo $error;
		}
		?>
	</td>
</tr>
<?php
$dbh = null; // <- どうしても必要なら、ここ。
?>

Comments

[earnest_gay]

早速の回答ありがとうございます！
ただ、エラーがでてしまいます...

SQLSTATE[HY093]: Invalid parameter number: no parameters were bound

$email = filter_input(INPUT_POST, 'email');
と
$error = "{$email}は既に登録されています";
は
$email = filter_input(INPUT_POST, 'email1');
と
$error = "{$email1}は既に登録されています";
でなくてよいのでしょうか？

[退会済みユーザー]

email1なんてどこにあるの？情報を出さずにデバッグさせるのはやめて。

[退会済みユーザー]

ああ、name=email1なのね。適当に修正してやってみてください。

[sounisi5011]

executeメソッド（22行目）の引数に$arrParamが指定されていません。
そのせいで、プリペアドステートメントのプレースホルダ（?の箇所）に値が割り当てられていないのが原因かと思われます。

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

このコードを、以下のように修正してください。

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

または、以下のように見やすくしても良いかもしれません。

$stmt->execute(array($email));

[退会済みユーザー]

thanks!

[earnest_gay]

お二方ありがとうございます！
提示していただいたコードで正常に動作しました！

合間にphpリファレンス見ながら自分でも改良してみました...

お時間がありましたらアドバイス頂けると嬉しいです。

$dsn = 'mysql:dbname=test;host=localhost;charset=utf8';
$user = 'root';
$password = '';

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

try{
$dbh = new PDO($dsn, $user, $password);
$dbh -> setAttribute(PDO::ATTR_EMULATE_PREPARES, false);
$dbh -> setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);

$sql = 'SELECT count(*) AS cnt FROM userdata WHERE email=?';
$prepare =$dbh->prepare($sql);

//プレースホルダーに$mail1をバインド
$prepare->bindValue(1,$email1,PDO::PARAM_STR);
$prepare->execute();

$result = $prepare->fetchColumn();
if($result > 0){
echo "{$email1}は既に登録されています。";
}
}catch (PDOException $e) {
echo "Error:".$e->getMessage();
}

[退会済みユーザー]

指摘されたことをやってください。なんで echo をそこでやるかなぁー

[earnest_gay]

すいません...処理では表示をさせてはいけないんでしょうか？

あと...
クラス化しなければ
if (isset($error)) { echo $error; }
で呼び出せるのですが

クラス化してしまうと
if (isset($error)) { echo $error; }
で呼び出してDBに登録しているアドレスを入力してみても
表示されないんです...

[退会済みユーザー]

そこでecho したら　<td>に表示できないでしょ？

[退会済みユーザー]

なに？クラスかだと？今更そんな事実を出さんでくれ！頼むから情報を小出しにするのはやめておくれ。

[退会済みユーザー]

クラスにするにしても、メソッド内でechoはダメダメソースですね。

[earnest_gay]

すいません、、、
クラス化はまた別で質問してみようと思います。

一応、最終的にはこういう使い方をしようとしていたんです。

class mail{
function collation(){
$dsn = 'mysql:dbname=test;host=localhost;charset=utf8';
$user = 'root';
$password = '';

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

try{
$option = array(PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION);
$dbh = new PDO($dsn, $user, $password, $option);

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

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

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

} catch (PDOException $e) {
$error = $e->getMessage();
return $error ;
}
}
}

<td><?php echo mail::collation(); ?></td>

これで<td>に表示ができます。

何度もお付き合い頂いて本当にありがとうございます。

[退会済みユーザー]

そうそう。これなら問題なしですね。ただ…重複がないときは何もreturn しないの？エラーが出るわけではありませんが。static の方がいいかなー

[退会済みユーザー]

https://teratail.com/users/Masaya_Gotou#question
質問を放置しすぎ。解決しているものは閉じてください。

Answer 2

echo
とりあえず、変数がどこで変化しているか、var_dump等で、チェックするクセをつけるとイイですよ。

Comments

[earnest_gay]

ありがとうございます！

[sounisi5011]

ver_dumpではなく、var_dumpです。

[退会済みユーザー]

失礼＾＾；修正しました。