do〜while文の複数条件を適用させたい

Question

//降順にソートするプログラム
import java.util.Arrays;
import java.util.Comparator;
import java.io.*;

class ToDo{
    int number;
    int month;
    int day;
    String item;
    int priority;
}

class sort8
{
    public static void main(String[] args) throws IOException
    {
        BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
	
	ToDo[] ToDoList = new ToDo[100]; //配列の宣言
	
	for(int i = 0; i < 100; i++){
	    ToDoList[i] = new ToDo();
	}
	
	int num_items = 0;
	
	while(true){
	    System.out.println("予定を追加するなら1を入力。予定を月順で表示するなら２、優先度順で表示するなら３を入力。");
	    
	    String str = br.readLine();
	    int num = Integer.parseInt(str);
	    
	    if(num == 1){
		do{
		    System.out.println("月を入力して下さい(1〜12)");
		
		    String month = br.readLine();
		    ToDoList[num_items].month = Integer.parseInt(month);

		}while(13 <= ToDoList[num_items].month && ToDoList[num_items].month <= 0);
		//↑1〜12以外が入力されたら再入力を求める

		do{
		    System.out.println("日を入力して下さい(1〜31)");
		
		    String day = br.readLine();
		    ToDoList[num_items].day = Integer.parseInt(day);

		}while(32 <= ToDoList[num_items].day);	    
		
		System.out.println("予定");
		
		String item = br.readLine();
		ToDoList[num_items].item = item;


		do{
		System.out.println("優先度を入力して下さい(1〜5)");
		
		String priority = br.readLine();
		ToDoList[num_items].priority = Integer.parseInt(priority);

		}while(6 <= ToDoList[num_items].priority);
		
		num_items++;
	    }
	    else if(num == 2){
		System.out.println("予定です。(月昇順)");
		
		Arrays.sort(ToDoList , 0, num_items, new Comparator<ToDo>(){
			public int compare(ToDo todo1, ToDo todo2){
			    if(todo1.month < todo2.month){
				    return -1;
			    }
			    else if(todo1.month == todo2.month){
				if(todo1.day <= todo2.day){
				    return -1;
				}
				else{
				    return 1;
				}
			    }
			    else{
				return 1;
			    }
			}});
		
		for(int i = 0; i < num_items; i++){
		    System.out.println(ToDoList[i].month + "月" + ToDoList[i].day + "日 " + ToDoList[i].item + "優先度:" + ToDoList[i].priority); // 昇順になる
		}
	    }
	    else if(num == 3){
		System.out.println("予定です。(優先度昇順)");
		Arrays.sort(ToDoList , 0, num_items, (a, b)->a.priority - b.priority);
		
		for(int i = 0; i < num_items; i++){
		    System.out.println(ToDoList[i].month + "月" + ToDoList[i].day + "日 " + ToDoList[i].item + "優先度:" + ToDoList[i].priority); // 昇順になる
		}
	    }
	}
    }
}

上記のプログラムで月を入力させる時に1〜12以外の数字を入力した時に再入力を求める様にしたいのですが、うまくいきません。
改善策を教えて下さい。
よろしくお願い致します。

Answer 1

このような汎用静的メソッドを用意しておくと便利かもしれませんね。

static final BufferedReader br = new BufferedReader(
    new InputStreamReader(System.in)
);

static int readInt(String message) throws IOException {
    while(true) {
        try {
            System.out.print(message);
            return Integer.parseInt(br.readLine());
        } catch(NumberFormatException e) {}
    }
}

使用イメージ

int day;
do {
    day = readInt("日を入力して下さい(1～31): ");
} while(!(1 <= day && day <= 31));
        
System.out.println(day);

日を入力して下さい(1～31): hoge
日を入力して下さい(1～31): fuga
日を入力して下さい(1～31): 32
日を入力して下さい(1～31): -19
日を入力して下さい(1～31): 4
4

---

java.time.LocalDateを適宜利用した方がコードが綺麗になる気もします。
ToDoクラスが構造体然としているのもだいぶ気になります。

Comments

[umyu]

readIntですが、br.readLine()でコンパイルの参照エラーになるかと。

[LouiS0616]

ご指摘ありがとうございます。
staticフィールドにbrを持つイメージで書いてます。

[umyu]

なるほどー。失礼しましたー。

Answer 2

 }while(13 <= ToDoList[num_items].month && ToDoList[num_items].month <= 0);

判断するための条件が正しくないからです。

 } while (!(1 <= ToDoList[num_items].month && ToDoList[num_items].month <= 12));

全体的に変更したソースコードを記載します。
ご参考まで。

import java.util.List;
import java.util.ArrayList;
import java.util.Comparator;
import java.io.*;

class ToDo {
	int number;
	int month;
	int day;
	String item;
	int priority;

	@Override
	public String toString() {
		return this.month + "月" + this.day + "日 " + this.item + "優先度:" + this.priority;
	}
}

public class Q105888 {
	public static void main(String[] args) throws IOException {
		try (BufferedReader br = new BufferedReader(new InputStreamReader(System.in))) {
			//配列から動的配列(ArrayList)に宣言を変更。
			List<ToDo> ToDoList = new ArrayList<>();

			while (true) {
				System.out.println("予定を追加するなら1を入力。予定を月順で表示するなら２、優先度順で表示するなら３を入力。");

				String str = br.readLine();
				int num = Integer.parseInt(str);
				switch (num) {
				case 1:
					ToDo todo = new ToDo();
					do {
						System.out.println("月を入力して下さい(1〜12)");
						String month = br.readLine();
						todo.month = Integer.parseInt(month);
					} while (!(1 <= todo.month && todo.month <= 12));
					// ↑1〜12以外が入力されたら再入力を求める
					
					do {
						System.out.println("日を入力して下さい(1〜31)");
						String day = br.readLine();
						todo.day = Integer.parseInt(day);
					} while (32 <= todo.day);

					System.out.println("予定");
					String item = br.readLine();
					todo.item = item;

					do {
						System.out.println("優先度を入力して下さい(1〜5)");
						String priority = br.readLine();
						todo.priority = Integer.parseInt(priority);
					} while (6 <= todo.priority);
					
					ToDoList.add(todo);
					break;
				case 2:
					display_month_order(ToDoList);
					break;
				case 3:
					display_priority_order(ToDoList);
					break;
				}
			}
		}
	}

	private static void display_month_order(List<ToDo> ToDoList) {
		System.out.println("予定です。(月昇順)");
		ToDoList.sort(new Comparator<ToDo>() {
			@Override
			public int compare(ToDo todo1, ToDo todo2) {
				// 標準ライブラリのInteger.compareを使用すると、compareをシンプルに記述することができます。
				int month = Integer.compare(todo1.month, todo2.month);
				// ガード節による入れ子条件記述の置き換え
				if (month != 0) {
					return month;
				}
				return Integer.compare(todo1.day, todo2.day);
			}
		});
		ToDoList.forEach(t -> System.out.println(t));
	}

	private static void display_priority_order(List<ToDo> ToDoList) {
		System.out.println("予定です。(優先度昇順)");
		ToDoList.sort((a, b) -> a.priority - b.priority);
		ToDoList.forEach(t -> System.out.println(t));
	}
}

Comments

[nejirin_boy]

ベストアンサーが遅くなり申し訳ございません。
回答だけでなく、改善したソースコードも記載いただきありがとうございました。