Javaの添削をお願いします。

Question

さらに追記します。

if (request.getParameter("deleteFlag") == "0") {
				// 更新処理
				return this.doUpdate(request, con);
			} else {
				// 削除処理
				return this.doDelete(request, con);
			}

としてみたのですが更新の場合、削除の場合ともにDELETE文を呼んでしまいます。deleteFlagには初期値で0が入っています。

追記です。

if (request.getParameter("update") != null) {
				// 更新処理
				return this.doUpdate(request, con);
			} else {
				// 削除処理
				return this.doDelete(request, con);
			}

の部分の("update") != null)を("deleteFlag") == 0)
とするとうまくいくと思うのですがString型とint型のため
値をこのまま入れることが出来ません。
うまく入れるためにはどうしたらよろしいでしょうか。

###前提・実現したいこと
こんにちは。
質問をさせてください。

ただいまSQLに接続するJavaのソースを書いておりまして、DELETEを実行した際に
UPDATEを実行するようになってしまっております。
おそらく45行目のあたりからが特に怪しいのですが、どのように直してもうまくいきません。
もしお分かりになる方がおられましたら、長いソースになりますがよろしくお願いいたします。かなり省略しています。
###ソースコード

public class CustomerDetail extends AbstractBizLogic {
	private static final long serialVersionUID = 1L;

	/** リクエストURL */
	private String requestPageUrl = "/page/customerDetail.jsp";

	/** 一覧表示画面URL */
	private String customerListUrl = "/page/customerList.jsp";

	/**
	 * コンストラクタ
	 */
	public CustomerDetail() {
		super();
	}

	/**
	 * @see common.AbstractBizLogic#getRequestPageUrl()
	 */
	protected String getRequestPageUrl() {
		return requestPageUrl;
	}

	/**
	 * @see common.AbstractBizLogic#execute(HttpServletRequest,HttpServletResponse,
	 *      Connection)
	 */
	protected ProcessResult execute(HttpServletRequest request,
			HttpServletResponse response, Connection con) throws Exception {
		if (request.getParameter("backList") != null) {
			// 一覧へ戻る
			ProcessResult result = new ProcessResult();
			result.setUrl(customerListUrl);
			result.setNormalEndFlg(true);

			super.setSessionAttribute("id", "");
			super.setSessionAttribute("name", "");
			super.setSessionAttribute("kana", "");
			super.setSessionAttribute("gender", "");

			return result;
		}

		if (request.getParameter("processType") == null) {
			// 登録画面初期表示
			return createInsertModoResult();
		} else if (request.getParameter("processType").equals("disp")) {
			// 更新・削除初期表示
			return this.doDispDetails(request, con);
		} else if (request.getParameter("processType").equals("insert")) {
			// 登録処理
			return this.doInsert(request, con);
		} else if (request.getParameter("processType").equals("update")) {
			if (request.getParameter("update") != null) {
				// 更新処理
				return this.doUpdate(request, con);
			} else {
				// 削除処理
				return this.doDelete(request, con);
			}
		} else {
			ProcessResult result = new ProcessResult();
			result.setUrl(requestPageUrl);
			result.setNormalEndFlg(false);

			// 処理モードをセットする。
			super.setSessionAttribute("processType", "");
			super.setErrMsg("不正なアクセス");
			return result;
		}
	}

	/**
	 * ユーザーマスタ情報を検索し詳細画面表示します。
	 *
	 * @param request
	 *            リクエスト情報
	 * @param con
	 *            コネクション
	 * @return 処理結果DTO
	 * @throws Exception
	 */
	private ProcessResult doDispDetails(HttpServletRequest request,
			Connection con) throws Exception {
		ProcessResult result = new ProcessResult();
		String id = request.getParameter("id");
		List<CustomerDto> retList = this.getCustomerList(id, con);
		if (retList.size() == 0) {
			super.setErrMsg("データがすでに削除された可能性があります。再度検索してください。");
			// 一覧検索画面に戻す。
			result.setUrl(customerListUrl);
			result.setNormalEndFlg(false);

			return result;
		}
		CustomerDto dto = retList.get(0);
		super.setSessionAttribute("id", dto.getId());
		super.setSessionAttribute("name", dto.getName());


		// 処理モードをセットする。
		super.setSessionAttribute("processType", "update");
		result.setUrl(requestPageUrl);
		result.setNormalEndFlg(true);
		return result;
	}

	/**
	 * ユーザーマスタの新規登録を行います。
	 *
	 * @param request
	 *            リクエスト情報
	 * @param con
	 *            コネクション
	 * @return 処理結果DTO
	 * @throws Exception
	 */
	private ProcessResult doInsert(HttpServletRequest request, Connection con)
			throws Exception {
		ProcessResult result = new ProcessResult();

		String id = request.getParameter("id");
		String name = request.getParameter("name");


		boolean errExistsFlg = false;
		// 必須チェック
		if (StringUtils.isBlank(id)) {
			super.setErrMsg("IDは必ず入力してください。");
			errExistsFlg = true;
		}
		if (StringUtils.isBlank(name)) {
			super.setErrMsg("名前は必ず入力してください。");
			errExistsFlg = true;
		}
		if (StringUtils.isBlank(kana)) {
			super.setErrMsg("フリガナは必ず入力してください。");
			errExistsFlg = true;
		}
		if (StringUtils.isBlank(height)) {
			super.setErrMsg("身長は必ず入力してください。");
			errExistsFlg = true;
		}
		if (StringUtils.isBlank(weight)) {
			super.setErrMsg("体重は必ず入力してください。");
			errExistsFlg = true;
		}
		if (StringUtils.isBlank(birthday)) {
			super.setErrMsg("生年月日は必ず入力してください。");
			errExistsFlg = true;
		}



		if (errExistsFlg) {
			// 入力チェックでエラーの場合は処理終了
			result.setNormalEndFlg(false);
			return result;
		}
		/***
		 * //ID存在チェック List<CustomorDto> retList = this.getCustomerList(id, con);
		 * if (retList.size() == 0) {
		 * super.setErrMsg("データがすでに削除された可能性があります。再度検索してください。");
		 * result.setNormalEndFlg(false); return result; }
		 */

		CustomerDto paramDto = new CustomerDto();
		paramDto.setId(id);
		paramDto.setName(name);
		paramDto.setKana(kana);
		paramDto.setGender(gender);


		CustomerUpdateDao updateDao = new CustomerUpdateDao();
		updateDao.setParam(paramDto);
		updateDao.execute(con);

		super.setSessionAttribute("id", id);
		super.setSessionAttribute("name", name);
		super.setSessionAttribute("kana", kana);


		result.setMsg("削除が完了しました。");
		result.setUrl(requestPageUrl);
		result.setNormalEndFlg(true);


	    return result;

	}

