Der Umgang mit Iterator - Wie ein Objekt aus einer ArrayList

[MitchGilliam]

Hallo Leute,

ich höre dieses Semester eine Programmiervorlesung zum Thema Java. Bisher hatte ich noch keinen Kontakt zu der Materie. Nachdem ich mich nun knapp 2 Stunden an einem kleinen Problem aufhalte, möchte ich dies mal hier posten, in der Hoffnung, das jemand bereit ist, mir zu helfen:

Zuerst grob um was es geht:

Ich habe eine ArrayList, gefüllt mit Objekten der Klasse "Passenger"

ArrayList<Passenger> passengers;

An die Klasse in der dies geschieht, wird nun ein neues Objekt vom Typ "Passenger" übergeben und folgendermaßen in die ArrayList passengers eingefügt:

if (destination !=  this.floorNumber){passengers.add(passenger);}

Nun habe ich eine mit Objekten gefüllte ArrayList - wie aber bekomme ich einzelne Objekte zurück, um mit diesen wiederrum etwas zu machen (z.B. den Namen abholen o.ä.)

Meine Lösung dazu sah folgendermaßen aus:

Iterator it = passengers.iterator();
while (it.hasNext()) {
Passenger help = (Passenger) it.next();
System.out.println(help.getID());
}

getID() ist eine Methode der Klasse Passenger.

Mir ist durchaus bewusst, dass ich auch die Länge von passengers abholen könnte und dann mit einer For-Schleife arbeiten.
Aber das Iterator-Objekt gibt es ja nicht umsonst und ich wüsste gerne, wie ich damit umgehe.

Vielen dank für eure Hilfen

MG

 

[Guest]

So ungefähr, es hängt aber davon ab, was du vor hast.

// Verwende für Variablen, Parameter, Rückgabetypen etc. immer die Interfaces, 
// nie die Implementierung (hier Interface List, Implementierung ArrayList)
List<Passenger> passengers = new ArrayList<Passenger>();
...

// Wenn eine generische Collection im Spiel ist, dann auch entsprechend den Iterator
// generisch schreiben. Das erspart dir die unnötigen Casts.
Iterator<Passenger> iterator = passengers.iterator();
while(iterator.hasNext())
{
   Passenger passenger = iterator.next(); // kein Cast, da Iterator generisch
   ...
   if(passenger.siehtHarmlosAus()) // ;-)
   {
      iterator.remove(); // Löscht den aktuellen Eintrag aus der Liste (über Iterator OK)
   }
}
// "iterator" ist an dieser Stelle immer noch erreichbar. 
// iterator.next(); das würde hier zu einer Exception führen

oder

// Scope von Iterator auf die Schleife beschränkt, also ausserhalb der Schleife nicht erreichbar
// Sieht gewöhnungsbedürftig aus, aber keine schlechte Idee den Scope immer zu beschränken.
for(Iterator<Passenger> iterator; iterator.next(); )
{
   Passenger passenger = iterator.next();
   ... // auf keinen Fall den Inhalt der Liste hier DIREKT, also über 'passengers' manipulieren!
}

oder gleich foreach

// Wenn du die Schleife einfach nur lesend durchgehen möchtest, ohne sie zu ändern,
// dann ist dies die einfachste Variante
for(Passenger passenger : passenger)
{
   ... // auf keinen Fall den Inhalt der Liste hier manipulieren
}

 

[Guest]

Ups! So wäre es korrekt

for(Passenger passenger : passengers)