Warum gibt diese Methode eine ArrayList mit den gleichen Objekten zurück?

Question

JAVA

Ich habe eine Klasse namens User und eine Datei Users.csv, wie unten genannt:

Benutzerklasse:

public class User 
{ 
private String name; 
private String rg; 
private String type; 

public String getName() 
{ 
    return name; 
} 

public String getRg() 
{ 
    return rg; 
} 

public String getType() 
{ 
    return type; 
} 

public void setName(String n) 
{ 
    name = n; 
} 

public void setRg(String r) 
{ 
    rg = r; 
} 

public void setType(String t) 
{ 
    type = t; 
} 
} 

Users.csv:

a,b,c 
d,e,f 
as,d,cf 

Auch ich habe eine Klasse namens test, die eine einzige Methode implementiert:

Testklasse:

public class test 
{ 
public ArrayList<User> readUsers(String path) throws IOException //read all users from file and returns an ArrayList 
{  
    ArrayList<User> list = new ArrayList<User>(); 
    String[] str; 
    User u = new User(); 
    BufferedReader buffRead = new BufferedReader(new FileReader(path)); 
    String line = buffRead.readLine(); 
    while(line != null) 
    { 
      str = line.split(","); 
      u.setName(str[0]); //sets each field read to "u" 
      u.setRg(str[1]); 
      u.setType(str[2]); 
      list.add(u); //adding user to the list to be returned 
      line = buffRead.readLine(); //reading next register 
    } 
buffRead.close(); 
return list; 
} 
} 

Das Problem ist in dem Verfahren readUsers(). Es gibt mir eine ArrayList zurück, in der jedes Element gleich ist, und sie sind die in der letzten Zeile von Users.csv.
Irgendein Tipp, warum es passiert? Ich kann es nicht herausfinden ...

Answer 1

Wie oft rufen Sie new User()? Das ist, wie viele Benutzerobjekte Sie erstellt haben: genau ein, und Sie tun dies einmal vor die Schleife.

Lösung, rufen Sie diese innerhalb der Schleife - auf diese Weise erstellen Sie einen neuen Benutzer für jedes Element, das Sie der Liste hinzufügen.

Answer 2

Sie haben nur eine einzige Instanz von User erstellt und ändern einfach ihre Eigenschaften. Sie müssen eine neue Instanz von User für jeden Eintrag in der CSV-

//User u = new User(); 
//... 
while(line != null) 
{ 
     str = line.split(","); 
     User u = new User(); 
     u.setName(str[0]); //sets each field read to "u" 
     u.setRg(str[1]); 
     u.setType(str[2]); 
     list.add(u); //adding user to the list to be returned 
     line = buffRead.readLine(); //reading next register 
} 

Answer 3

Java Sammlungen speichern Referenzen-Datei erstellen. Daher gibt es in Ihrem Code ein einziges Benutzerobjekt, und der Verweis darauf wird bei jeder Iteration zur ArrayList hinzugefügt (anders als in C++, wo der Vektor eine Kopie erstellt). Sie brauchen etwas wie:

while(line != null) 
{ 
    str = line.split(","); 
    User u = new User(); 
    u.setName(str[0]); //sets each field read to "u" 
    u.setRg(str[1]); 
    u.setType(str[2]); 
    list.add(u); //adding user to the list to be returned 
    line = buffRead.readLine(); //reading next register 
}