	/**
	 * ユーザーマスタの削除を行います。
	 *
	 * @param request
	 *            リクエスト情報
	 * @param con
	 *            コネクション
	 * @return 処理結果DTO
	 * @throws Exception
	 */
	private ProcessResult doDelete(HttpServletRequest request, Connection con)
			throws Exception {
		ProcessResult result = new ProcessResult();
		result.setUrl(requestPageUrl);
		// 処理モードをセットする。
		super.setSessionAttribute("processType", "update");

		String id = request.getParameter("id");
		String name = request.getParameter("name");
		String kana = request.getParameter("kana");

		/***
		 * ID存在チェック List<CustomorDto> retList = this.getUserList(id, con); if
		 * (retList.size() == 0) {
		 * super.setErrMsg("データがすでに削除された可能性があります。再度検索してください。");
		 * result.setNormalEndFlg(false); return result; }
		 */

		CustomerDto paramDto = new CustomerDto();
		paramDto.setId(id);
paramDto.setAddress(address);
		paramDto.setPhone(phone);
		paramDto.setDeleteFlag(deleteFlag);

		CustomerDeleteDao deleteDao = new CustomerDeleteDao();
		deleteDao.setParam(paramDto);
		deleteDao.execute(con);

		super.setSessionAttribute("id", id);
		super.setSessionAttribute("phone", phone);
		result.setMsg("削除が完了しました。");
		result.setUrl(requestPageUrl);
		result.setNormalEndFlg(true);

		return result;
	}

	/**
	 * ユーザーマスタの削除を行います。
	 *
	 * @param request
	 *            リクエスト情報
	 * @param con
	 *            コネクション
	 * @return 処理結果DTO
	 * @throws Exception
	 */
	private ProcessResult createInsertModoResult() {
		// 新規登録表示
		ProcessResult result = new ProcessResult();
		result.setUrl(requestPageUrl);
		result.setNormalEndFlg(true);

		super.setSessionAttribute("id", "");
		super.setSessionAttribute("name", "");
		super.setSessionAttribute("kana", "");
		super.setSessionAttribute("phone", "");
		// 処理モードをセットする。
		super.setSessionAttribute("processType", "insert");
		return result;
	}

	/**
	 * ユーザーマスタを検索しリストをかえします。
	 *
	 * @param id
	 *            ユーザーID
	 * @param con
	 *            コネクション
	 * @return ユーザーマスタDTOリスト
	 * @throws Exception
	 */

	private List<CustomerDto> getCustomerList(String id, Connection con)
			throws Exception {
		CustomerDto paramDto = new CustomerDto();
		paramDto.setId(id);

		CustomerSelectDao dao = new CustomerSelectDao();
		dao.setParam(paramDto);
		dao.execute(con);
		return dao.getResultList();

###補足情報(言語/FW/ツール等のバージョンなど)

Answer 1

文字列の比較には通常、equalを使用します。
if (request.getParameter("deleteFlag").equal("0"))

Comments

[yk-211]

回答ありがとうございます。
おっしゃるとおりです、忘れておりました。
そちらでやってみたところ、UPDATEが動きませんでした。

Answer 2

if (request.getParameter("update") != null) {
    // 更新処理
    return this.doUpdate(request, con);
} else {
    // 削除処理
    return this.doDelete(request, con);
}

DELETEのはずなのにUPDATEになるということは、上記部分のifの判定が怪しいですね。
デバッグやprintlnでrequest.getParameter("update")を確認し、UPDATE、DELETEそれぞれどのような値が格納されているか見てみましょう。

Javaの文字列の比較は慣れないとわかりづらいかもしれません。
参考:Javaでの比較

Comments

[yk-211]

ありがとうございます。
そこの部分でdeleteFlagに1が入ったときはdeleteになるようにするとうまくいきそうで、
if (request.getParameter("deleteFlag") == 0)としたいのですが、String型とint型のためうまくいかないようです。
どのようにしたらうまくいきますでしょうか。

Answer 3

４５行目てどこですか＾＾；

System.out.println( request ); // requestの中身を見た方がいいですね。

if (request.getParameter("update") != null) {
// 更新処理
return this.doUpdate(request, con);
} else if(request.getParameter("delete") != null) { ← 入れた方がいいですね。
// 削除処理
return this.doDelete(request, con);
}

Comments

[yk-211]

回答ありがとうございます。
分かりづらく申し訳ありませんでした。
MasaakiIrie様の指摘していただいているそちらの部分が問題だと思いますが、
まずはrequestの中を見てみます。
ありがとうございます。