2013-10-01 47 views
9

我想通过使用迭代器遍历我的日志列表遍历列表。目标是搜索包含与新日志相同的电话号码,类型和日期的日志java.util.NoSuchElementException在java中使用迭代器

但是,在我的条件语句中出现java.util.NoSuchElementException。有谁知道什么可能会导致问题?

我的代码

public void addLog(String phonenumber, String type, long date, int incoming, int outgoing) 
{ 
    //Check if log exists or else create it. 
    Log newLog = new Log(phonenumber, type, date, incoming, outgoing); 

    //Log exists 
    Boolean notExist = false; 

    //Iterator loop 
    Iterator<Log> iterator = logs.iterator(); 


    while (iterator.hasNext()) 
    { 
     //This is where get the exception 
     if (iterator.next().getPhonenumber() == phonenumber && iterator.next().getType() == type && iterator.next().getDate() == date) 
     { 

      updateLog(newLog, iterator.next().getId()); 
     } 
     else 
     { 
      notExist = true; 
     } 

    } 

    if (notExist) 
    { 
     logs.add(newLog); 
    } 

} 

回答

37

要调用next()一堆倍在一次迭代中迫使Iterator移动到一个不存在的元素。

而不是

if (iterator.next().getPhonenumber() == phonenumber && iterator.next().getType() == type && iterator.next().getDate() == date) 
{ 
    updateLog(newLog, iterator.next().getId()); 
    ... 

使用

Log log = iterator.next(); 

if (log.getPhonenumber() == phonenumber && log.getType() == type && log.getDate() == date) 
{ 
    updateLog(newLog, log .getId()); 
    ... 

每次调用Iterator#next()时,它向前移动光标底层。