0

在 person.txt 中，我们存储了此人的详细信息。好像是这样。

John
Smith
aösldkjf
5
8645
asdfasf
0441234545
++++++
Adam
Gilchrist
ads
asf
asf
asfd
0441234546
++++++

然后我们构建了 FileManager 类来从这个文件中读取数据。它标识有两个不同的条目。但它总是读取前 8 行并且不会继续前进。由于第一个人（例如：- John Smith）被两次添加到“LinkedList”中，命名为AddressBook。

//文件管理器类

public class FileManager {

    public static void readFile() {

        Scanner x;

        LinkedList<String> tempList = new LinkedList<String>();

        try {
            x = new Scanner(new File("Person.txt"));

            @SuppressWarnings("unused")
            String temp = null;

            while (x.hasNext()) {
                tempList.add(x.next());
                tempList.add(x.next());
                tempList.add(x.next());
                tempList.add(x.next());
                tempList.add(x.next());
                tempList.add(x.next());
                tempList.add(x.next());
                tempList.add(x.next());

                Person person = new Person();

                person.addFilePerson(tempList);

                Main.addressBook.add(person);

            }
        } catch (Exception e) {
            System.out.println("could't find the file");
        }
    }
}

// 类 Person 中的 addFilePerson 方法

public void addFilePerson(LinkedList<String> list){

    vorname = list.get(0);
    nachname = list.get(1);
    strasse = list.get(2);
    hausnummer = list.get(3);
    plz = list.get(4);
    telefon = list.get(5);
    wohnort = list.get(6);
}
4

5 回答 5

1

您正在创建一个 LinkedList<String>并反复添加它。移动这一行：

LinkedList<String> tempList = new LinkedList<String>();

进入while循环。或者 - 最好是 IMO - 对不同部分使用单独的属性：

// TODO: Consider what happens if the file runs out half way through a person...
while (x.hasNext()) {
    Person person = new Person();
    person.setFirstName(x.next());
    person.setLastName(x.next());
    person.setStreet(x.next());
    person.setTown(x.next());
    person.setTelephoneNumber(x.next());
    person.setCity(x.next()); // Or whatever...

    Main.addressBook.add(person);
}

Person围绕创建“构建器”类型并使其自身不可变还有其他选择Person，您可能想要创建一个单独的Address类型......

于 2012-04-11T13:10:49.727 回答
1

您应该在从文件中读取人员之间清除（或重新创建）您的列表。否则，您将继续将同一个人（您阅读的第一个人）添加到通讯录中。因此，要么按照 Jon 的建议每次在循环中重新创建你的临时列表，要么在每一轮之后清除它：

        while (x.hasNext()) {
            tempList.add(x.next());
            ...

            Main.addressBook.add(person);

            tempList.clear();
        }
于 2012-04-11T13:11:18.463 回答
1

它实际上继续前进。这一行：

person.addFilePerson(tempList);

您将 thetempList作为参数发送，但在addFilePerson方法中您总是阅读tempList的前 7 个条目。您应该在循环的每次迭代中清除tempList。

于 2012-04-11T13:14:19.667 回答
0

您应该使用 nextLine() 和 hasNextLine() 而不是 next() 和 hasNext()。扫描器是上下文感知的，因此默认的令牌读取行为可能不是基于行的。

于 2012-04-11T13:15:38.963 回答
0

你最好做

Person person = new Person();
Address address = new Address();
person.setAddress(address);

person.setFirstName(x.next());
person.setLastName(x.next());
address.setStreetName(x.next());
address.setHouseNumber(x.next());
address.setZipCode(x.next());
person.setPhoneNumber(x.next());
address.setCityName(x.next());
于 2012-04-11T13:18:08.883 回答