JAVA:Arraylist 覆盖以前的条目

Question

这个问题已经有答案了:

Why does my ArrayList contain N copies of the last item added to the list? (5 个回答)

已关闭 8 年前。

初学者，请友善。

每当我通过“添加人员”添加到数组列表时，它都会覆盖以前的条目。因此，当我运行排序时，我会得到两个完全相同的条目，即最后输入的条目。我该如何解决？我该怎么办？

这是我的类文件:

import java.util.Calendar;
import java.util.Date;
import java.text.*;

public class Person implements Comparable {

    private static int totalNumber;

    public static int getTotal() {

        // Returns total number of employees
        return totalNumber;
    }

    private String fName;
    private String lName;
    private Date lastModified;
    private String address;
    private String city;
    private String state;
    private String zip;
    private String phone;

    public Person(String fName, String lName, String address, String city,
            String state, String zip, String phone) {
        this.fName = fName;
        this.lName = lName;
        this.address = address;
        this.city = city;
        this.state = state;
        this.zip = zip;
        this.phone = phone;
        Calendar calobj = Calendar.getInstance();
        this.lastModified = calobj.getTime();
    }

    public String getfName() {
        return this.fName;
    }

    public void setfName(String fName) {
        this.fName = fName;
    }

    public String getlName() {
        return this.lName;
    }

    public void setlName(String lName) {
        this.lName = lName;
    }

    public Date getLastModified() {
        return this.lastModified;
    }

    public void setLastModified(Date lastModified) {
        this.lastModified = lastModified;
    }

    public String getAddress() {
        return this.address;
    }

    public void setAddress(String address) {
        this.address = address;
    }

    public String getCity() {
        return this.city;
    }

    public void setCity(String city) {
        this.city = city;
    }

    public String getState() {
        return this.state;
    }

    public void setState(String state) {
        this.state = state;
    }

    public String getZip() {
        return this.zip;
    }

    public void setZip(String zip) {
        this.zip = zip;
    }

    public String getPhone() {
        return this.phone;
    }

    public void setPhone(String phone) {
        this.phone = phone;
    }

    @Override
    public String toString() {
        DateFormat df = new SimpleDateFormat("dd/MM/yy HH:mm:ss");
        return "\n First Name= " + fName + "\n Last Name= " + lName
                + "\n Address= " + address + "\n City= " + city + "\n State= "
                + state + "\n Zip= " + zip + "\n Phone= " + phone
                + "\n Last Modified= " + df.format(lastModified);
    }

    @Override
    public int compareTo(Object other) {
        // TODO Auto-generated method stub
        return this.lName.compareToIgnoreCase(((Person) other).lName);

    }

}

这是我的测试文件:

import java.util.*;

public class testAddressBook {

    public static void main(String[] args) {
        // TODO Auto-generated method stub

        ArrayList<Person> addressBook = new ArrayList<Person>();
        Person newPerson = new Person(null, null, null, null, null, null, null);

        @SuppressWarnings("resource")
        Scanner sc = new Scanner(System.in);

        boolean switcher = true;
        do {
            System.out.println("\n\tAddress Book Menu");
            System.out.println("\n\t\tEnter A to (A)dd Person ");
            System.out.println("\t\tEnter D to (D)elete Person");
            System.out.println("\t\tEnter M to (M)odify Person");
            System.out.println("\t\tEnter S to (S)earch Address Book ");
            System.out.println("\t\tEnter L to (L)ist ALL (sorted) ");
            System.out.println("\t\tEnter Q to Quit");
            System.out.print("\n\tPlease enter your choice: ");
            char choice = sc.nextLine().toUpperCase().charAt(0);

            while ((choice != 'A') && (choice != 'D') && (choice != 'M')
                    && (choice != 'S') && (choice != 'L') && (choice != 'Q')) {
                System.out
                        .println("Invalid choice!  Please select (A)dd, (D)elete, (M)odify, (S)earch, (L)ist or (Q)uit: ");
                choice = sc.nextLine().toUpperCase().charAt(0);
            }

            switch (choice) {
            case 'A':
                System.out.println("\nTo add a person, follow the prompts.");

                System.out.print("\nEnter First Name: ");
                newPerson.setfName(sc.nextLine());

                System.out.print("\nEnter Last Name: ");
                newPerson.setlName(sc.nextLine());

                System.out.print("Enter Address: ");
                newPerson.setAddress(sc.nextLine());

                System.out.print("Enter City: ");
                newPerson.setCity(sc.nextLine());

                System.out.print("Enter State: ");
                newPerson.setState(sc.nextLine());

                System.out.print("Enter Zip: ");
                newPerson.setZip(sc.nextLine());

                System.out.print("Enter Phone Number: ");
                newPerson.setPhone(sc.nextLine());

                addressBook.add(newPerson);

                System.out
                        .println("\nYou have successfully added a new person!");

                break;

            case 'D':

                break;
            case 'M':

                break;
            case 'S':
                Collections.sort(addressBook);

                for (int i = 0; i < addressBook.size(); i++) {
                    System.out.println(addressBook.get(i));

                }
                System.out.println();

                break;
            case 'L':

                break;
            case 'Q':
                switcher = false;
                System.exit(0);
                break;
            default:

            }

        } while (switcher != false);

    }
}

Answer 1

您需要在循环中实例化一个新的 Person。例如声明:

Person newPerson;

然后在 while 循环中实例化它:

switch (choice) {
        case 'A' :      
            System.out.println("\nTo add a person, follow the prompts.");       
            newPerson = new Person(null, null, null, null, null, null, null);
 .... so on

如果您也在循环外实例化它，则会导致创建一个额外的对象。