ArrayListに入っている数字をremoveで削除する方法

Question

###前提・実現したいこと
コマンドプロンプト上で動くToDoリストを作っています。
登録した予定を削除する機能を実装中に以下のエラーメッセージが発生しました。

改善策をご享受お願い致します。

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

sort14.java:107: エラー: シンボルを見つけられません
		ToDoList[i].remove(ToDo.indexOf(str2));
		         ^
  シンボル:   変数 i
  場所: クラス sort14
sort14.java:107: エラー: シンボルを見つけられません
		ToDoList[i].remove(ToDo.indexOf(str2));
		           ^
  シンボル:   メソッド remove(int)
  場所: クラス ToDo
エラー2個

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

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

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

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

	ToDo[] ToDoList = new ToDo[100]; //配列の宣言

	//ArrayListに変換
	ArrayList<ToDo> ToDo = new ArrayList<ToDo>(Arrays.asList(ToDoList));//<>の中にToDoを入れる(ToDoListはToDo型の配列であるため)
	
	for(int i = 0; i < ToDo.size(); i++){
	    ToDoList[i] = new ToDo();
	}

	int num_items = 0;

	while(true){
	    System.out.println("予定を追加するなら1を入力。予定を月順で表示するなら２、優先度順で表示するなら３を入力。予定の削除をするなら4を入力");
	    
	    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(!(1 <= ToDoList[num_items].month && ToDoList[num_items].month <= 12));

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

		}while(!(1 <= ToDoList[num_items].day && ToDoList[num_items].day <= 31));	    
		
		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(!(1 <= ToDoList[num_items].priority && ToDoList[num_items].priority <= 5));
		
		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(i+1  +". "+ 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(i+1 + ". " + ToDoList[i].month + "月" + ToDoList[i].day + "日 " + ToDoList[i].item + " 優先度:" + ToDoList[i].priority); // 昇順になる
		}
	    }
	    else if(num == 4){
		System.out.println("削除したい予定の番号を入力して下さい。(月順の場合)");
		
		String str2 = br.readLine();
		ToDoList.remove(ToDo.indexOf(str2));
		
		System.out.println("予定が削除されました。");
	    }
	}
    }
}

###試したこと
ToDoList.remove(ToDo.indexOf(str2));の行を何度か書き換えて試しましたが成功しませんでした。
###補足情報(言語/FW/ツール等のバージョンなど)
より詳細な情報

Answer 1

removeメソッドを実行するのは、「ToDoList」ではなく「ToDo」ではないでしょうか？
配列にはremoveメソッドは無いと思います･･･。

Comments

[nejirin_boy]

回答ありがとうございます。
ToDoに書き換えたところ、コンパイル通りました。ありがとうございます。
もう一点教えていただきたいことがありまして、キーボードから入力した値をremoveに反映させる方法を教えていただけないでしょうか？

[rinchu_17]

自分はコマンドプロンプト上で動くプログラムを全くと言っていいほど書いたことがないので、詳しいことは他の方に聞くかネットで調べた方がいいとは思いますが、気になったことだけ述べさせていただきますね。

ToDo.remove(ToDo.indexOf(str2));
この文についてですが、indexOf()の引数はToDo型のオブジェクトでなければならないはずです。
というのも、ToDo(リスト)はToDoオブジェクトをいくつも格納しているもので、indexOfメソッドは「リストが持ってるこのToDoオブジェクトがリストの中で何番目なのか教えてくれ」っていう命令なので、『このToDoオブジェクト』に該当する引数を渡さないといけないはずです。コンパイルは通るんですかね？そのへんはわからないですが。

それで今のコードを見てみると、indexOfの引数にString型の変数を渡しているので正しく動かないのだと思います。

解決法としては、コマンドプロンプトで入力された番号をもとにあらかじめToDoオブジェクトを特定しておいて、それをindexOfにぶち込む、といった感じでしょうか。

今ケータイから打っているので具体的なコードは割愛しますが、もし必要でしたら気軽に言ってください。微力ながらお手伝いします。

[退会済みユーザー]

すでにコードの中で行われているように、入力値をintに変換し、そこから1を引いてremove(int)に渡してやればよいです。indexOfは不要ですね。
なぜ、1を引くかは、一覧表示のとき1を足して表示しているからです。
あと、入力値のエラー判定（例えば範囲外の数値を入れられたとか）が大事です。
また削除のような重要な処理では、あとで復活させることができませんから、削除前に確認を促すなどがあったほうがよいでしょう。

[rinchu_17]

なるほど、たしかにそうですね。
indexOfの引数に気を取られて気付きませんでした。笑

ただ、質問者様のコード内のコメントを見た限りだと、おそらく「月順ないし優先度順で表示させたあとに、その表示順の番号を渡して削除」ということだと思いますので(違ったら申し訳ないですが)、月順または優先度順で表示させた際に別のToDo型リストか配列として保持しておいて、そのリストから与えられた番号-1のオブジェクトをget、それと同一のオブジェクトをindexOfに･･･とか色々考えたんですが、参照型云々で出来ないのかな～と。
もしくは月順優先度順表示の際に、ToDoリストのインデックスを表示順にint配列として保持しておくかですかね。
自分も経験が浅いのでこういうケースの最適解がわからないので教えていただければと！

[nejirin_boy]

ベストアンサーにさせていただきました。
遅くなってしまい申し訳ありません。