booleanの使い方

Question

研修で昨日からJAVAをやることになり課題を出されたのですが、booleanとelse～ifの組み合わせ方がいまいち分かりません。3時間考え抜いた末にできた中途半端なコードになりますが載せますので、どのように組めば良いか御教授願います。
課題　
int ageの値により以下を表示すること
5以下 無料
6－12歳 800円
13－22歳 1000円
それ以外 1500円
変数を用いて、条件式を終了した後にprintlnで料金を表示する
優待券をもっていたら100円引き
boolean yuutaiFlg
大人（それ以外）の女性の場合、水曜日であれば３００円引き
boolean WedFlg

public class Main {
	public static void main(String[] args) {
		int age = 1;
		int WOMAN = 1;
		boolean yuutaiFlg;
		boolean WedFlg;
		WedFlg = (WOMAN == 1);
		String str;
		if (age <= 5){
		str = "無料になります";
		}
		else if (age >= 6 && age <= 12){
		str = "800円になります";
        }
		else if (age >= 13 && age <= 22){
		str = "1000円になります";
		}
		else if (WedFlg){
		str = "1200円になります";
		}
		else
		{
		str = "23以上の男性は1500円になります";
		}
		System.out.println(str);
	}
}

Comments

[maisumakun]

えっと、現状のコードでは「どのような場合」に「どう動くべき」ものが「実際にはどのようになっている」のでしょうか？

そこを言葉にするのがスタートラインです。

[tsuru1]

失礼しました。
5歳以下の場合、無料と表示
6歳～12歳の場合、800円と表示される
13～22歳の場合、1000円と表示される
優待券を持っていたら100円引き
それ以外(男性)の場合、1500円と表示される
それ以外(女性でかつ水曜日)の場合、300円引かれて1200円と表示される
5～22歳まではint ageの値を変えれば想定される表示が出るのですが、boolean型を用いて100円引きにする方法と23歳以上で女性かつ水曜日の場合1200円にするためのコードの書き方が分からない状況です。

[momon-ga]

↑のうち、できていないのは、どこですか？というのが、maisumakunさんの聞いてることです。

５歳以下の場合、無料と表示はできている。
ｘｘｘは、できている。
ｘｘｘについて、ｙｙｙと表示したいのに、ｚｚｚと表示されてしまう。
のような感じです。

[tsuru1]

申し訳ございません。
5歳以下の場合、無料と表示できている
6歳～12歳の場合、800円と表示できている
13～22歳の場合、1000円と表示できている
---以下ができない-----
優待券を持っていたら100円引き
それ以外(男性)の場合、1500円と表示される
それ以外(女性でかつ水曜日)の場合、300円引かれて1200円と表示される
5～22歳まではint ageの値を変えれば想定される表示が出るのですが、boolean型を用いて100円引きにする方法と23歳以上で女性かつ水曜日の場合1200円にするためのコードの書き方が分からない状況です。

お二方ありがとうございます。

[m.ts10806]

「研修で」なら聞くべきは講師であって赤の他人ではないのでは。

Answer 1

考え方は大きく２通りかなと思います。

１．if文をネストさせる

else if (age >= 6 && age <= 12){
    if(yuutaiFlg){    
        str = "700円になります";
    } else {
        str = "800円になります";
    }
}

ネストしない場合は、最初の条件に含めることで同じことが可能です。

else if (age >= 6 && age <= 12 && yuutaiFlg){
    str = "700円になります";
}
else if (age >= 6 && age <= 12 && yuutaiFlg == false){
    str = "800円になります";

この書き方は、23歳以上で女性かつ水曜日の場合、かつ優待券の有無とかあるので、正直面倒なので
あまり使う人は少ないかと思います。

ネストするにしても、深くなるので好みではないです。

２．割引き金額を求めて計算する

int waribiki = 0;
if(yuutaiFlg){
    waribiki = waribiki + 100;
}
if(age >= 23 && WedFlg){
    waribiki = waribiki + 300;
}
// 割引きは重複する前提ですが、そうでない場合、if～elseを使って片方しか通らないように修正が必要

中略

   str = 1500 - waribiki + "円になります";

あと、WedFlgが女性という条件だけでtrueになってますが、正しい仕様でしょうか？

Comments

[tsuru1]

すみません、間違えてベストアンサーを押してしまいましたが、なんとか解決できました。
女性だけでtrueにしたのは試行錯誤段階で正しくはないです！

Answer 2

いろいろあるので、箇条書きとしていきます。

• まずは、「水曜日かどうか判定する」コードを書いてみましょう。その元データがなければ、「水曜日の場合は」という処理は不可能です。
• 条件分岐した後の結果を文字列で引き回していると、「100円を引く」という処理が素直に組めません。年齢別で分岐した処理の結果として、数値を入れておきましょう。