javaでのif分岐

Question

前提・実現したいこと

こちら(http://kitako.tokyo/lib/JavaExercise.aspx?id=3)のサイトに掲載されいているjava練習問題集を利用させていただいているのですが、練習問題3-9が解けません。当方初心者ですので、大変初歩的な質問となってしまい恐縮ですがお力添えいただけると嬉しいです。

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

火曜/金曜で試してみると"予約可能です"と"誠に申し訳ございません。休診日です"のメッセージが同時に出力されてしまいます。

該当のソースコード

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;

public class E3_9 {

	public static void main(String[] args) {
		// TODO 自動生成されたメソッド・スタブ
		BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));

		try {

			System.out.println("希望診察日を入力してください 0:日曜1:月曜2:火曜3:水曜4:木曜5:金曜6:土曜");
			String line = reader.readLine();
			int date = Integer.parseInt(line);

			System.out.println("希望のお時間を入力してください0:午前1:午後2:夜間");
			line = reader.readLine();
			int time = Integer.parseInt(line);

			if(date != 0) {
				if(date ==1||date ==4) {
					System.out.println("予約可能です。");
				}if(date == 2) {
						if(time ==1 || time==2) {
							System.out.println("予約可能です。");
						}else {
							System.out.println("誠に申し訳ございません。休診日です	");
				}if(date == 5) {
					if(time ==1 || time==2) {
						System.out.println("予約可能です。");
					}else {
						System.out.println("誠に申し訳ございません。休診日です	");
					}
				}if(date == 3) {
					if(time ==0||time==1) {
						System.out.println("予約可能です。");
					}else {
						System.out.println("誠に申し訳ございません。休診日です	");
					}
				}else {
					if(time == 0) {
						System.out.println("予約可能です。");
					}else {
						System.out.println("誠に申し訳ございません。休診日です	");
					}
				}
				}
			}else {
				System.out.println("誠に申し訳ありません。休診日です。");
			}

		} catch (IOException e) {
			// TODO 自動生成された catch ブロック
			e.printStackTrace();
		}


	}

}

Answer 1

表をもとにすべてを分岐させるとどうしてもコードが長くなり、読みづらくなります

表をもとに開いていたらtrue,休診日ならfalseとして以下のような2次配列をあらかじめ用意します

        boolean [][]data = {
            {false,true,false,true,true,false,true},
            {false,true,true,true,true,true,false},
            {false,true,true,false,true,true,false}
        };

あとは配列の番号に曜日と時間帯のそれぞれに対応する番号が入り一つのif~else文で書けます

import java.util.*;

public class Main {
    public static void main(String[] args) {
        
        Scanner sc = new Scanner(System.in);
        boolean [][]data = {
            {false,true,false,true,true,false,true},
            {false,true,true,true,true,true,false},
            {false,true,true,false,true,true,false}
        };
        int week = Integer.parseInt(sc.nextLine());
        int time = Integer.parseInt(sc.nextLine());
        
        if(data[time][week]){
            System.out.println("予約可能です");
        }
        else{
            System.out.println("休診日です");
        }
    }
}

Answer 2

if分の使い方が間違っています。

if(date ==1||date ==4) {
    ～
}if(date == 2) {  // << ココ

これだと、date==1 || date == 4の場合はブロックの中を実行、それとは別に、次の date == 2なら次を実行となります。

あるブロック(date == 1 || date == 4)の条件が満たされなくて次に別の条件を書く場合は else if です。

if(date ==1||date ==4) {  // dateが1か4なら次のブロックを実行
    System.out.println("予約可能です。");
}else if(date == 2) {  // <↑ が満たされない場合にdateが2なら実行

Comments

[noir-robin]

迅速な回答ありがとうございました。早速試してみたところ、火曜日と金曜日で失敗していたところが正しく返されるようになりました! ありがとうございました。

Answer 3

それぞれのif文の中で、System.out.println("予約可能です。"); などを出力するのではなく、if文の結果を格納するbooleanを用意して、最後に文字列を出すようにするといいのではないでしょうか。

そうすると原因がわかりやすくなるでしょう。参考までに、

boolean result = false;

if (date != 0) {
  if (date ==1||date ==4) {
    result = true;
  } if (date == 2) {
    if (time ==1 || time==2) {
      result = true;
    }
  } if (date == 5) {
    .... 
  }
  ....
}

if (result) {
  System.out.println("予約可能です。");
} else {
  System.out.println("誠に申し訳ございません。休診日です    ");
}

Comments

[noir-robin]

迅速なご回答ありがとうございました。boolean型の変数を用意しておけば書いてくださったとおりすっきりするのですね。試してみます!