多次元リストの検索、要素の取り出しについて

Question

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;
import java.util.ArrayList;

class ArrayTest
{
	public static void main(String[] args) throws IOException
	{
		System.out.println("----------------------------------------------------");
		System.out.println("メインメニュー");
		System.out.println("1.学生情報新規登録 (*)");
		System.out.println("2.学生情報検索 (*)");
		System.out.println("3.学生情報削除");
		System.out.println("4.学生情報更新");
		System.out.println("5.閉じる (*)");
		System.out.println("----------------------------------------------------");

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

			System.out.println("番号を選択してください。");
			String str = br.readLine();
			int res = Integer.parseInt(str);

			switch(res) {
			case 1:InputT.input();
				main(args);
				break;
			case 2:SearchT.search();
				main(args);
				break;
			case 3:DeleteT.delete();
				main(args);
				break;
			case 4:/*Updata.updata();*/
				main(args);
				break;
			case 5:
				System.out.println("終了します。");
				System.exit(0);
				break;
			default:
				System.out.println("入力された番号に該当するメニューがありません。");
				main(args);
				break;
			}
	}
}

//リストの作成
class SL{
	static ArrayList listall;
	static ArrayList list;

	// 多次元リストの宣言
	static ArrayList<ArrayList<String>> person = new ArrayList<ArrayList<String>>();

	public static void create() throws IOException {
		ArrayList<String> ps = new ArrayList<>(4);
		String id = "ID";
		String name = "NAME";
		String birth = "BIRTHDAY";
		String adr = "ADDRESS";

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

		for(int i = 0; i < 1; i++) {
		System.out.println("IDを入力してください。");
		id = br.readLine();
		ps.add(id);
		System.out.println("\n名前を入力してください。");
		name = br.readLine();
		ps.add(name);
		System.out.println("\n生年月日を入力してください。(例：XXXX/XX/XX)");
		birth = br.readLine();
		ps.add(birth);
		System.out.println("\n住所を入力してください。");
		adr = br.readLine();
		ps.add(adr);
		}

		person.add(ps);

		listall = person;
		list = ps;
	}

	public ArrayList getlist1() {

		SL sl = new SL();
		return sl.listall;
	}
	public ArrayList getlist2() {

		SL sl = new SL();
		return sl.list;
	}
}

//登録
class InputT{

	static void input() throws IOException {

		//学生情報
			System.out.println("\n学生情報の登録をします。");
			SL sl = new SL();

			SL.create();

			ArrayList listZ = sl.getlist2();

			System.out.println("\n\n" + listZ);
			System.out.println("\n" + listZ.get(1) + "さんの情報を登録しました。\n");

	}
}

//検索
class SearchT{
	static void search() throws IOException {

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

		SL b = new SL();
		ArrayList listX = b.getlist1();
		ArrayList listZ = b.getlist2();

			System.out.println("検索する文字列を選択してください。");
			String moji = br.readLine();

			try{

				System.out.println(((ArrayList) listX.get(0)).get(1));
				System.out.println(moji + "のインデックスは" + listX.indexOf(moji) + "です。");
				System.out.println(listX);
				System.out.println(listZ);
				System.out.println("要素数 = " + listX.size());
				for(Object s:listX) {
					System.out.println(s);
				}
			}
			catch(java.lang.NullPointerException e){
				System.out.println("入力した文字列は登録されていません。");
				System.out.println("メニューに戻ります。");
			}
	}
}

//削除
class DeleteT{
	static void delete() throws IOException {
		BufferedReader br =
				new BufferedReader(new InputStreamReader(System.in));

		SL b = new SL();
		ArrayList listX = b.getlist1();
		ArrayList listZ = b.getlist2();

			System.out.println("削除する文字列を選択してください。");
			String moji = br.readLine();

			try{
				if (listZ.contains(moji));
				listZ.remove(moji);
				System.out.println(moji + "を削除しました。");
				System.out.println(listX);
			}
			catch(java.lang.NullPointerException e){
				System.out.println("入力した文字列は登録されていません。");
				System.out.println("メニューに戻ります。");
			}
	}
}

/*//更新
class Updata{
	static void updata() throws IOException {

}*/

