Break in dem Zusammenhang "legitim"?

[Super Gogeta]

Hallöchen!

Ich hab grad ne Funktionalität gebastelt, die auch funktioniert, bei der ich aber den teils verpönten break-Befehl genutzt habe. Mir wurde auf Arbeit gesagt, dass man den eigentlich nicht benutzt, nur weiß ich mir in dem Fall keine andere Lösung. Hier mal der Code:

Set<String> keys = kartenLog.keySet();
byte[] werte = new byte[4];

for (int i = 1; i < 6; i++) {
    	for (String key : keys) {
    		if (kartenLog.get(key)) {
    			werte = werteLog.get(key);
    			gegnerWerte[i] = werte;
    			kartenLog.put(key, false);
    			break;
    		}
	 }
}

Wie gesagt funktioniert alles. Ist das so in Ordnung oder gibt´s auch nen besseren Weg ohne break (obwohl ich nicht verstehe, warum der Befehl so schlecht sein soll)?

 

[Gast2]

Das schaut schon sehr frikkelig aus, was genau versuchst du da zu machen?

 

[Super Gogeta]

Hab noch den boolean rausgenommen. Der war nicht mehr nötig.

Ich lese da eine Map aus und prüfe, welche Keys true sind (also eine Map <String, Boolean>).
Ist dem so, wird aus einer anderen Map ein Array entsprechend dem Key mit 4 Zahlen geholt und dann in die Matrix gegnerWerte in der aktuellen Zeile (i) gelegt. Es funktioniert auch alles, mir geht´s nur um die Nutzung von break.

 

[Gast2]

So wie dein Code jetzt ausschaut brauchst du das

break

doch gar nicht oder? Aber ich finde nicht dass nen

break

so böse ist. Wenn man weiß dass man die Schleife nicht weiter durchlaufen muss dann kann man die ruhig abbrechen. Was allerdings wirklich hässlich ist sind break to labels. Das macht den Code sehr undurchsichtig.

 

[Super Gogeta]

Brauchen tu ich das, weil sonst der zweite true-Wert dafür sorgt, dass die aktuelle Zeile mit den Werten überschrieben wird. Das kann dann darin resultieren, dass nur Zeile 1 belegt ist und die anderen nicht, weil die keys ja auf false gesetzt sind und bei den anderen Zeilen nicht mehr berücksichtigt werden.

Aber ok, dass das so passt, ich finde break ziemlich nützlich.