removeが反映されない。

Question

###前提・実現したいこと
キーボードから月日、予定、優先度を入力して表示するToDoリストを作っています。
配列をArrayListに変換し、入力した予定を削除プログラムをしようとしたところでエラーが発生しました。

改善策をご享受ください。

###発生している問題・エラーメッセージ
エラーメッセージは表示されず、コンパイルは通ります。
削除したい予定の番号を入力しても、反映されないです。

エラーメッセージ

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

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 remove1 = br.readLine();	
		ToDo.remove(Integer.parseInt(remove1)-1);//-1する　削除されない
		
		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); // 昇順になる
		}
	    }
	}
    }
}

###試したこと
num_itemsをToDo.sizeに書き換えましたが、エラーは改善されませんでした。

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

Comments

[swordone]

エラーメッセージがないのにエラーが発生したと判断している理由は何でしょうか。

Answer 1

まず大前提として…
変数名は小文字始まりで書いてください。
クラス名のToDoと変数名のToDoが共存していて読みにくいことこの上ないです。
それでなくても習慣としてクラス名は大文字始まり、変数名は小文字始まりで書いてください。

さて、いろいろおかしいので突っ込んでいきます。

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

    //ArrayListに変換
    ArrayList<ToDo> ToDo = new ArrayList<ToDo>(Arrays.asList(ToDoList));//<>の中にToDoを入れる(ToDoListはToDo型の配列であるため)

これも混乱の元ですね。配列なのに変数名にListが入っているし、さらに本当のArrayListが別にあるしで。

それはさておき、ToDoListとToDoで内容は共有していないことに注意してください。
問題のremove以外で、ToDoのリストToDoを触っている場所が一か所もありません。
それ以外は配列のToDoListをいじっているだけで、ToDoはノータッチです。
つまり、ToDoには最初から最後まで変化しません。最初の時点でnullが100個入っているだけです。
removeではこのnullのうち一つが削除されますが、結局表示しているのは配列の情報なので影響がないというだけの話です。

参考
コードをEclipseに突っ込んで変数ToDoをハイライト(宣言部を含めて3か所のみ)

Answer 2

ArrayListであるToDoを使っていたり、ToDo[]であるToDoListを使っていたり。
どちらか一方にしないと正常な動作は期待できないでしょう。

今回の場合は前者に統一すべきかと。

Comments

[nejirin_boy]

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

Answer 3

Eclipseにコードをコピペしてみたら大量の警告が出ました(実行自体はできましたが)

IDEの導入を推奨します

Comments

[nejirin_boy]

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

Answer 4

1,月順または優先度順にソートしていますが、ソートだとArrayList内部の順番が変わります。
予定を登録時にlong型のidを振って、削除時はidを元に削除するという形にしたほうが、ブレがないです。

2,以前質問者様が質問されたこの質問do〜while文の複数条件を適用させたいにて、
改善ソースコードとして配列からArrayListに変更したソースコードを記述したのですが、
目を通していらっしゃらなかったりしちゃいますか？

3,public static void main(String[] args)メソッドの中で処理を行いすぎです。
0. 予定を追加
0. 予定を月順で表示
0. 予定を優先度順で表示
0. 予定の削除

の4つのメソッドに分けたほうが各処理の影響範囲が少なくなるのでよいと思います。
自分で書いたソースコードを管理できなくなる前に対策することをお勧めいたします。

Comments

[nejirin_boy]

回答ありがとうございます。
以前改善していただいたソースコードは拝見いたしましたが、自分で考えたものを使おうと思っておりましたのでそのままにしていました。

[nejirin_boy]

ですが、自分で考えていたものがわからなくなってきていますので、
今後はumyu様に改善していただいたソースコードを参考にさせていただきます。

Answer 5

私は趣味でC++やっています。
Javaはあまりわかりませんが。

可能性として挙げるとすると、

---

エラーってどのようなエラーでしょうか?

もしかしてクラッシュするとか?

ソースコード読んでいないのでわかりませんが、もしかしたら例外が飛んできた可能性もありますね。

例えば要素数10しかないのに 要素番号100 にアクセスしようとしたとか。

その場合は例外処理を設ければいいはず。

それで、例外が飛んできた場合の処理に移ったら例外が飛んでいる証拠。

そうじゃないなら...どうでしょうね。

可能性1:
例外が飛んできたが補足( catch ) していないため強制終了

後は、C++でstd::vector ( Vector に相当? ) をunique したりする場合、その部分が残ったままになります。

vec[0] = データ1
vec[1] = データ2
vec[2] = NULL
vec[3] = データ3

みたいな感じで。

これみたいになっているかもしれない。

可能性2:
unique や delete した場合でもぬるぽになるだけで、要素数までは変わらない可能性。

さらに可能性2が深まって、ぬるぽなのに動かそうとしたとかでクラッシュ。

可能性3:
可能性2により、ぬるぽ状態だがそのまま動かそうとしたため例外が投げられているが、補足していないためクラッシュ

後は、ぱっと読んでいて、sortメソッドに ラムダ式っていうのでしょうか?

C++だと

[](引数){処理}

でやるやつを渡しているようですね。

C++の場合は[] の中に「参照であり、書き換えが可能である」みたいな明示が必要です。

それを行わないと変更ができないっていう場合がありますから。

Javaだと違うかもしれませんが。

可能性4:
ラムダ式の仕様によるもの

かなと。

ほかの可能性もありますが、今思いつくだけではこれだけ。

「エラーが出た」じゃなくてどういうエラーか明示したほうがいいです。

人によっては実行結果が違う場合も「エラーが出た」と表現する人いますからね。

( 某 知○袋では多い。 )

Comments

[nejirin_boy]

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