DBに登録す前に主キーが重複するかどうかをチェックしたい

(一応JAVAでやっているので)

メールアドレスが主キーになっているテーブルがあります。
そのテーブルにデータを登録するとき、入力されたメールアドレスが重複になるかどうかをチェックしたいです。

適切なやり方は絶対にあると思いますが、私のやり方は...

既に登録済みのメールアドレスを登録した場合SQLExceptionが発生するため、Exception messageを正規表現使って、
重複した値（メールアドレス）を入力画面にreturnするようにしています。

// できれば他の入力された値と一緒にバリテーションチェックなど行って重複したらすぐ返したいですね。
try {
     // 登録処理
}catch (SQLException e) {
    // e.getMessage()　から正規表現使って重複した値（メールアドレス）を画面に返す
}

この方法はやるべきか、やらない方がいいかを教えてください。
他にもっとカッコイイ方法などありましたら、ぜひ教えてください。

m.ts10806

2019/01/31 06:21

何が問題でしょうか。できてるならそれでいいのでは？

rev_timo

2019/01/31 06:30 編集

Exception メッセージは重複だけには限らないため、他にもいろいろなExceptionが発生する場合もありますね。そこで、私は if () で重複エラーかを調べたり、重複だったら正規表現を使って重複した値を取得したりする処理を書くと (適切かどうか分かりませんが) なにか変な気がします。そもそもこのやり方は一般的なやり方であるかどうかも分からないので、アドバイスとかいただきたいです…

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

回答3件

ベストアンサー

あくまで私見ですが。
Exceptionも一概に悪いとは言えません。

今回使えるか別としてDeplicateKeyExceptionのようなものなど、用途にあわせたExceptionをthrowしてcatchするようにしたら使い分けもできるかもしれませんし。(自身でそれ用のExceptionを定義してもいいし)

重複を「異常」とするかどうかは設計とか仕様にもなってくるので、実現方法がひとつでない以上は「適切」「ベストプラクティス」はないと思います。

投稿2019/01/31 06:33

m.ts10806

総合スコア80850

rev_timo

2019/01/31 06:49

回答ありがとうございます。

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

ユーザーと対話しながら登録するプログラムでしたら, SELECT して確認してから INSERT します.
Exception を解析しなければならないような事案とは思えません.

投稿2019/01/31 17:32

jimbe

総合スコア12646

rev_timo

2019/02/01 04:06 編集

回答ありがとうございます。例えばですが、大量のデータを一括登録した場合は結構遅くなってしまいますね......

jimbe

2019/02/01 08:40

そうですね, バッチ処理の場合はその為に用意されているデータベースの機能を使うようにしますので, java 側ではそのための前準備と結果を表示する程度になると思います.

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

意見
ひとつのSQL文でレコードを挿入するかしないかを判断させたい。
例外が発生すると処理に時間がかかる。また、例外で判定しようとすると、正常、異常の区別が曖昧になりコードがわかりにくくなると思うからです。

SQL
INSERT SELECT、WHERE NOT EXISTSと、ダミーテーブル(レコードが確実に1件だけ存在するテーブル)を使用します。

SQL
1INSERT INTO <更新対象テーブル> (<主キー項目>, ... ) 
2SELECT '<挿入する主キーの値>', ... FROM DUAL
3WHERE NOT EXISTS (SELECT 1 FROM <更新対象テーブル> WHERE <主キー項目> = '<挿入する主キーの値>');