質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.03%

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

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 1,484

score 39

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


<%@ page language="java" contentType="text/html; charset=EUC_JP"
    pageEncoding="Windows-31J"%>
<%@ taglib prefix="c" uri="/WEB-INF/tld/c.tld" %>
<%@ taglib prefix="fn" uri="/WEB-INF/tld/fn.tld" %>
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=EUC_JP">
<c:import url="./head.jsp" />
<link rel="stylesheet" type="text/css" href="../css/customerDetail.css">

<title>編集画面</title>
</head>
<body>

    <form action="./CustomerDetail" method="post">
    <table>
        <tr>
            <th>顧客ID</th>
            <td>
            <c:choose>
                <c:when test="${processType == 'insert'}">
                    <input type="text" name="id" size="12" value="${id}">
            </c:when>
            <c:otherwise>
                <c:out value="${id}"/>
                <input type="hidden" name="id" value="${id}">
            </c:otherwise>
            </c:choose>
            </td>
        </tr>
        <tr>
            <th>顧客漢字名</th>
            <td><input type="text" name="name" size="20" value="${name}"></td>
        </tr>
        <tr>
            <th>顧客カナ名</th>
            <td><input type="text" name="kana" size="20" value="${kana}"></td>
        </tr>
        <tr>
            <th>性別</th>
            <td><select name="gender"><OPTION value="${0}" >男性</OPTION>
                 <OPTION value="${1}">女性</OPTION></select></td>
        </tr>
         <tr>
            <th>身長</th>
            <td><input type="text" name="height" size="12" value="${height}">cm</td>
        </tr>
         <tr>
            <th>体重</th>
            <td><input type="text" name="weight" size="12" value="${weight}">kg</td>
        </tr>
        <tr>
            <th>生年月日</th>
            <td><input type="text" name="birthday" size="12" value="${birthday}"></td>
        </tr>
        <tr>
            <th>郵便番号</th>
            <td><input type="text" name="zipCode" size="14" value="${zipCode}">(ハイフン無しで入力してください)</td>
        </tr>
        <tr>
            <th>住所</th>
            <td><input type="text" name="address" size="40" value="${address}"></td>
        </tr>
        <tr>
            <th>電話番号</th>
            <td><input type="text" name="phone" size="14" value="${phone}">(ハイフンありで入力してください)</td>
        </tr>
        <tr>
            <th>削除</th>
            <td><input type="checkbox" name="deleteFlag" size="12" value="${1}"></td>
        </tr>

</table>
<br>

<c:choose>
        <c:when test="${processType == 'insert'}">
            <input type="submit" value="OK" name="insert" />
            <input type="hidden" name="processType" value="insert">
        </c:when>
        <c:otherwise>
            <input type="submit" value="OK" name="update" />&nbsp;
            <input type="hidden" name="processType" value="update">
        </c:otherwise>
    </c:choose>
    &nbsp;
    <input type="submit" value="一覧へ" name="backList" />
    </form>
</body>
</html>



前提・実現したいこと

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

発生している問題・エラーメッセージ

DELETEしたい場合にもUPDATEを実行してしまう。
DeleteFlagには初期値で0が入っており、削除のチェックボックスにチェックを入れると1となり削除が実行されるというものです。

ソースコード

 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;
        }
    }
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

+3

// 不思議な設計ですね。この3つが別けてあるのに、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")) {
}



// もう1つはですかね。
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 20:48

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

    キャンセル

  • 2015/06/10 20:52

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

    キャンセル

  • 2015/06/10 20:53

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

    キャンセル

  • 2015/06/10 21:03

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

    キャンセル

0

書かれているロジックだと、DeleteFlagの値に関係なく「update」というパラメーターがNULL以外の値だった場合には更新処理が実行されますね。空文字とかでもダメです。

Updateというパラメーターにどんな値が入るのかわかりませんが、DeleteFlagの値を優先させたいのであれば

if (DeleteFlag == 1) { 
        // 削除処理 
return this.doDelete(request, con); 
} else { 
        // 更新処理 
        return this.doUpdate(request, con); 


ってしないとだめでは?

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2015/06/11 01:01

    回答ありがとうございます。
    JSPの書き方が間違っているためDELETE文に
    渡らないのではないかとの指摘を受けましたので、
    追記に載せてみます。
    よろしければ添削をお願いいたします。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.03%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る