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

Question

###前提・実現したいこと
Javaで入力した予定を追加・削除するプログラムを作っています。
削除する機能を実装中に問題が発生しました。

118行目のdo-while分の継続条件がうまく反映されません。
自分では予定を削除したいときに追加されている予定の数以外の数字を入力した時に
数字の再入力を求める様にしたいのですが、
予定の数以外の数字を入力すると、プログラムが終了してしまいます。

適用条件を他の変数に変えて試したのですが、うまくいきませんでした。

改善策を享受いただけないでしょうか？

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

予定削除の時に予定の数以外の数字を入力すると、以下の内容が表示されます。
Exception in thread "main" java.lang.IndexOutOfBoundsException: Index: 1, Size: 1
	at java.util.ArrayList.rangeCheck(ArrayList.java:653)
	at java.util.ArrayList.remove(ArrayList.java:492)
	at sort24.main(sort24.java:117)

###該当のソースコード

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

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

class sort24
{
    public static void main(String[] args)throws IOException
    {
	BufferedReader br = new BufferedReader(new InputStreamReader(System.in));

	ArrayList<ToDo> todo = new ArrayList<ToDo>();
	
	while(true){
	    
	    System.out.println("予定を追加するなら1を入力。");
	    System.out.println("予定を月順で表示するなら2を入力");
	    System.out.println("予定を優先度順で表示するなら3を入力");
	    System.out.println("予定を削除するなら4を入力");
	    
	    String str = br.readLine();
	    int num = Integer.parseInt(str);
	    
	    ToDo tmp = new ToDo();

	    switch(num){
	    case 1:
		
		do{
		    System.out.println("月を入力して下さい(1〜12)");
		
		    String month = br.readLine();
		    tmp.month = Integer.parseInt(month);

		}while(!(1 <= tmp.month && tmp.month <= 12));

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

		}while(!(1 <= tmp.day && tmp.day <= 31));	    
		
		System.out.println("予定");
		
		String item = br.readLine();
	        tmp.item = item;

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

		}while(!(1 <= tmp.priority && tmp.priority <= 5));

		System.out.println("予定が追加されました。");
		
	        todo.add(tmp);

		break;
		
	    case 2:
		System.out.println("予定です。(月昇順)");
		
		todo.sort(new Comparator<ToDo>() {
			
			public int compare(ToDo todo1, ToDo todo2) {
		    
			    int month = Integer.compare(todo1.month, todo2.month);
		    
			    if (month == 0) {
				return Integer.compare(todo1.day, todo2.day);
			    }
			    else {
				return month;
			    }
			}
		    });
		
		System.out.println("==========");
		
		for(int i = 0; i < todo.size(); i++){
		    System.out.println(i + 1 +". "+ todo.get(i).month + "月" + todo.get(i).day + "日 " + todo.get(i).item + " 優先度:" + todo.get(i).priority); // 昇順になる
		}

		System.out.println("==========");
		break;
		
	    case 3:
		System.out.println("予定です。(優先度昇順)");

		todo.sort((a,b)-> a.priority - b.priority);

		System.out.println("==========");
			
		for(int i = 0; i < todo.size(); i++){
		    System.out.println(i + 1+". "+ todo.get(i).month + "月" + todo.get(i).day + "日 " + todo.get(i).item + " 優先度:" + todo.get(i).priority); // 昇順になる
		}

		System.out.println("==========");
		break;
		
	    case 4:
		System.out.println("削除したい予定の番号を入力して下さい。(月順の場合)");

		do{
		    String remove1 = br.readLine();
		    todo.remove(Integer.parseInt(remove1)-1);//-1する
		}while(!(1 <= todo.size()));//ここがうまくいかない。
		
		System.out.println("予定が削除されました。");

		System.out.println("==========");
		  
		for(int i = 0; i < todo.size(); i++){
		    System.out.println(i + 1 +". "+ todo.get(i).month + "月" + todo.get(i).day + "日 " + todo.get(i).item + " 優先度:" + todo.get(i).priority); // 昇順になる
		}

		System.out.println("==========");
		break;
	    }
	}
    }
}

###試したこと
継続条件の変数を色々変えて試しましたが、うまくいきませんでした。

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

Answer 1

そもそも入力したインデックスが予定リストの範囲外(リストサイズより大きい数値)の場合、removeで例外が発生してしまい、whileでは捕まえられません。
removeの前にその数値が正しいかをチェックする必要があります。

Comments

[nejirin_boy]

ご回答ありがとうございました。

Answer 2

do～whileだと処理後に判定されるため、whileを使った方がいいのではないでしょうか？

Comments

[unz.hori]

まず、todoにデータが存在する場合、かつ指定されたindex値が正しい場合に削除を行うべきですね。

[nejirin_boy]

ご回答ありがとうございました。