条件分岐が多いIF分の書き方について。

Question

テキスト入出力でのIF分についてご質問がございます。
例えば条件が多いときに、結果が得られない時があるのですが
書き方で何か問題があるのならご指摘ください。
下記のコードでご説明します。

//コード１の文字列が04でコード２の文字列が01の場合00001を表示

if (コード1.compareTo ("04") == 0  && コード2 .compareTo ("01") == 0 ) {

out.print ("00001");

//コード１の文字列が04でコード3の文字列が2の場合00005を表示

} else if (コード1.compareTo ("04") == 0  && コード3 .compareTo ("02") == 0 ) {

out.print ("00005");

//コード１の文字列が08でコード3の文字列が07の場合10000を表示

} else if (コード1.compareTo ("08") == 0  && コード3 .compareTo ("07") == 0 ) {

out.print ("10000");

//コード１の文字列が10でコード5の文字列が07の場合30000を表示

} else if (コード1.compareTo ("10") == 0  && コード5 .compareTo ("07") == 0 ) {

out.print ("30000");

//それ以外エラー

} else {
out.print("エラー");}

}

値がIF分の条件とマッチしていても

if (コード1.compareTo ("04") == 0  && コード2 .compareTo ("01") == 0 ) {

out.print ("00001");

//コード１の文字列が04でコード3の文字列が2の場合00005を表示

} else if (コード1.compareTo ("04") == 0  && コード3 .compareTo ("02") == 0 ) {

out.print ("00005");

//コード１の文字列が08でコード3の文字列が07の場合10000を表示

} else if (コード1.compareTo ("08") == 0  && コード3 .compareTo ("07") == 0 ) {

out.print ("10000");

このコードまでに当てはまらない条件はすべて
("10000");
の結果になり
つまりそれ以降のelse if の条件に
当てはまらなくなります。
else if をいくらでも増やして条件分岐出来ると学習して
検証しているのですが・・・
最終的にがIF～else if ～elseで
30パターンぐらい条件分岐したいと思ってます。
初歩的なご質問で恐縮ですが
アドバイス
よろしくお願いします。

kmee · Accepted Answer

> 条件がかなり多くても対応できますでしょうか？？

できます。

> それか大きなグループとしては３通りありますので
> IF分をおおきつ３つに分け
> その中に各条件を指定し対応するようなことは可能でしょうか？？

できます。

> switchの中にIF分を書いたりすることは出来るのでしょうか？？

できます。

プログラミングのコツは「楽する方法を考える。そのためにはどんな努力も惜しまない」です。
今のコードは「まじめ過ぎ」です。

> ・制度コード＝”０４”または”０８”または”０９”以外で
> A種コードが01の場合
....

「制度コード＝”０４”または”０８”または”０９”以外」が共通してますから
if( (制度コード.compareTo(”０４”)!=0) &&  (制度コード.compareTo(”０8”)!=0) && (制度コード.compareTo(”０9”)!=0) ){
　if( A種コード.compareTo("01")) {～
}
とまとめられます。

※ 日本語にだまされないでください。
 「または」ということで ||を使っていますが、意味を考えれば
 「(”０４”または”０８”または”０９”)ではない」
 です。
 (制度コード.compareTo(”０４”)!=0) || (制度コード.compareTo(”０8”)!=0) || (制度コード.compareTo(”０9”)!=0)
は
「(”０４”でない)または(”０８”でない)または(”０９”でない)」
⇒ 常に真です。(04→「08でない」ので真、04以外→「04でない」ので真、すなわち、全てで真)
 「(”０４”または”０８”または”０９”)ではない」のままに
! ((制度コード.compareTo(”０４”)==0) || (制度コード.compareTo(”０8”)==0) || (制度コード.compareTo(”０9”)==0))
にするか、ドモルガンの法則を使って 「(”０４”でない)かつ(”０８”でない)かつ(”０９”ではない)」にするか、です。

制度コードで分類できるなら
 if (制度コード.compareTo(”０４”)==0) {
// 04関連
} else if (制度コード.compareTo(”０8”)==0) {
// 08関連
} else if  (制度コード.compareTo(”０9”)==0)) {
// 09関連
} else {
//ここは「(”０４”または”０８”または”０９”)ではない」
}

とすれば、大分すっきりします。なにより ! ((制度コード.compareTo(”０４”)==0) || (制度コード.compareTo(”０8”)==0) || (制度コード.compareTo(”０9”)==0)) みたいな長々しい条件を何度も書かずにすみます。

これができるかどうかは、条件を解析する必要があります。
その努力は惜しんではいけません。

jjon-com · Answer

> おっしゃる通りです。（回答ANo.3への補足）

え？　ANo.3は質問者の意図どおりだとおっしゃっているのですか？
質問者は次の日本語の意味をちゃんと理解していらっしゃいますか？

--------
(a)「制度コードが"04"である（注：それ以外の変数は検査しない）」
または
(b)「制度コードが"08"ではない（注：それ以外の変数は検査しない）」
または
(c)「制度コードが"09"ではない，かつ，年金種類a種コードが"01"である」
--------

(a)の"04"は，条件(b)に包含されているからそもそも不要な判定です。
条件(b)に適合すれば条件(c)は不問だから，制度コード=="08"でないものは年金種類a種コードを検査することなくすべてprint("老齢")される。
最後に条件(c)で，制度コード=="08"かつ年金種類a種コード=="01"のものがprint("老齢")される。

再度確認します。
以上が質問者の意図した条件式であるなら，そうおっしゃっていただけますか？

もしもそうでなかったのなら，
小学校の算数で×÷よりも＋－を優先させたいときにカッコで囲うのと同様に，
優先させたい条件式を（）で囲ってください。

jjon-com · Answer

論理和（または，||）よりも 論理積（かつ，&&）の方が結合が強いですから，

--------
//************** の直前にある以下のif文は，

} else if (制度コード.compareTo ("04") == 0 && 年金種類b種コード .compareTo ("01") != 0 || 年金種類b種コード .compareTo ("02") != 0 ) {

次の意味になります。

「制度コードが"04"であり，かつ，年金種類b種コードが"01"ではない」
または「年金種類b種コードが"02"ではない（注：制度コードの値は検査しない）」

--------
//************** の直後にある以下のif文は，

} else if (制度コード.compareTo ("04") == 0 || 制度コード .compareTo ("08") != 0 || 制度コード .compareTo ("09") != 0 && 年金種類a種コード.compareTo ("01") == 0 ){

次の意味になります。

「制度コードが"04"である（注：それ以外の変数は検査しない）」
または「制度コードが"08"ではない（注：それ以外の変数は検査しない）」
または「制度コードが"09"ではない，かつ，年金種類a種コードが"01"である」

--------
以上が質問者の意図した条件式であるなら，そうおっしゃっていただけますか？
もしもそうでなかったのなら，
小学校の算数で×÷よりも＋－を優先させたいときにカッコで囲うのと同様に，
優先させたい条件式を（）で囲ってください。

kmee · Answer

if～elseはいくらでも繋がりますが、それをやるのは、あまりに無謀です。
条件を整理して、規則を見付けたりまとめたりして、すっきりさせるのが、結局近道だと思いますよ。

すくなくとも、私はこんなif elseばかりのコードを仕様通り正しく書ける自信はありません。

例えば
if (コード1.compareTo ("04") == 0 && コード2 .compareTo ("01") == 0 ) {
out.print ("00001");
} else if (コード1.compareTo ("04") == 0 && コード3 .compareTo ("02") == 0 ) {
out.print ("00005");
}

ってありますけど、
outCode="10000";
if (コード1.compareTo ("04") == 0) {
if( コード2 .compareTo ("01") == 0 ) {
outCode="00001";
} else if ( コード3 .compareTo ("02") == 0 ) {
outCode="00005";
}
}
out.print(outCode)

と書けば
・コード1.compareTo("04")の条件は全てここに入るので、チェック抜け防止になる
・「それ以外」用のif/elseをわざわざ作る必要がない
・out.printが1つなので、出力前に"1000"にリセットすることができる
と、大分楽になるのでは?

それとか
コードを表わす正規表現→出力コード のようなテーブルを用意して
コード1+コード2+コード3+... と正規表現がマッチしたら、対応する出力コード
のように判定させるとか
コード1.compareTo ("04") == 0 && コード3 .compareTo ("02") == 0 
→ "^0401" とマッチ

jjon-com · Answer

元のJavaソースコード中に「コード1」や「コード2(半角空白)」が登場するわけではないですよね。
一部分だけを提示してもコンパイルできないという状態でかまいませんから，
異常が起こるというそのJavaソースコードを加工なくそのままCopy&Pasteで提示していただく方がトラブルの特定は早いです。

条件分岐が多いIF分の書き方について。

> 条件がかなり多くても対応できますでしょうか？？

この回答への補足

> おっしゃる通りです。

この回答への補足

論理和（または，||）よりも 論理積（かつ，&&）の方が結合が強いですから，

この回答への補足

if～elseはいくらでも繋がりますが、それをやるのは、あまりに無謀です。

この回答への補足

元のJavaソースコード中に「コード1」や「コード2(半角空白)」が登場するわけではないですよね。

この回答への補足

似たような質問が見つかりました

関連するカテゴリからQ&Aを探す

デイリーランキングこのカテゴリの人気デイリーQ&Aランキング

マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング

論理和（または，||）よりも論理積（かつ，&&）の方が結合が強いですから，