既に登録されている情報と全く同じ情報の登録を無効化したい

Question

[前提]
言語：Java
フレームワーク：struts2

ショッピングサイトの宛先登録を製作中。

[問題]
タイトルにもあるように、入力した宛先情報が既に登録されている場合、その登録を無効化、要するに登録出来ないようにしたいのですが、うまくいかなくて詰まっています。

public List<String> doubleCheck(AddressDTO addressDTO) throws SQLException {

		DBConnector db = new DBConnector();
		Connection con = db.getConnection();
		List<String> errorMsg = new ArrayList<String>();

		String sql = "select count(id) as count from destination_info where user_id=? and family_name=? and first_name=? and family_name_kana=? and first_name_kana=? and email=? and tel_number=? postal_code=? user_address=?";

		try {
			PreparedStatement ps = con.prepareStatement(sql);

			ps.setString(1,addressDTO.getUserId());
			ps.setString(2,addressDTO.getFamilyName());
			ps.setString(3,addressDTO.getFirstName());
			ps.setString(4,addressDTO.getFamilyNameKana());
			ps.setString(5,addressDTO.getFirstNameKana());
			ps.setString(6,addressDTO.getEmail());
			ps.setString(7,addressDTO.getTelNumber());
			ps.setString(8,addressDTO.getPostalCode());
			ps.setString(9,addressDTO.getAddr11());

			ResultSet rs = ps.executeQuery();

			if(rs.getInt("count") > 0) {
				errorMsg.add("こちらの宛先は既に登録されています");
			}

		}catch(Exception e) {
			e.printStackTrace();
		}finally {
			con.close();
		}

		return errorMsg;

	}

AddressDTOのコード

public class AddressDTO {

	private int addressId;
	private String userId;
	private String familyName;
	private String firstName;
	private String familyNameKana;
	private String firstNameKana;
	private String postalCode;//郵便番号
	private String addr11;//住所
	private String telNumber;
	private String email;

	public int getAddressId() {
		return addressId;
	}

	public void setAddressId(int addressId) {
		this.addressId = addressId;
	}

	public String getUserId() {
		return userId;
	}

	public void setUserId(String userId) {
		this.userId = userId;
	}

	public String getFamilyName() {
		return familyName;
	}

	public void setFamilyName(String familyName) {
		this.familyName = familyName;
	}

	public String getFirstName() {
		return firstName;
	}

	public void setFirstName(String firstName) {
		this.firstName = firstName;
	}

	public String getFamilyNameKana() {
		return familyNameKana;
	}

	public void setFamilyNameKana(String familyNameKana) {
		this.familyNameKana = familyNameKana;
	}

	public String getFirstNameKana() {
		return firstNameKana;
	}

	public void setFirstNameKana(String firstNameKana) {
		this.firstNameKana = firstNameKana;
	}

	public String getPostalCode() {
		return postalCode;
	}

	public void setPostalCode(String postalCode) {
		this.postalCode = postalCode;
	}

	public String getAddr11() {
		return addr11;
	}

	public void setAddr11(String addr11) {
		this.addr11 = addr11;
	}

	public String getTelNumber() {
		return telNumber;
	}

	public void setTelNumber(String telNumber) {
		this.telNumber = telNumber;
	}

	public String getEmail() {
		return email;
	}

	public void setEmail(String email) {
		this.email = email;
	}

}

実際の登録処理コード

public class AddressRegiCompleteAction extends ActionSupport implements SessionAware{

	public Map<String,Object> session;

	public String execute() throws SQLException{

		String result = ERROR;

		AddressDTO addressDTO = new AddressDTO();

		List<String> errorMsg = new ArrayList<String>();

		addressDTO = (AddressDTO)session.get("AddressDTO");

		AddressRegiCompleteDAO addressRegiCompleteDAO = new AddressRegiCompleteDAO();

////		全く同じ内容のものが既にDBに格納されていないか判定。
		errorMsg = addressRegiCompleteDAO.doubleCheck(addressDTO);
		if(!errorMsg.isEmpty()) {
			return result;
		}

		int count = addressRegiCompleteDAO.registerAddress(addressDTO);
//		countで登録されたかの判定。
		if(count > 0) {
			session.remove("AddressDTO");
			result = SUCCESS;
		}

		return result;
	}


	public Map<String,Object> getSession(){
		return session;
	}
	public void setSession(Map<String,Object> session) {
		this.session = session;
	}

}

思い付く限りでコードを書いて見たのですが、現状、全く同じ宛先情報でも登録されてしまいます。

ご回答宜しくお願いします。
コード的に改善点（こう書いた方がいいよ）等ありましたら重ねてお願い致します。

Answer 1

ResultSet rs = ps.executeQuery();

// ここで「rs.next()」してないので、取得結果の先頭にカーソルが移動してない。

if(rs.getInt("count") > 0) {
    errorMsg.add("こちらの宛先は既に登録されています");
}

コメント返しの追記：
ResultSetの使い方を理解されてないようです。下記を参照ください。
javadrive

Comments

[Tatsuo_Iriyama]

if(rs.getInt("count").next() > 0 ) {
errorMsg.add(~~~);
}
こんな感じですか？

Answer 2

登録部分のソースコードがなく、またAddressDTOへの設定情報も不明なため、推測で回答致します。

二重チェックが有効とならない理由としては以下が考えられます。

• doubleCheck()に渡しているAddressDTOに設定している値正しくない
• doubleCheck()にてExceptionが発生しており、チェックが正しく行えていない

実行しているSQLのデバッグや、以下の処理を通過していることの確認をしてみると良いと思います。
以下の処理を通っている（Exceptionが発生していない）場合、実行されているSQLの確認や、rs.getInt("count")の値の確認をするのが良いですね。

           if(rs.getInt("count") > 0) {
                errorMsg.add("こちらの宛先は既に登録されています");
            }

コード的に改善点（こう書いた方がいいよ）等ありましたら重ねてお願い致します。

今回の検証と合わせてですが、以下のExceptionのcatch句では、エラーを握りつぶしてしまっているので、適切な処理（ログ出力や、戻りの設定（Exceptionの再throwなど））を行うようにした方が良いですね。

        }catch(Exception e) {
            e.printStackTrace();
        }finally {
            con.close();

Comments

[Tatsuo_Iriyama]

コード追加しました！
あと登録を行った際、「You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near 'postal_code='○○○' user_address='○○○'' at line 1」
こういうエラー文がコンソール上に表示されたのですがどういう意味を示しているのでしょうか？

[退会済みユーザー]

postal_code=? と user_address=? の前に and がありません。

[Tatsuo_Iriyama]

上手くいきました！ありがとうございました！
1つ質問なんですが、コードについてご指摘いただいたんですが、あまりイメージを沸くことができず、エラーを握りつぶしているとはどういうことかもう少し噛み砕いた形で教えて頂けると嬉しいです。

[takyafumin]

問題切り分けにはなりましたが、実際の解決は他の回答者さまの方が参考になっていたかもですね（汗

> エラーを握りつぶしているとはどういうことか

try〜catch句でExceptionが発生した場合、catch句に処理が飛びます。
そしてcatch句の処理が行われるのですが、提示いただいたソースですと、スタックトレースの出力しか行っておらず、結果的にエラーメッセージの入っていないerrorMsgがreturnされてしまいます。
（SQL実行エラーが発生した場合に、正常動作と同じ動きをするのは好ましくないです）

この点が、「エラーが（発生しているにも関わらず、あとで発生したことが分からないよう）握りつぶされている」と表現しました。
対処は回答に記載した通りです。