Javaを勉強中の者です。
上記ソースでキーボードから値を受け入れる形式の多次元リストを制作しているのですが、格納した値を検索または削除する際、あらかじめpersonリストの中に格納しておいた複数のpsリストのインデックスが一部で最後に格納した値の物以外破棄されてしまっているようです。
具体例を挙げますと、あらかじめpsリストに[11,22,33,44]、[55,66,77,88]と値を格納し、それをさらにpersonリストにまとめて格納しています。この時点でpersonリストを出力すると[[11,22,33,44],[55,66,77,88]]となっています。
その後、indexOfで22のインデックスを調べるとperson、psリストのどちらからも-1、つまり格納されていないと返ってきます。しかし、personリストにget(0).get(1)で要素の取得をさせるとちゃんと22が返ってきます。
この場合、古い方([11,22,33,44])のインデックスを保持したままにして検索、削除が出来るようにするにはどこを修正すればよいのでしょうか。アドバイスの方お願い致します。

開発環境：Eclipse 4.7 Oxygen 3a(32bit版　Full Edition)

Answer 1

listallは常にpersonと同じリスト(「StringのArrayList」のArrayList)を参照し、listはcreate()のたびに最後に入力した人物データのリストで上書きされます。
結果、search()でやってることは、対象の文字列を、

• listall,つまり「StringのArrayList」のArrayListから探す
• list,つまり最後に入力された人物データから探す

ということをすることになります。
前者は、対象の文字列と一致する「StringのArrayList」を探すことになり、絶対に見つかりません。文字列とリストは違うものなので。
後者は、最後のデータしか探せないため、最初の人物データは当然探せません。

改善方法

前述の通り、listallとpersonは同じ参照なので、どちらかを消します。例えばpersonを消した場合、personにアクセスするコードは全てエラーになるので、listallに書き換えます。
未加工型を使うのもやめましょう。listallはArrayList<ArrayList<String>>,listはArrayList<String>で宣言します。
そして、listallからArrayListを一つずつ取り出して、それぞれに対してindexOfを使う、というのが最低限の使い方になります。

Comments

[friday_13]

ご回答ありがとうございます。参考にして改修してみます。

Answer 2

indexOfで22のインデックスを調べるとperson、psリストのどちらからも-1、つまり格納されていないと返ってきます。

System.out.println(((ArrayList) listX.get(0)).get(1));
System.out.println(moji + "のインデックスは" + listX.indexOf(moji) + "です。");

↑listXは「ArrayList<ArrayList<String>>」でしょ？
「ArrayList<ArrayList<String>>」に「indexOf(String)」かけてもそりゃ見つかりませんよ。

Comments

[friday_13]

ご指摘ありがとうございます。アイコンのハス太くんかわいいですね。
Javaに関しては浅学の身であり恥ずかしながら多次元リストの要素の取り出し方すらteratail様などで調べつつ入れている有様ですので、できれば改善案も併せてお教えいただけると助かります。キャストすれば解決するのでしょうか。

[tkturbo]

> キャストすれば解決
↑単なるキャストでは解決しません。
「((ArrayList) listX.get(0)).indexOf(moji)」ならArrayList<String>にindexOfしていることになります。

[friday_13]

重ねてお礼申し上げます。参考にさせて頂きます。

Answer 3

註: 『どのように解決するべきか』に主眼を置いた回答です。

今のコードをそのまま拡張すると非常にこんがらがったコードになります。
まずは、生徒の情報を管理するクラスを作ってはいかがでしょうか。

public class StudentInfo {
    private String name;
    private String birth;  // 文字列じゃない方が扱いやすいかも
    private String address;

    public StudentInfo(String name, String birth, String address) {
        this.name    = name;
        this.birth   = birth;
        this.address = address;
    }
}

そして、idと生徒情報をハッシュマップで結び付けます。

Map<Integer, Student> map = new HashMap<>();
map.put(1, new Student("Yamada", "2000/1/1", "Tokyo"));

このようにしておけば、Mapの機能を利用できるので種々の実装が一気に楽になります。

Comments

[friday_13]

ご回答ありがとうございます。Mapを使ったことがなかったので調べながらの質問になってしまうのですが、教えて頂いた
Map<Integer, Student> map = new HashMap<>();
map.put(1, new Student("Yamada", "2000/1/1", "Tokyo"));
の部分はpsリストを作っている部分をMapに作り替える形であってますでしょうか？

[LouiS0616]

そのとおりです。

[friday_13]

重ねてお礼申し上げます。頂いたご意見を参考に改修してみます。