реализация Итератор вызывает бесконечный цикл

Question

Я пытаюсь сделать цикл по каждому элементу, как следует для LinkedList-эск класса, который я построил:

boolean contains = false; 
for(Thing t : front) { 
    if(t.equals(something)) { 
     t.doSomething(); 
     contains = true; 
     break; 
    } 
} 

Это те методы, итераторы я реализовал:

@Override 
public Iterator<Thing> iterator() { 
    Thing current = this; 
    return new Iterator<Thing>() { 
     public boolean hasNext() { 
      return current.hasNext(); 
     } 

     public Thing next() { 
      return current.next; 
     } 
    }; 
} 

Я пробовал отлаживать это, и кажется, что он не возвращает следующий элемент в связанном списке, тем самым вызывая бесконечный цикл, когда я запускаю его и пытаюсь выполнить цикл foreach.

Извините, если на этот вопрос был дан ответ или есть действительно глупая ошибка, я искал какое-то время и не смог найти ответ. Это мой первый раз, написав итератор, поэтому, пожалуйста, будьте осторожны. :)

Answer 1

Вы должны продвигать его государство Iterator «s в next():

public Thing next() { 
     current = current.next; 
     return current; 
    } 

В противном случае, все вызовы next() будет возвращать один и тот же элемент.

EDIT:

Вы должны переместить локальную переменную декларацию Thing current = this; внутри экземпляра анонимного класса (то есть превратить его в переменный экземпляре).

@Override 
public Iterator<Thing> iterator() { 
    return new Iterator<Thing>() { 

     private Thing current = Thing.this; 

     public boolean hasNext() { 
      return current.hasNext(); 
     } 

     public Thing next() { 
      current = current.next; 
      return current; 
     } 

    }; 
}