Fix: Fehlermeldung wenn keine Buchungen vorhanden

[dippeal]

Gibt eine Fehlermeldung aus, wenn noch keine Buchung vorhanden ist.

[willuhn]

Ich würde es deutlich lesbarer finden, wenn du oben lediglich folgendes eingefügt hättest:

if (list == null || !list.hasNext())
{
  throw new ApplicationException("Abbruch! Es existiert noch keine Buchung.");
}

Dann müsste nicht der gesamte folgende Block in eine if-Bedingung verschoben werden. Durch die Umformatierungen kann ich nicht wirklich erkennen, ob und was sich in all den anderen Zeilen geändert hat.

[dippeal]

Ich würde es deutlich lesbarer finden, wenn du oben lediglich folgendes eingefügt hättest:

if (list == null || !list.hasNext())
{
  throw new ApplicationException("Abbruch! Es existiert noch keine Buchung.");
}

Dann müsste nicht der gesamte folgende Block in eine if-Bedingung verschoben werden. Durch die Umformatierungen kann ich nicht wirklich erkennen, ob und was sich in all den anderen Zeilen geändert hat.

Ich habe den Code umgebaut wie du vorgeschlagen hast. Durch die Einrückung des else Block kommt es im diff zu den vielen Zeilen Änderung.
Primäre Änderungen Z. 29-30 und Z. 135-141.

[willuhn]

Das "else" kannst du weglassen. Wenn die if-Bedingung zum Tragen kommt, wird eine Exception geworfen und die Methode wird verlassen.

[willuhn]

Sieht gut aus