从文本文件写入Java ArrayList给我带来问题

Question

我的任务是从文本文件(有 2 列 - 名称和经理名称 - 以及 9 行(中提取所有记录，从每条记录(名称和经理(创建一个 Employee 对象，然后将 employee 对象添加到接受 Employee 对象的 ArrayList 中。

出于某种原因，当我稍后尝试遍历列表时，似乎最后一条记录是为所有记录存储的内容。

下面是我的代码。我什至放了一些打印行语句，以确保在添加令牌之前是正确的名称和经理名称，但是同样，当我之后循环遍历它时，所有记录都显示为"Veronica and Bob"。

这是文本文件：

Betty     Sam
Bob       Sally
Dilbert   Nathan
Joseph    Sally
Nathan    Veronica
Sally   Veronica
Sam Joseph
Susan   Bob
Veronica

这是我的方法：

List<Employee> employees = new ArrayList();
 public void loadData() throws FileNotFoundException, IOException {
    FileReader fr = new FileReader("C:\Users\jeff\Documents\NetBeansProjects\Java3Project1\src\java3project1\employee.txt");
    BufferedReader br = new BufferedReader(fr);
    String line;
    Employee e = new Employee();
    while ((line = br.readLine()) != null) {
        StringTokenizer st = new StringTokenizer(line);
        int counter = 0;
        while (st.hasMoreTokens()) {
            String token = st.nextToken();
            if (counter==0){
                e.setName(token);
                System.out.println("Name added: " + token);
            }
            if (counter == 1) {
                    e.setManager(token);
                    System.out.println("Manager set: " + token);
            }   
            employees.add(e);
            counter++;
        }//end while2
    }//end while1
} //end loadData()

当我在调用此方法后检查列表时，所有记录都显示为 姓名：维罗妮卡 经理：鲍勃

我

似乎无法弄清楚我做错了什么。非常感谢所有帮助。如果您有任何问题或需要更多澄清，请告诉我。

谢谢！

Answer 1

arraylist 中的所有员工都引用您在开始读取文件之前创建的同一e实例(他们都共享同一个实例(。

你需要改变

Employee e = new Employee();
while ((line = br.readLine()) != null) {

自

while ((line = br.readLine()) != null) {
    Employee e = new Employee();

为了在将它添加到数组列表之前，为每个Employee创建一个新的、单独的实例。

编辑：

你在内部循环的每次迭代(运行两次(结束时添加一个新员工，这就是你让记录翻倍的原因。

while ((line = br.readLine()) != null) {
    Employee e = new Employee();
    StringTokenizer st = new StringTokenizer(line);
    int counter = 0;
    while (st.hasMoreTokens()) {
        String token = st.nextToken();
        if (counter==0){
            // Set the name
        }
        if (counter == 1) {
            // Set the manager
        }
        // employees.add(e); // Move this line from here
        counter++;
    }//end while2
    employees.add(e); // to here : the name & the manager are now both set
}//end while1

Answer 2

您只在以下行创建一次 Employee 对象：

Employee e = new Employee();

然后，您只需一遍又一遍地为该 Employee 对象分配一个新名称，因此最终该对象具有文件中最后一个员工的名称。您需要为您读入的每个员工创建一个新的 Employee 对象。