java - HashMap 中的内容适当的数据

想象一下您有一本学生评价日记。每个学生在日记中都有每个科目的分数。我想将其存储在HashMap<>但我不明白为什么标记会合并。

在期刊课上:

public class Journal {
    private static HashMap<String, HashMap<String, ArrayList<Integer>>> journal = new HashMap<>(); // "Student" -> "Subject", mark[]

    private HashMap<String, ArrayList<Integer>> journalContainer = new HashMap<>();
    private ArrayList<Integer> marks = new ArrayList<>();

    public void addMark(String student, String subject, int mark) {
        marks.add(mark);
        journalContainer.put(subject, mark);
        journal.put(student, journalContainer);
    }

    public static void outputMarks() {
        for(HashMap.Entry<String, HashMap<String, ArrayList<Integer>>> entry : journal.entrySet())
        {
            System.out.println(entry.getKey() + "/" + entry.getValue());
        }
    }
}

在主课中:

public class Main {
    public static void main(String[] argc) {
        getJournal().addMark("Alex", "math", 4); // name, subject, mark
        getJournal().addMark("Alex", "math", 2);
        getJournal().addMark("George", "english", 2);
        getJournal().addMark("George", "english", 2);

        Journal.outputMarks();
    }
}

所以输出是：

Alex/{english=[4, 2, 2, 2], math=[4, 2, 2, 2]}
George/{english=[4, 2, 2, 2], math=[4, 2, 2, 2]}

但正确的输出应该是：

Alex/{math=[4, 2]}
George/{english=[2, 2]}

我不明白我做错了什么。有人可以帮忙吗？

问题是你有一个marks = new ArrayList<>();你在所有内在的价值观中使用的实例Maps，以及单个journalContainer = new HashMap<>();您在外部用作值的实例Map.

我会删除这两个实例变量，并使用局部变量。

你应该使用不同的ArrayLists 和内部HashMaps 作为你的价值观Maps:

public void addMark(String student, String subject, int mark) {
    HashMap<String, ArrayList<Integer>> journalContainer = journal.get(student);
    if (journalContainer == null) {
        journalContainer = new HashMap<>();
        journal.put(student,journalContainer);
    }
    ArrayList<Integer> marks = journalContainer.get(subject);
    if (marks == null) {
        marks = new ArrayList<>();
        journalContainer.put(subject, marks);
    }
    marks.add(mark);
}

顺便说一句，拥有一个修改静态成员的实例方法是没有意义的（journal Map）。要么使该方法静态，要么使Map非静态。