Listの中にある重複を取り除くには【Java】

Question

前提・実現したいこと

eclipseでcsvファイルを読み込み、Listに格納しています。
きちんと格納で来ているかも怪しいのですが、ここからさらに、重複しているデータを一つにまとめたいです。

以下のようなcsvファイルがあり、それを読み込んだという前提があります。
（以前にも似たような質問をさせていただいたのですが、クラスはなくても良いことになりました。）

山田太郎,数学,98
太田靖,国語,79
南義弘,理科,65
藤波健吾,社会,78
太田靖,社会,77
田中康子,理科,88
岡崎理恵,国語,77
山田太郎,社会,68
（以下制限は特になく続く）

その上で、例えば山田太郎は2回登場していますが、この重複をなくし、出力結果には

山田太郎,合計点:（calcSumなどで算出）,平均点:（calcAverageなどで算出）

とされるようにしたいと考えています。

cvsファイルの読み取りクラスとPersonクラスが存在しており、calcSum()メソッドはPersonクラスに実装しました。

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

重複した名前を持つデータの結合？の仕方が掴めない

該当のソースコード

// 読み取りクラス
public void readCsv(){
    
    List<String> data_a = new ArrayList<String>();
    String data_s[] = {null}; // 要素数未定のためnull

    File fl = new File("testdata");

    try(FileReader fr = new FileReader(fl));
        BufferedReader in = new BufferedReader(fr){
        
        String line;
        while((line = in.readLine()) != null){
            data_s = line.split(",", 0);
            Person person = new person(data[0], data[1]);

            for(int i = 0; i >= data_s.length ;i++){
                // indexOfなど使って重複しているデータを探す？
            }
        }
    }
}

// Personクラス
public class Student(){

    private String name;
    private List<Subject> subjects = new ArrayList<Subject>(); // 受けた教科を入れる

　　// コンストラクタ
    public Student(String _name){
        name = _name;
    }

    // メソッド
    // 受けた教科の合計点数を算出したい
    public Bigdecimal calcSum(){
        // BigDecimalの初期化
        for(Subject subject : subjects){
            result.add(subject.getscore());
        }
        return result;
    }
}

試したこと

indexOfを使ってみたのですが、それだとindexが取得できるだけなのでうまくいきませんでした。

補足情報（FW/ツールのバージョンなど）

Comments

[momon-ga]

重複がないようにデータを保持したいのか、それとも重複したデータをマージしたいのかで変わると思いますが。

> indexOfを使ってみたのですが、それだとindexが取得できるだけなのでうまくいきませんでした。

indexが取得できるなら、オブジェクトを取得できますよね？
※　そのコードを載せてみれば？

[momon-ga]

結局、Personなの？
> Person person = new person(data[0], data[1]);
↑これは、そもそもコンパイルエラーでしょ？
Studentなの？
> public Student(String _name)

[R.Mizukami]

> きちんと格納で来ているかも怪しいのですが
そこを確認せずに次に進むのは無謀です。格納したデータをそのまま出力してみるなど、まずは重複排除以前の部分が適切に作成できているか確認してください。

また、提示コードのエラー箇所が多すぎてどう説明していいか困るレベルなので、せめてコンパイルできるコードを作成し、コピペで記載いただけませんか。
9 行目からのの try のカッコの対応がおかしい
13 行目の while のカッコの対応がおかしい
new person の大文字小文字間違い 等

Answer 1

HashMapへ名前をキーにしてStudentを格納すれば、名前の重複は排除できると思います。
containsKeyメソッドで、既にキー値が格納されているかどうか確認できます。

HashMap (Java Platform SE 8)

Answer 2

次の質問すべてに答えていただければ、まともな回答ができると思います。

1. 読み取りクラスの名前は何ですか？

readCsv はメソッド名であり、クラス名ではありません。
Java のプログラムは、クラスの中にフィールやメソッドがあり、
メソッドの中に変数の宣言や文があるという構造になっています。
いきなり readCsv というメソッドで始まるコードは変です。

