Wie schön oder unschön ist dieser code - alternative ?

[Harryv]

Ist dieser (funktionierede) Code in Bezug auf die catch Anweisung in Ordnung oder sollte man das besser machen <grübel>

int i = 0;

for (MyObject myObject : getMyObjects()) {
 
      try {
             //getElements() ist eine ArrayList
             myObject.setElement(getElements().get(i++));

      }catch (IndexOutOfBoundsException ex) {
              break;
      }
}

 

[SlaterB]

setze das try/catch außen, dann wird die Schleife automatisch beendet ohne break,

Exception abfangen ohne Ausgabe ist generell sehr bedenklich,

IndexOutOfBoundsException abfangen ist generell sehr bedenklich bis skandalös,
prüfe lieber vorher
if (getElements().size() <= i) {
}else {
}
usw.


verwende lieber gleich eine for-Schleife, als die hier so versteckt nachzubauen

 

[maki]

Autsch... das ist wirklich schlecht, sehr sogar.
Es gibt Bücher die so etwas als negativ Beispiel Anführen (Effective Java).

Benutze doch die size() Methode der List.

 

[Harryv]

ok danke. Habe das durch die Methode size() ersetzt. Dann kann man sogar auf die Exception (fast) verzichten.

 

[SlaterB]

streiche '(fast)'

Spielerei: nicht 100x size() aufrufen in einer 100er-Schleife,
speichere die Zahl vorher in einer int-Variablen size

 

[Harryv]

cool. noch besser. danke