Endlosschleife beim Ausgeben einer LinkedList

[software]

Hi,
wenn ich diesen Code ausführe, dann habe ich eine Endlosschleife.
Es wird unendlich mal "Berlin" ausgegeben.
Woran liegt das?

import java.util.Iterator;
import java.util.LinkedList;

public class MyLinkedList {

    public static void main(String[] args) {
        LinkedList<String> myLinkedList= new LinkedList<>();

        myLinkedList.add("Berlin");
        myLinkedList.add("Hamburg");
        myLinkedList.add("Frankfurt");
        myLinkedList.add("München");
        myLinkedList.add("Stuttgart");
        myLinkedList.add(1,"Bremen");

        while(myLinkedList.iterator().hasNext()){
            System.out.println(myLinkedList.iterator().next());
        }
    }
}

 

[fhoffmann]

Du holst dir ständig einen neuen Iterator von der Liste. Der steht dann immer auf dem ersten Element.
Stattdessen solltest du dir vor der while-Schleife nur einmal einen Iterator holen.

 

[kneitzel]

Evtl. nur noch zur Ergänzung das generelle Pattern:

Iterator<MyType> myIterator = ....;
while (myIterator.hasNext()) {
  MyType currentValue (myIterator.next());
  // Do something with currentValue.
}

Das zeigt genau das, was @fhoffmann kurz und prägnant gesagt hat. Nur eben direkt als (Pseudo)code.

Bei Dir wäre also denkbar (so iterator() ein Iterator<String> zurück gibt):

Iterator<String> listIterator = myLinkedList.iterator();
while (listIterator.hasNext()) {
  System.out.println(listIterator.next());
}