Iterator schaltet nicht weiter?!

[DonMarcoHH]

Moin,

ich hab jetzt schon mehrfach das gleiche Problem und verstehe nicht warum, bisher konnte ich es aber umgehen.
Der folgende Javacode, sollte ja eigentlich jeweils den nächsten Eintrag einer Liste liefern;

List<String> liste = new LinkedList<String>();
String text = "";
while (liste.hasNext()){
    text = liste.next();
}

Wieso funktioniert das nicht? Konkret habe ich das im folgenden Codebeispiel, wo er mir immer und immer wieder den ersten Listeneintrag liefert:

    private Set<String> findType(String line,
            Set<String> initSet, Set<String> keySet){
        String key;
        while (initSet.iterator().hasNext()){
            if (line.matches(key = initSet.iterator().next())){
                keySet.add(key);
            } //if-else
        } //while    
        return keySet;
    } //findType

 

[SlaterB]

Denksport-Hinweis:
angenommen er würde auf diese Weise weiterschalten,
auf welchem Wege würdest du dann ein Set zweimal hintereinander durchlaufen?

 

[Mofi]

Das ist jetzt nur geraten, aber ich denke, dass liegt daran, dass er dir immer und immer wieder einen neuen Iterator gibt und der fängt logischerweise wieder beim ersten Eintrag an.

Holt dir am Anfang einfach ein Iterator Objekt

Iterator it = initSet.iterator();

(Ob der Code 100% stimmt weiß ich nicht, ich arbeite nicht so oft mit Iteratoren. Wahlweise nochmal nachschlagen wie man sich den genau holt)
Und dann arbeitest du einfach mit dem Iterator-Objekt "it" weiter
Also

while(it.hasNext()){
//was auch immer getan werden soll
}

 

[ARadauer]

1. liste.hasNext())... das hat kein hasNext... du meinst wahrscheinlich liste.iterator().hasNext().
warum geht das nicht? weil iterator immer einen neuen iterator erzeugt, der am anfang steht...

so gehts

      List<String> liste = new LinkedList<String>();
      liste.add("a");
      liste.add("b");
      String text = "";
      Iterator<String> it = liste.iterator();
      while (it.hasNext()){
          text = it.next();
          System.out.println(text);
      }

 

[DonMarcoHH]

Danke...
irgendwie hatte ich meins wohl von irgendeiner anderen Programmiersprache...

 

[Antoras]

Noch als Ergänzung:

Anstatt einer while-Schleife sollte man in Bezug auf einen Iterator eine for-Schleife benutzen:

List<String> l = ...
for (Iterator<String> i = l.iterator(); i.hasNext();) {
  String s = i.next();
}

Das hat den Vorteil, dass wenn du mehrmals über eine Liste iterierst du nicht aus Versehen wieder den gleichen Iterator benutzt:

Iterator i = ...
while (i.hasNext())

Iterator i2 = ...
while (i.hasNext) // aua

Statt den Iterator manuell zu erzeugen kannst du aber auch die foreach-Schleife benutzen:

List<String> l = ...
for (String s : l) {
  // use s
}

Dann kannst du einen Fehler wie jedes mal einen neuen Iterator zu erzeugen gar nicht mehr machen.