guter Stil?

[pktm]

Hallo!

Ist sowas in Ordnung?
Wenn nicht, wie sollte es besser aussehen?

for(Iterator i = result2.iterator(); i.hasNext(); result1.add(i.next()));

Grüße, pktm

 

[Gast]

for each schleife benutzen, oder ne while

 

[Gast]

Ja, das sind die beiden anderen Möglichkeiten, die so auf der Hand liegen. Aber es fehlt noch die Begründung, warum man diese Möglichkeiten preferieren sollte.

 

[tfa]

Wie wär's mit collection1.addAll(collection2)?

 

[byte]

Unguter Stil diese for-Schleife.

 

[maki]

Ich finde sie nicht so schlecht, allerdings sollte sie dann schon auf 3 Zeilen gebrochen werden.

Eine while Schleife halte ich meist für deplaziert wenn es um's iterieren geht.

 

[Marco13]

Das
result1.add(i.next())
im Schleifenkopf ist IMHO ganz häßlich. Wenn man es in den Schleifenkörper zieht, ist aber die letzte Anweisung im Schleifenkopf leer, was auch häßlich ist.... Die Alternativen wurden schon genannt...

 

[quippy]

Man sollte meiner Meinung nach erst mal die add-Anweisung in den Schleifenrumpf verfrachten, da man an der Stelle eigentlich nur eine Iterator-Erhöhung (also nur i.next()) erwarten würde.

Da das i.next() jedoch nicht alleine stehen kann (liefert ja das Objekt) sollte man es als while ausformulieren.
Ist IMHO leichter zu lesen.