在不移动迭代器的情况下从链表返回值

Question

我有两个类:学生和注册。注册表是用来保存学生名单的。当我尝试从注册表类返回学生的值时，迭代器移动到下一个学生以获得下一个值，因此当只有2个学生存在时，我得到nosuchelement异常。

public String format() 
{
    ListIterator<Student> iterator 
            = studentList.listIterator();
    for (int i = 0; i < studentList.size(); i++)
    {
        System.out.println("if Statement");
        if (iterator.hasNext())
        {
            System.out.println("hasNext");
            return String.format("%0s%0s%0s", 
            iterator.next().getSurName(), iterator.next().getForeName(),
            iterator.next().getStudentId());
        }
    }        
    return null;
}

我希望它返回stud1的所有3个值，然后移动到stud2，等等。

这是一个指向Student和Registry源文件的链接，以及两个源文件的测试程序;https://www.dropbox.com/sh/ol2jyfbuyyjnrcg/1J4LlEaqc6

我会张贴更多的代码，如果它的要求。

由于答案已关闭，因此没有更多的代码和链接中的文件已被删除。

Answer 1

每次调用.next()时，都在推进迭代器。对于当前迭代，只调用一次。

一样:

Student res = iterator.next();
return String.format("%0s%0s%0s", res.getSurName(),res.getForeName(),res.getStudentId());

Answer 2

要在不使用Iterator的情况下遍历列表，应该使用Enhanced for循环。然后你的代码变成-

public String format() 
{
ListIterator<Student> iterator 
        = studentList.listIterator();
for (Student student: studentList)
{
    System.out.println("For Loop");       
        return String.format("%s%s%s", 
        student.getSurName(), student.getForeName(),
        student.getStudentId());     
}   
 return null;   
}

Answer 3

您应该只调用一次next方法，因为每次调用都会转发光标。

if (iterator.hasNext()) {
    Student currStudent = iterator.next();
    //use currStudent instead of calling next

我希望它返回stud1的所有3个值，然后移动到stud2，等等。

由于迭代器是方法的局部迭代器，因此一旦方法返回，它就会消失。我认为最好将您的format方法改为static(实用)方法，并将其签名更改为以Student作为参数。public static String format(Student student)。这样就可以在遍历列表时格式化所有学生。

---

你可以这样写一个实用程序方法:

public static String format(Student student) {
    return String.format("%0s%0s%0s", student.getSurName(), student.getForeName(), student.getStudentId());
}

对于可以格式化所有学生并返回格式的方法-

public List<String> formatAll() {
    ListIterator<Student> iterator 
            = studentList.listIterator();
    List<String> formats = new ArrayList<String>();//to keep formats of all students
    //for (int i = 0; i < studentList.size(); i++)
    //{
        //System.out.println("if Statement");
        while (iterator.hasNext()) //while loop is better for iterators
        {
            Student currStudent = iterator.next();
            formats.add(format(student)); //calling utility method here
        }
    //}        
    return formats; // finally return the list of formats
}