ArrayListの要素をremoveを用いて削除。よろしく御願い致します。

Question

お世話になっております。
現在私はテキストファイルの入出力について勉強しております。
ある程度は完成したのですが、最後に入力された数値に応じて、listに追加されているデータを削除したいのですが、どうしてもエラーが出てしまい削除が出来ません。3日ほど自分なりに考えているのですが、どうしても分かりませんでした・・・。
エラー箇所は下から10行目あたりでコメントつけておきました。

どなたかお力をお貸し下さい。よろしく御願い致します。
//エラー内容
Exception in thread "main" java.lang.IndexOutOfBoundsException: Index: 0, Size: 0
at java.util.ArrayList.rangeCheck(ArrayList.java:653)
at java.util.ArrayList.remove(ArrayList.java:492)
at Text.main(Text.java:147)

import java.io.BufferedReader;
import java.io.BufferedWriter;
import java.io.File;
import java.io.FileReader;
import java.io.FileWriter;
import java.io.IOException;
import java.io.PrintWriter;
import java.util.ArrayList;

public class Text {
	 // 1人分のデータを格納するクラス
	class Person {

		private String name;
		private String address;
		private String tel;


		public String getName() {
			return name;
		}
		public void setName(String name) {
			this.name = name;
		}
		public String getAddress() {
			return address;
		}
		public void setAddress(String address) {
			this.address = address;
		}
		public String getTel() {
			return tel;
		}
		public void setTel(String tel) {
			this.tel = tel;
		}
	}

	private boolean checkBeforeReadFile(File file) {
		if (file.exists()) {
			if (file.isFile() && file.canRead()) {
				return true;
			}
		}
		return false;
	}


	// ファイルを読み込む
	public ArrayList<Person> readFile(File file) {
		int i = 0;
		ArrayList<Person> list = new ArrayList<Person>();
		BufferedReader br = null;
		try {
			if (checkBeforeReadFile(file)) {
				br = new BufferedReader(new FileReader(file));
				String line = null;
				Person person = null;
				while ((line = br.readLine()) != null) {
					switch (i) {
					case 0:
					// Person クラスのインスタンスを生成
						person = new Person();
						// 名前を格納
						person.setName(line);
						// 次は i = 1 → 住所
						i++;
						break;
					case 1:
						person.setAddress(line);
						i++;
						break;
					case 2:
						person.setTel(line);
						// 1人分のデータの格納が完了したので、リストに追加
						list.add(person);
						i = 0;
						break;
					}
				}
			}
		}
		catch (IOException e) {
			e.printStackTrace();
		}
		finally {
			if (br != null) {
				try {
					br.close();
				}
				catch (IOException e) {
					e.printStackTrace();
				}
			}
		}
		return list;
	}

	// 全員分のデータを表示する
		void showPeople(ArrayList<Person> list) {
			int j=0;
			for (Person person : list) {
				System.out.println("<No." + j + ">");
				System.out.println("名前：" + person.getName());
				System.out.println("住所：" + person.getAddress());
				System.out.println("電話番号：" + person.getTel());
				j++;
			}
		}

	//コンソール制御
	public static void main(String[] args) {
		File file = new File("C:\\pleiades\\workspace\\JavaTest\\src\\test.txt");
		Text text = new Text();
		ArrayList<Person> list = text.readFile(file);
		text.showPeople(list);
		System.out.println("0:新規登録 1:削除");
		int input = new java.util.Scanner(System.in).nextInt();
		try{
			if(input == 0){
				PrintWriter pw = new PrintWriter(new BufferedWriter(new FileWriter(file,true)));
				BufferedReader br = new BufferedReader(new FileReader(file));
				System.out.println("名前を入力してください");
				String name = new java.util.Scanner(System.in).nextLine();
				System.out.println("住所を入力してください");
				String address = new java.util.Scanner(System.in).nextLine();
				System.out.println("電話番号を入力してください");
				String tel = new java.util.Scanner(System.in).nextLine();
				pw.println(name);
				pw.println(address);
				pw.println(tel);
				pw.close();
				System.out.println("連絡先を追加しました");
			}
		}
		catch (IOException ex) {
			System.out.println("err");
		}
			if(input == 1){
			System.out.println("削除するデータを選んでください");
			int n = new java.util.Scanner(System.in).nextInt();
			for(int i=0; i<3; i++){
//ここでエラーが起き、削除が出来ない
				list.remove(n*3);
			}
             System.out.println("<No." + (n) + ">のデータを削除しました");
			try{
				PrintWriter pw = new PrintWriter(new BufferedWriter(new FileWriter(file)));
				for(Person result : list){
					pw.println(result);
				}
				pw.close();
			}
			catch (IOException ex) {
				System.out.println("err");
			}
		}
	}
}

