PHPでDBに重複してデータが登録された場合に何が重複したのか教えてくれるようにしたいです。

前提・実現したいこと

下記のプログラムのとおりになにか重複したらウィンドウのアラートで教えてくれるようにしましたが、アラートで何が重複したか教えてくれるようにしたいです。

if文で項目ごとに処理をするのはわかりましたが、どのような文を作ればいいのかわかりません。

どうかよろしくお願いします。

該当のソースコード

<?php
$dsn = 'mysql:dbname=example;host=localhost';
$user = 'root';
$password = 'shapshap';
try{
	$dbh = new PDO($dsn, $user, $password);
	$stmt = $dbh->prepare('insert ignore into example (C_nam, C_nm, C_phn, C_add) values (:C_nam, :C_nm, :C_phn, :C_add)');
	$stmt->execute(array(':C_nam' => $_GET['C_name'], ':C_nm' => $_GET['C_num'], ':C_phn' => $_GET['C_phon'], ':C_add' => $_GET['C_sa']));
	if($stmt->rowCount()==0);
	$alert = "<script type='text/javascript'>alert('データは既存のものです。');</script>";
	echo $alert;	
}
catch (PDOException $e){
	print('Error:'.$e->getMessage());
	die();
}
$dbh = null;
?>
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="ja" lang="ja">
<head>
	<meta charset="UTF-8">
	<meta name="viewport" content="width=device-width">
	<title>顧客登録</title>
	<link rel="stylesheet" href="ser_style.css">
</head>
<body>

<div class="content">
	<h1>顧客情報を登録しました。</h1>
	<form action="c_regi.php" method="GET">
		<div class="control">
			<input type="submit" value="顧客登録へ">
		</div>
	</form>
</div>
</body>
</html>

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

回答3件

例えば、ログイン認証で
「パスワードが違います」「ログインIDが違います」など個別に出していたらそのうち突破されてしまいます。
「何が重複されているか」を他ユーザーに知らせることはセキュリティ的によくない実装です。「NG」のみ返すべきです。

やりたいのは別として「設計としてまずい」ことは認識してください。

※厳密に言えば「重複している」と返すのも要件によっては微妙。存在を知らせることになりますからね

でないなら、結局はSELECTで項目別にチェックしていくしかありません。
前の質問で既に回答した通り。
INSERT結果では「登録できたかどうか」しか判断できませんので。

投稿2021/01/27 04:34

m.ts10806

総合スコア80875

noizi.

2021/01/27 04:51

解答ありがとうございます。つまりはセキュリティを重視するならアラートのメッセージを「登録できませんでした」のようにあえて曖昧な文を出力したほうがいいということですか？

m.ts10806

2021/01/27 05:26 編集

要件次第です。サービスとして何を重んじるか、守るか、何が問題となり得るか。

noizi.

2021/01/27 05:44

なるほど、分かりました、ご指摘ありがとうございました。

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

ベストアンサー

なにが・・・の定義がわかりませんが
それをするなら事前にselectしてヒットしたものを表示するのが手っ取り早く確実でしょう

投稿2021/01/27 04:25

yambejp

総合スコア116724

INSERT IGNORE INTOでrowCount()==0の場合にアラートを出しているわけですから、その際にINSERTしようとしたデータの主キーが重複しているわけです。それを素直に表示するだけでは?

投稿2021/01/27 05:32

ockeghem

総合スコア11705

noizi.

2021/01/27 05:47

回答ありがとうございます。重複に反応する命令が必要ということですか？

ockeghem

2021/01/27 06:00

いえ、あなたの書いたコードに「重複に反応する命令」があります。下記の部分です。ただ、今見たらバグがありますね。 if($stmt->rowCount()==0); $alert = "<script type='text/javascript'>alert('データは既存のものです。');</script>"; echo $alert; この箇所ですが、以下のように書くべきでしょう。 if($stmt->rowCount()==0) { $alert = "<script type='text/javascript'>alert('データは既存のものです。');</script>"; echo $alert; }

noizi.

2021/01/27 10:54

ご指摘ありがとうございます。エラーが出ていなかったので気づきませんでした。

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

あなたの回答