1. data_a は readCsv のローカル変数でよいのですか？

ローカル変数は、メソッドの実行が終了すると無くなってしまいます。これを
防ぐためには、readCsv がそのローカル変数の値を return で呼び出し元へ返すか、
あるいは、data_a を読み取りクラスのフィールドにして、
readCsv の呼出し終了後にも、保持する必要があります。

1. Person person = new person(data[0], data[1]) とありますが、

Personクラスの定義はどこにありますか？
data の宣言はどこにありますか？

1. result.add(subject.getscore()); とありますが、

result の宣言はどこにありますか？

すべての質問に答えてください。
分からなければ、何が分からないのかを詳しく書いてください。

質問に多くの人からアドバイスをもらってもそれに応えないのは
かなり混乱しているからでしょう。

ではアプローチを変えてみます。
次のコードは思った通りに動きますか？
この問いになら返答できるでしょう。

import java.io.*;
import java.util.*;

class Subject {
	private String name;
	private int score;

	public Subject(String name, int score) {
		this.name = name; this.score = score;
	}

	public String getName() { return name; }
	public int getScore() { return score; }
}

class Student {
    private String room;
    private String name;
    private List<Subject> subjects = new ArrayList<Subject>();

    public Student(String room, String name) {
        this.room = room; this.name = name;
    }

	public void add(Subject subject) { subjects.add(subject); }

	public boolean equals(Object o) {
		Student st = (Student)o;
		return name.equals(st.name);
	}

    public int calcSum(){
		int sum = 0;
        for (Subject sj : subjects) sum += sj.getScore();
        return sum;
    }

	public int subjectCount() { return subjects.size(); }

    public String getRoom() { return room; }
    public String getName() { return name; }
    public List<Subject> getSubjects() { return subjects; }
}

// 読み取りクラス
class DataReader {
	private List<Student> data_a = new ArrayList<Student>();

	public List<Student> getData() { return data_a; }

	public void readCsv() {
		File fl = new File("testdata");
		try (FileReader fr = new FileReader(fl);
				BufferedReader in = new BufferedReader(fr)) {

			String line;
			while ((line = in.readLine()) != null) {
				String[] data_s = line.split(",");
				Student st = new Student(data_s[0], data_s[1]);
				Subject sj = new Subject(data_s[2], Integer.valueOf(data_s[3]));
				int i = data_a.indexOf(st);
				if (i >= 0)
					data_a.get(i).add(sj);
				else {
					st.add(sj); 
					data_a.add(st);
				}
			}
		} catch (IOException ex) {
			 ex.printStackTrace();
		}
	}
	public void showData() {
		for (Student st : data_a) {
			System.out.print(st.getRoom() + " " + st.getName() + ":");
			List<Subject> subjects = st.getSubjects();
			for (Subject sj : subjects)
				System.out.print(" " + sj.getName() + ":" + sj.getScore());
			System.out.println();
		}
	}

	public void calcData() {
		for (Student st : data_a) {
			int sum = st.calcSum();
			double ave = (double)sum / st.subjectCount();
			System.out.println(st.getName() + ", 合計点:" + sum + ", 平均点:" + ave);
		}
	}
}

class Main {
	public static void main(String[] args) {
		DataReader reader = new DataReader();
		reader.readCsv();
		reader.showData();
		reader.calcData();
	}
}

このコードは一つの例であり、質問者の要求に完全に一致するものではない
かもしれません。これを参考に自分のコードが書けるようになってほしいのです。

回答にコードをもらった人の中には、「できました」とか「解決しました」と
いう言葉を残して去っていく人がいます。
本当に内容を理解したのでしょうか？
そうでなければ、何の解決にもなりません。

とにかく動くコードが得られたのですから、文がどの順番に実行され、
変数がどのように変化するのかを追っていかなければなりません。
分からないところがあればそれを調査し、それでも分からなければ
どんどん質問しましょう。

回答へのコメントをお願いします。