JSPの正しい記述について 条件式

Question

[リンク内容]
(https://teratail.com/questions/10948)
###前提・実現したいこと
こんにちは。
分からないことがありますので、質問です。
JSPを記述していて、条件式のところで分岐がうまく出来ませんので、
どなたか教えていただけないでしょうか。
deleteFlagが1のときに、OKボタン押下でDELETEが実行されると
記述したいのですが、うまく出来ておりません。
ソースを添付いたしますので、見てやってください。
よろしくお願いいたします。

###ソースコード

<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" />
            <input type="hidden" name="processType" value="update">
		</c:otherwise>
</c:choose>
        <c:if test="${deleteFlag == 1}" >
		    <input type="hidden" value="OK" name="delete" />

		</c:if>

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

Comments

[kaputaros]

同じ箇所についての質問だったら、乱立させずに、１つにまとめて質問したほうがいいと思います。
一から聞くより全体を見たほうが、回答してもらいやすくなると思います。
関連する質問のURLを記載しておいてください。

[yk-211]

ご指摘いただきありがとうございます。
関連するものを追記いたします。

Answer 1

どの質問に回答して良いか迷いますが、とりあえずここに。

submitは変えないで、hiddenのprocessTypeにまとめたほうが良いと思います。
このように1つのchooseにまとめてみてはどうでしょうか？

<input type="submit" value="OK" />
<c:choose>
  <c:when test="${processType == 'insert'}">
    <input type="hidden" name="processType" value="insert" />
  </c:when>
  <c:when test="${deleteFlag == '1'}">
    <input type="hidden" name="processType" value="delete" />
  </c:when>
  <c:otherwise>
    <input type="hidden" name="processType" value="update" />
  </c:otherwise>
</c:choose>

JSTLの場合、文字列は==でもequalsとして判定してくれたと思います。

---

（追記）

最終的なSQLで判断するのではなく、それぞれの処理で何が起きているのかをひとつずつ切り分けてください。

この場合は、JSPが生成したHTMLをブラウザーの「ソースの表示」で見てみてください。その時点でdeleteになっているのなら、画面を表示するときのprocessTypeが'insert'ではない（<c:when test="${processType == 'insert'}">がtrueにならない）可能性があります。
proctype=<c:out value="${processType}" />を処理の前に埋め込んで、内容を確認してみてください。

もうひとつ、肝心なことを確認していませんでしたが、実行環境やバージョンによって動作が変わる可能性がありますので、その辺の情報を質問10948に記載したほうが良いかも知れません。

Comments

[yk-211]

ありがとうございます。
そちらでやってみたのですが、更新時も削除フラグ立てた場合も
どちらもDELETE文を取ってきてしまうようです･･･
他のところに原因があるかもしれません。

[yk-211]

回答ありがとうございます。
proctype=<c:out value="${processType}" />を実行してみたところupdateとなっておりました。まずここが間違っているということですよね？
そうなると直すべきところはどこになりますでしょうか。

[argius]

最終的にどういう仕様なのかが分からなくなってしまいました。

私は↓のように解釈したのですが、合っていますか？
・processType == 'insert'ならdeleteFlagに関係なく、INSERT文が実行される。
・上記以外でdeleteFlag == '1'ならDELETE文が実行される。
・上記以外ならUPDATE文が実行される。

合っているなら、deleteFlagの値も怪しいのでc:outで確認してみてください。

[argius]

もしかしたら、画面で後からチェックしたdeleteFlagのオンオフをJSPで参照しようとしていますか？ そうだとしたら、それはできません。JavaScriptの出番です。

[yk-211]

何度も回答ありがとうございます･･･
JAVAのところで1箇所間違いがあり、そこを直すことによって
JSPが正確に分岐してくれるようになりました。
大変大変ありがとうございました。

Answer 2

javaでの比較に==を使うと、プリミティブ型なら値が同じかどうかを判定することになりますが、
オブジェクト型だと、「同じインスタンスか」という比較にしかなりません。
オブジェクト型の値が同じかどうかを比較するには、equals()を使用します。

processTypeとdeleteFlagは何型として扱いたいのでしょうか？

また、想定される値がきちんと渡ってきているかの確認はしましたでしょうか？
eclipseで実行しているのであれば、debugモードを実行することもできるかと思います。
できなければ、あらゆる場所にログを追加して、値がきているかどうかを確認して、問題部分を割り出すことをお勧めします。

Comments

[yk-211]

回答ありがとうございます。
値はきちんと渡っておりました。

オブジェクト型で扱っておりました。