NoElementException 但我打印元素并得到预期结果

Question

我想做的是将 Move 对象保存到名为 topMoves 的 Vector 中。 将会有许多 Move 对象，这就是我在循环内创建对象的原因。

PastPriceMap 存储过去某个时间（在本例中为一分钟前）的股票价格。 currPriceMap 存储最后一秒内某个时间的股票价格。

我收到以下异常：

Exception in thread "Timer-0" java.util.NoSuchElementException

这是导致问题的行： amove.setInitPrice(pastPriceMap.get(iter.next()));

代码片段如下。 当我执行 System.out.println 语句时，我得到了预期的输出：

Iterator<String> iter = sortedTopCodes.iterator();

while(iter.hasNext()){

    System.out.println(currPriceMap.get(iter.next()));
    System.out.println(pastPriceMap.get(iter.next()));

        Move amove = new Move();
    amove.setSecCode(iter.next());
    amove.setPrice(currPriceMap.get(iter.next()));
    amove.setInitPrice(pastPriceMap.get(iter.next()));
    topMoves.add(amove);
}

    return topMoves;

---

Move 类如下所示：

private String secCode;
private double price;
private double initPrice;

public String getSecCode() {
    return secCode;
}
public void setSecCode(String secCode) {
    this.secCode = secCode;
}
public double getPrice() {
    return price;
}
public void setPrice(double price) {
    this.price = price;
}

public double getInitPrice() {
    return initPrice;
}
public void setInitPrice(double lastPrice) {
    this.initPrice = lastPrice;
}

Answer 1

// while there are more lines
while(scanner.hasNextLine())
{
    final String   line;
    final String[] words;

    // get the next line
    line = scanner.nextLine();

    // break the line up into the words (\\s+ should break it up via whitespace)
    words = line.split("\\s");

    if(words.length != 5)
    {
        throw new WhateverExceptionMakesSense(line + " must contain 5 words");
    }

    System.out.println(currPriceMap.get(words[0]));
    System.out.println(pastPriceMap.get(words[1]));

    Move amove = new Move();
    amove.setSecCode(words[2]);
    amove.setPrice(currPriceMap.get(words[3]));
    amove.setInitPrice(pastPriceMap.get(words[4]));
    topMoves.add(amove);
}

Answer 2

简短回答：

对于每次调用 hasNext() ，应该只调用一次 next()

在您的代码中，您有 5 个 next()只有一个 hasNext()

在这里，阅读以下内容： http://java.sun.com/javase/6/docs/api/java/util/Iterator.html

编辑

更长的答案：

基本上迭代器是用于......很好地迭代“某物”的元素，通常是一个集合，但它可以是任何东西（假设任何东西返回一个 Iterator ）。

由于您可能不知道该“任何东西”有多少个元素，因此必须有一种方法来停止迭代，对吗？ （如果它是一个数组，您可以通过 length 属性来判断，但迭代器用于“封装”实现中使用的数据结构）无论如何。

迭代器 API 定义了这两个方法

-hasNext(): boolean 
-next(): Object ( or <E> since Java 1.5 ) 

因此典型的习惯用法是这样的：

 while( iterator.hasNext() ) { // reads: while the iterator has next element
      Object o = iterator.next(); //  give me that element
 }

如果迭代器只有两个项目会发生什么？

 while( iterator.hasNext() ) { // the first time will return true, so the next line will be executed.

      Object o = iterator.next(); // give me that item. ( 1st element ) 

      Object b = iterator.next(); // oops dangerous by may work ... ( 2nd element ) 

      Object c = iterator.next(); // eeeerhhh... disaster: NoSuchElementException is thrown.

}

这就是发生在你身上的事情。 您没有验证迭代器是否有另一个元素，您只是检索它。 如果迭代器碰巧有一些元素，它可能会工作一段时间，但有一段时间（正如您刚刚看到的）它会失败。

顺便说一句，不要考虑捕获 NoSuchElementException。 这是一个运行时异常，它表明代码逻辑中的某些内容应该修复。

请参阅此答案了解有关异常的更多信息。

Answer 3

这是使用新 for 循环的版本：

for ( String secCode : secCodeList ) {

        System.out.println(currPriceMap.get(secCode));
        System.out.println(pastPriceMap.get(secCode));

        Move amove = new Move();
        amove.setSecCode(secCode);
        amove.setPrice(currPriceMap.get(secCode));
        amove.setInitPrice(pastPriceMap.get(secCode));
        topMoves.add(amove);
}

以旧方式：

String secCode = null;    
for ( Iterator<String> it = secCodeList.iterator(); it.hasNext() ) {
    secCode = it.next();
    System.out.println(currPriceMap.get(secCode));
    System.out.println(pastPriceMap.get(secCode));

    Move amove = new Move();
    amove.setSecCode(secCode);
    amove.setPrice(currPriceMap.get(secCode));
    amove.setInitPrice(pastPriceMap.get(secCode));
    topMoves.add(amove);
 }