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

JSPファイルが間違っているため条件式で分岐できないのではないか
との指摘を受けましたのでJSPファイルを添付させていただきます。
もし分かる方がいらっしゃったらご回答をよろしくお願いします。

lang
1<%@ page language="java" contentType="text/html; charset=EUC_JP"
2    pageEncoding="Windows-31J"%>
3<%@ taglib prefix="c" uri="/WEB-INF/tld/c.tld" %>
4<%@ taglib prefix="fn" uri="/WEB-INF/tld/fn.tld" %>
5<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">
6<html>
7<head>
8<meta http-equiv="Content-Type" content="text/html; charset=EUC_JP">
9<c:import url="./head.jsp" />
10<link rel="stylesheet" type="text/css" href="../css/customerDetail.css">
11
12<title>編集画面</title>
13</head>
14<body>
15
16    <form action="./CustomerDetail" method="post">
17	<table>
18        <tr>
19			<th>顧客ID</th>
20			<td>
21			<c:choose>
22				<c:when test="${processType == 'insert'}">
23			        <input type="text" name="id" size="12" value="${id}">
24		    </c:when>
25			<c:otherwise>
26				<c:out value="${id}"/>
27				<input type="hidden" name="id" value="${id}">
28			</c:otherwise>
29			</c:choose>
30			</td>
31		</tr>
32        <tr>
33		    <th>顧客漢字名</th>
34			<td><input type="text" name="name" size="20" value="${name}"></td>
35		</tr>
36		<tr>
37		    <th>顧客カナ名</th>
38			<td><input type="text" name="kana" size="20" value="${kana}"></td>
39		</tr>
40        <tr>
41		    <th>性別</th>
42			<td><select name="gender"><OPTION value="${0}" >男性</OPTION>
43 				<OPTION value="${1}">女性</OPTION></select></td>
44		</tr>
45		 <tr>
46		    <th>身長</th>
47			<td><input type="text" name="height" size="12" value="${height}">cm</td>
48		</tr>
49         <tr>
50		    <th>体重</th>
51			<td><input type="text" name="weight" size="12" value="${weight}">kg</td>
52		</tr>
53        <tr>
54		    <th>生年月日</th>
55		    <td><input type="text" name="birthday" size="12" value="${birthday}"></td>
56		</tr>
57        <tr>
58		    <th>郵便番号</th>
59		    <td><input type="text" name="zipCode" size="14" value="${zipCode}">(ハイフン無しで入力してください)</td>
60		</tr>
61		<tr>
62		    <th>住所</th>
63		    <td><input type="text" name="address" size="40" value="${address}"></td>
64		</tr>
65		<tr>
66		    <th>電話番号</th>
67		    <td><input type="text" name="phone" size="14" value="${phone}">(ハイフンありで入力してください)</td>
68		</tr>
69        <tr>
70		    <th>削除</th>
71		    <td><input type="checkbox" name="deleteFlag" size="12" value="${1}"></td>
72		</tr>
73
74</table>
75<br>
76
77<c:choose>
78		<c:when test="${processType == 'insert'}">
79			<input type="submit" value="OK" name="insert" />
80            <input type="hidden" name="processType" value="insert">
81		</c:when>
82		<c:otherwise>
83			<input type="submit" value="OK" name="update" />&nbsp;
84            <input type="hidden" name="processType" value="update">
85		</c:otherwise>
86	</c:choose>
87	&nbsp;
88	<input type="submit" value="一覧へ" name="backList" />
89	</form>
90</body>
91</html>
92
93

###前提・実現したいこと
こんばんは。
5時間程度考えておりますが、まったく進まないため、再度質問させてください。
ただいまSQLに接続するJavaのソースを書いておりまして、DELETEを実行した際に
UPDATEを実行するようになってしまっております。
該当の箇所を添付いたしますので、アドバイスをくださいませ。
ソースは一部省略しております。
よろしくお願いいたします。

###発生している問題・エラーメッセージ
DELETEしたい場合にもUPDATEを実行してしまう。
DeleteFlagには初期値で0が入っており、削除のチェックボックスにチェックを入れると1となり削除が実行されるというものです。

###ソースコード

lang
1 protected ProcessResult execute(HttpServletRequest request,
2            HttpServletResponse response, Connection con) throws Exception {
3        if (request.getParameter("backList") != null) {
4            // 一覧へ戻る
5            ProcessResult result = new ProcessResult();
6            result.setUrl(customerListUrl);
7            result.setNormalEndFlg(true);
8
9            super.setSessionAttribute("id", "");
10            super.setSessionAttribute("name", "");
11            super.setSessionAttribute("kana", "");
12            super.setSessionAttribute("gender", "");
13
14            return result;
15        }
16
17        if (request.getParameter("processType") == null) {
18            // 登録画面初期表示
19            return createInsertModoResult();
20        } else if (request.getParameter("processType").equals("disp")) {
21            // 更新・削除初期表示
22            return this.doDispDetails(request, con);
23        } else if (request.getParameter("processType").equals("insert")) {
24            // 登録処理
25            return this.doInsert(request, con);
26        } else if (request.getParameter("processType").equals("update")) {
27            if (request.getParameter("update") != null) {
28                // 更新処理
29                return this.doUpdate(request, con);
30            } else {
31                // 削除処理
32                return this.doDelete(request, con);
33            }
34        } else {
35            ProcessResult result = new ProcessResult();
36            result.setUrl(requestPageUrl);
37            result.setNormalEndFlg(false);
38
39            // 処理モードをセットする。
40            super.setSessionAttribute("processType", "");
41            super.setErrMsg("不正なアクセス");
42            return result;
43        }
44    }
45
46

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

回答2件

ベストアンサー

// 不思議な設計ですね。この３つが別けてあるのに、updateにくっつける意味がよくわかりません。
disp, insert, update

//なんとしてもこれであるなら
} else if (request.getParameter("processType").equals("update")) {
if (request.getParameter("update") != null) {
// 更新処理
return this.doUpdate(request, con);
} else if( DeleteFlag == 1 ) {
// 削除処理
return this.doDelete(request, con);
}
}
} else if (request.getParameter("processType").equals("delete")) {
}

// もう１つはですかね。
if( DeleteFlag == 1 ){
} else if (request.getParameter("processType") == null ){
} else if (request.getParameter("processType").equals("disp")) {
} else if (request.getParameter("processType").equals("insert")) {
} else if (request.getParameter("processType").equals("update")) {
}

投稿2015/06/10 11:39

MasaakiIrie

総合スコア1021

yk-211

2015/06/10 11:48

回答ありがとうございます！こちらを入力してみましたが通りませんでした。私が間違っているかもしれません、すみません。

MasaakiIrie

2015/06/10 11:52

内容を推測するかぎりでは、おそらく、DeleteFlag に関係なく request.getParameter("processType").equals("update")) が返るようですね。まずは、request.getParameter("processType")にupdateが返ることが問題ですね。

yk-211

2015/06/10 11:53

ありがとうございます。でしたらこちらはどのように解決するべきでしょうか。

MasaakiIrie

2015/06/10 12:03

HttpServletResponse response 今見返すと、requestを見てるのですが・・・　これてどんな関数なんですか？要求を出す方なのか、要求の結果を見る関数なのか。結果がどのような値を取りうるかで、if文を書くのが良いですね。 update と deleteは全く違う要求なので、一緒のところを通るのが変ですね。

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