SteamWar/SpigotCore
Archiviert
13
0

Add versionDependantCall, See #144 BauSystem #80

Manuell gemergt
Lixfel hat 11 Commits von VersionDependantCalls nach master 2020-12-25 23:10:54 +01:00 zusammengeführt
Nur Änderungen aus Commit e0f6b2e5bf werden angezeigt - Alle Commits anzeigen

Datei anzeigen

@ -40,7 +40,7 @@ public class VersionedCallable<T> {
try {
Review

Kann man nicht einfach einen reverse-iterator verwenden? Wäre wesentlich sauberer

Kann man nicht einfach einen reverse-iterator verwenden? Wäre wesentlich sauberer
Review

Ich glaube nicht so ganz

Ich glaube nicht so ganz
Review

Außerdem wäre ein Reverse-Iterator wieder ein Objekt mehr. Ich glaube die einfache Schleife reicht da.

Außerdem wäre ein Reverse-Iterator wieder ein Objekt mehr. Ich glaube die einfache Schleife reicht da.
return versionedCallable.callable.call();
} catch (Exception e) {
Review

Kein überprüfen auf i == 0 hier!

Kein überprüfen auf i == 0 hier!
Review

Wenn ich dies nicht mache, können NullPointerExceptions fliegen!

Wenn ich dies nicht mache, können NullPointerExceptions fliegen!
Review

Bitte erstmal weiterlesen! Ja! Wenn hier nichts zutrifft, ist das ein Fehler!

Bitte erstmal weiterlesen! Ja! Wenn hier nichts zutrifft, ist das ein Fehler!
Review

Habe ich ja jetzt schon behoben!

Habe ich ja jetzt schon behoben!
throw new RuntimeException("Could not run version dependant code", e);
throw new RuntimeException("Could not run version dependent code", e);
}
}
}
Review

Wird nie aufgerufen & gefällt mir nicht. Wenn diese Zeile erreicht wird, ist das ein Fehler und sollte auch entsprechend geworfen werden (als Programmierer, also Silent fehler, z.B. SecurityException)

Wird nie aufgerufen & gefällt mir nicht. Wenn diese Zeile erreicht wird, ist das ein Fehler und sollte auch entsprechend geworfen werden (als Programmierer, also Silent fehler, z.B. SecurityException)