为什么我不能用 Java 填充数组列表

Question

这是我的方法：

public void Read() {
    try{
        File fXmlFile = new File("C://Students.xml");
        DocumentBuilderFactory dbFactory = DocumentBuilderFactory.newInstance();
        DocumentBuilder dBuilder = dbFactory.newDocumentBuilder();
        Document doc = dBuilder.parse(fXmlFile);

        doc.getDocumentElement().normalize();

        NodeList nList = doc.getElementsByTagName("student");

        Student s = new Student(); // Creating a new object

        for (int i = 0; i < nList.getLength(); i++) {
            Node nNode = nList.item(i);

            if (nNode.getNodeType() == Node.ELEMENT_NODE){
                Element eElement = (Element) nNode;

                s.First_Name = eElement.getElementsByTagName("firstname").item(0).getTextContent();
                Student.Last_Name = eElement.getElementsByTagName("lastname").item(0).getTextContent();

                Student.ID = eElement.getAttribute("id");
                Student.Spec = eElement.getElementsByTagName("spec").item(0).getTextContent();
                Student.Course = eElement.getElementsByTagName("course").item(0).getTextContent();

                Student.Ratingstr = eElement.getElementsByTagName("rating").item(0).getTextContent();

                students.add(new Student());
            }
        }
    }catch(Exception e){}
}

public static void main(String argv[]){
    Student y = new Student();
    y.Read();        
}

这些是我的类字段：

ArrayList<Student> students = new ArrayList<>();
public static String First_Name;
public static String Last_Name;
public static String ID;
public static String Spec;
public static String Course;
public static String Ratingstr;
public float Rating;

为什么我要填充第二个“s”对象或第二个学生而不是第一个？ 我的意思是当我在第一个周期中用我的第一个学生对象编写第一个索引时，我需要用第二个学生填充第二个索引，但我只用第二个学生填充第一个和第二个索引。我在第二个循环中覆盖了第一个索引。这是为什么呢？

Answer 1

不要使用静态字段。如果您使用静态字段，它们对于类的每个对象都是相同的。

Answer 2

首先，您不能将students 放在Student 类中。这不是一个好主意，您必须创建一个 ClassRoom 类和 students 列表：

类似：

class Student {
    public String firstName;
    public String fastName;
    public String id;
    public String spec;
    public String course;
    public String ratingstr;
    public float rating;
}

和

class ClassRoom  {
    ArrayList<Student> students = new ArrayList<>();

    // PUT HERE THE READ METHOD!!!!


    // optional:
    public String print() {
        // print each student inside room.students 
        for (Student s : room.students) {
            // you must have a Student.print() method!!!
            s.print();
        }
    }
}

然后：

public static void main(String argv[]) {
    ClassRoom room = new ClassRoom();
    room.read();

    room.print();
}

---

第二：你没有以正确的方式填充你的数组列表：

Student.Last_Name 

是静态引用类Student，而不是Student s实例，正确的方法是：

s.Last_Name 

此行在学生列表中插入一个新的 Student 空实例，不是您填写的 s 实例。

students.add(new Student());

所以你在 ClassRoom 类的 read 方法旁边的迭代将是这样的：

if (nNode.getNodeType() == Node.ELEMENT_NODE) {
    Element eElement = (Element) nNode;
    s.firstName = eElement.getElementsByTagName("firstname").item(0).getTextContent();
    s.lastName = eElement.getElementsByTagName("lastname").item(0).getTextContent();
    s.id = eElement.getAttribute("id");
    s.spec = eElement.getElementsByTagName("spec").item(0).getTextContent();
    s.course = eElement.getElementsByTagName("course").item(0).getTextContent();
    s.ratingstr = eElement.getElementsByTagName("rating").item(0).getTextContent();

    // add the FILLED instance of Student s to ClassRoom.students!!!
    students.add(s);
}

---

第三（可选）：

• 强烈建议您声明Student 属性private 并创建getters 和setters 以安全地修改它们。

• Java 约定说变量名必须以LOWERCASE 开头，Class 名称必须以UPPERCASE 开头，因此：class Student 和String firstName。

Answer 3

您需要在每次循环迭代中创建 Student 的一个新实例，您需要填充该实例，然后您需要将该特定实例（不是另一个新实例）添加到列表中，即：

    for (int i = 0; i < nList.getLength(); i++) {
        Node nNode = nList.item(i);
        if (nNode.getNodeType() == Node.ELEMENT_NODE) {
            Element eElement = (Element) nNode;
// create a new instance of the Student in every loop iteration
            Student s = new Student();// Creating a new object
// ...and then populate that instance:
            s.First_Name = eElement.getElementsByTagName("firstname").item(0).getTextContent();
...
// add that specific instance (not another new one) to the list
            students.add(s);

Answer 4

这样做。

s.First_Name = eElement.getElementsByTagName("firstname").item(0).getTextContent();
s.Last_Name = eElement.getElementsByTagName("lastname").item(0).getTextContent();
s.ID = eElement.getAttribute("id");
s.Spec = eElement.getElementsByTagName("spec").item(0).getTextContent();
s.Course = eElement.getElementsByTagName("course").item(0).getTextContent();
s.Ratingstr = eElement.getElementsByTagName("rating").item(0).getTextContent();

students.add(s);

同样在循环的开头，创建一个Student 的新实例。现在你只做了一个。每次运行循环时都应该制作一个。