PHPでMYSQLを操作時に重複してしまう

PHPからMySQLサーバーのuserテーブルでuser_idの重複があれば、書き込みをスキップするという流れで書いているのですがuser_idが重複しても書き込まれてしまいます。

問題のコード↓

id	user_id
1	1234
2	1234

重複する

PHP
1function register($srv, $uid) {
2    $query = <<<QUERY
3INSERT INTO `user` (`user_id`)
4            SELECT
5            $uid
6            FROM DUAL
7            WHERE NOT EXISTS (
8                SELECT * FROM `user`
9                WHERE `user_id` = $uid
10            )
11QUERY;
12
13    $sth = $srv->prepare($query);
14
15    $sth->execute();
16
17    return $sth;
18}

ヒアドキュメントで$uid変数を使わずにやると問題はないのですが(以下例）
問題ないコード↓

id	user_id
1	1234

※書き込みされない

PHP
1    $query = <<<QUERY
2INSERT INTO `user` (`user_id`)
3            SELECT
4            `1234`
5            FROM DUAL
6            WHERE NOT EXISTS (
7                SELECT * FROM `user`
8                WHERE `user_id` = `1234`
9            )
10QUERY;

何か特別に展開方法を変えないといけなかったりするのでしょうか？
よろしくご教授お願いします。

m.ts10806

2018/08/18 07:50

DBを扱う質問ではテーブル定義またはCREATE文とサンプルデータのINSERT文なども提示してください。コードだけでは具体的なアドバイスが難しく、再現確認をとってから回答をすることもあるからです。

papinianus

2018/08/20 03:32

user_idカラムのデータ型は文字列系ですよね？$uidって、空白とか見えない制御文字が入っている可能性はないのでしょうか？

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

回答2件

もし、「user_idが絶対に重複しない」というのであれば、データベースレベルでUNIQUEインデックスをかけておきましょう。そうすれば、どんなプログラムを書こうが、重複するデータを入れようとすればエラーになってくれます（INSERT IGNOREを使えば、エラーにはならずに「0行挿入」で戻ってきます）。

あと、MySQLでは` ... `と' ... 'の意味は違います。引用符は正しく使いましょう。そして、SQL文の中で変数展開するとSQLインジェクションの原因となりますので、プレースホルダを使いましょう。

投稿2018/08/18 04:52