Comments

[issei.]

ソースコードをコードタグで囲って下さい。それと、可能であればエラーのスタックトレースを貼って下さい(仮説の裏付けとして欲しいです)。

[akagami_bb]

申し訳ございません。この機能を知りませんでした。ご指摘ありがとうございました。

[issei.]

ありがとうございます！

[coco_bauer]

古いコード（一人分のデータを配列の連続した３要素（テキストファイルの３行）を使って保持する方式）と、新しいコード（一人分のデータを１つのclass Personのオブジェクトに保持する）が混ざっていませんか？　一人分のデータを削除するならlistから１項目（一人分）をremoveすれば良いのに、forループでremoveを３回繰り返しているのは、古いコードで３項目(名前、住所、電話番号）を順に削除していた名残のような気がします。

[akagami_bb]

ご回答ありがとうございます。
おっしゃるとおりで、編集しなおしたところ、上手く削除することが出来ました！また機会がありましたらよろしく御願い致します。

Answer 1

            System.out.println("削除するデータを選んでください");
            int n = new java.util.Scanner(System.in).nextInt();
            for(int i=0; i<3; i++){
//ここでエラーが起き、削除が出来ない
                list.remove(n*3);
            }

            System.out.println("削除するデータを選んでください");
            int n = new java.util.Scanner(System.in).nextInt();
            if( n >= 0 && n < list.size()){
                list.remove(n); // n番目の要素を削除
            }else{
                System.out.println("番号が範囲外です");
            }

現在環境が無いのでテストしていませんが、こんな感じになると思います。

Comments

[akagami_bb]

ご回答ありがとうございます。
上記の内容を編集して使用したところ、上手く削除をすることが出来ました。
今回はありがとうございました！また機会がありましたらよろしく御願いいたします。

Answer 2

何のエラーが出ているか分かりませんが、大方IndexOutOfBoundsExceptionではないですか？
remove(int)はArrayListの、引数の位置にある要素を削除するもので、引数が負またはArrayListのサイズ以上のとき例外になります。

Comments

[akagami_bb]

ご回答ありがとうございます。
エラー時に隣に数字がでるので、変更したところ上手く削除がいきました。
わざわざありがとうございました！

Answer 3

読みやすさなどを一端置いておいて、こうすれば動くように思います（ごめんなさい検証環境用意できてません）

• 変更前

for(int i=0; i<3; i++){
//ここでエラーが起き、削除が出来ない
  list.remove(n*3);
}

• 変更後

  list.removeRange(n*3, n*3 + 2);

Comments

[akagami_bb]

ご回答ありがとうございます。
しかし、「メソッド removeRange(int, int) は型 ArrayList<Text.Person> で不可視です」
と出てエラーになってしまいました。私のソースコードが悪いのだと思いますが、わざわざ回答してくださったのに申し訳ないです。

[swordone]

それはコードが悪いわけではなさそうです。
removeRangeはprotectedメソッドなので、「同クラス内、同一パッケージ内、またはサブクラス内」でのみ使用可能です。
この箇所はいずれにも該当しないのでそもそも使えないのです。
これを使いたいのなら、このメソッドをオーバーライドしておく必要があるでしょう。

[akagami_bb]

ご回答ありがとうございます。
また一つ勉強になりました。これから使用するときは気をつけたいと思います。

[issei.]

protectedでしたか、失礼しました。