データベースにメールアドレスが登録されているかどうかで値がnullになってしまい困ってます

DB名前.java
1public String seldb(String Mail) {/*メールアドレスが登録されているかどうか*/
2		try{
3
4			rset=stmt.executeQuery("SELECT mail from staff where mail='"+Mail +"'");/
5
6			while(rset.next()){
7				if (Mail == rset.getString(Mail)) {/*一致したら*/
8					return rset.getString(Mail);
9				}
10			}
11			return "";/*一致しなかった処理*/
12
13		} catch (Exception e) {/*ここは変更点無し*/
14			e.printStackTrace();
15			return  null;
16		}
17
18
19	}

※Mailは引数 mailはテーブル名Staffのカラム名

修正したい点登録されているかどうか確認せずに何でも登録されてしまうので
治したいです

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

回答3件

ベストアンサー

違和感を感じる点を２つ

１．目的のメールアドレスが入っているレコードをＤＢから取り出しているのに、あらためてそのメールアドレスが一致するか確認しようとしている

目的のメールアドレスの有無を調べたいだけなら、rset.next() を1回だけ判断したらよさそうです

２．getString()の使い方

if (Mail == rset.getString(Mail))

これでは 目的のメールアドレスの列名 を取り出そうとしているので、getString(Mail)の結果は常にnullになってしまうと思います

この場合、正しくは

if (Mail == rset.getString("mail"))

if (Mail == rset.getString(1))

ではないでしょうか？
（といっても、前述の通りこのチェックは無用に思いますが）

ついでにいうと return rset.getString(Mail); も return rset.getString("mail"); ですかね

投稿2015/12/18 01:17

編集2015/12/18 01:19

takito

総合スコア3111

kentros511

2015/12/18 01:26

気づきませんでしたありがとうございます

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

java
1if (Mail == rset.getString(Mail)) {/*一致したら*/

これは必要でしょうか？
rset=stmt.executeQuery("SELECT mail from staff where mail='"+Mail +"'");/
この時点で、mail が引数（Mail）と同じmail を取得するとなっています。
この場合、select の結果は、Mail と同じ mail か null が返ってきます。
null を返したくないのであれば
"SELECT COALESCE(mail, 'nothing') from staff where mail='"+Mail +"'"
上記の様に変更すれば、mail が null だった場合に「nothing」が
null 以外の場合には、mail のデータが返ってくると思います。

すいませんDB2は使ったことがありませんので、COALESCEで良いかわかりません。

投稿2015/12/18 01:20