SteamWar/SpigotCore
Archiviert
13
0

Add BauweltMemberConfig #102

Manuell gemergt
Lixfel hat 9 Commits von BauweltMemberConfig nach master 2021-05-14 15:45:59 +02:00 zusammengeführt
Besitzer
Keine Beschreibung angegeben.
YoyoNow hat 1 Commit 2021-04-29 11:59:31 +02:00 hinzugefügt
Autor
Besitzer

Ich würde ganz gerne hier von Lixfel schonmal ein kleinen PR bekommen, damit ich weiß was sich vllt noch ändern sollte.

Ich würde ganz gerne hier von Lixfel schonmal ein kleinen PR bekommen, damit ich weiß was sich vllt noch ändern sollte.
YoyoNow hat ein Review von Lixfel 2021-04-29 12:00:24 +02:00 angefragt
Lixfel hat 2021-04-29 20:12:08 +02:00 Änderungen angefragt
@ -0,0 +28,4 @@
public final class BauweltMemberConfig {
private BauweltMemberConfig() {
throw new IllegalStateException("Utility Class");
Besitzer

Werf das Raus, der fucking Konstruktor ist schließlich private

Werf das Raus, der fucking Konstruktor ist schließlich private
Lixfel markierte diese Unterhaltung als gelöst
@ -0,0 +42,4 @@
public static String getPlayerConfig(int id) {
ResultSet config = SQL.select("SELECT * FROM MemberConfig WHERE UserID = ?", id);
try {
if (config == null || !config.next()) {
Besitzer

config sollte nie null sein?

config sollte nie null sein?
Autor
Besitzer

Warum nicht?

Warum nicht?
Lixfel markierte diese Unterhaltung als gelöst
@ -0,0 +47,4 @@
}
return config.getString("BauConfig");
} catch (SQLException e) {
return null;
Besitzer

Nein WTF? Werf ne SecurityException, irgendwas läuft da gewaltig schief!

Nein WTF? Werf ne SecurityException, irgendwas läuft da gewaltig schief!
Lixfel markierte diese Unterhaltung als gelöst
YoyoNow hat 1 Commit 2021-04-30 14:58:04 +02:00 hinzugefügt
YoyoNow hat 1 Commit 2021-04-30 16:33:50 +02:00 hinzugefügt
YoyoNow hat 2 Commits 2021-04-30 17:48:16 +02:00 hinzugefügt
YoyoNow hat 1 Commit 2021-04-30 17:56:13 +02:00 hinzugefügt
YoyoNow hat ein Review von Lixfel 2021-04-30 17:58:05 +02:00 angefragt
YoyoNow hat den Titel von WIP: Add BauweltMemberConfig zu Add BauweltMemberConfig 2021-05-02 21:01:47 +02:00 geändert
YoyoNow hat die Aufforderung zum Review an Lixfel 2021-05-05 21:07:27 +02:00 entfernt
YoyoNow hat ein Review von Lixfel 2021-05-05 21:07:30 +02:00 angefragt
Lixfel hat 2021-05-06 20:59:20 +02:00 Änderungen angefragt
@ -0,0 +51,4 @@
}
}
public static void updatePlayerConfig(Player player, String configType, String config) {
Besitzer

Können wir konsistent zu den Spalten bleiben und das durchgehend config und value nennen?

Können wir konsistent zu den Spalten bleiben und das durchgehend config und value nennen?
Autor
Besitzer

Ja habe ich.

Ja habe ich.
YoyoNow markierte diese Unterhaltung als gelöst
@ -0,0 +67,4 @@
SQL.update("INSERT INTO UserConfig (User, Config, Value) VALUES (?, ?, ?) ON DUPLICATE KEY UPDATE Value = VALUES(Value)", id, configType, config);
}
public static void removePlayerConfig(Player player, String configType) {
Besitzer

Üblicherweise haben wir keine Player-Methoden in den Datenbankmethoden, sondern nur UUID und id-Support.

Üblicherweise haben wir keine Player-Methoden in den Datenbankmethoden, sondern nur UUID und id-Support.
YoyoNow markierte diese Unterhaltung als gelöst
YoyoNow hat 1 Commit 2021-05-07 09:05:33 +02:00 hinzugefügt
YoyoNow hat ein Review von Lixfel 2021-05-07 09:06:16 +02:00 angefragt
Lixfel hat 2021-05-12 20:38:18 +02:00 Änderungen angefragt
@ -0,0 +41,4 @@
}
return configResult.getString("Value");
} catch (SQLException e) {
throw new SecurityException();
Besitzer

Ungeil, da geht der ursprüngliche Fehler verloren => Stacktrace hat keine aussagekraft mehr. Pack noch einen String mit kurzbeschreibung und daran anhängend die Exception dran aka. new SecException("Bla", e);

Ungeil, da geht der ursprüngliche Fehler verloren => Stacktrace hat keine aussagekraft mehr. Pack noch einen String mit kurzbeschreibung und daran anhängend die Exception dran aka. new SecException("Bla", e);
YoyoNow markierte diese Unterhaltung als gelöst
YoyoNow hat 2 Commits 2021-05-12 20:41:08 +02:00 hinzugefügt
YoyoNow hat ein Review von Lixfel 2021-05-12 20:41:23 +02:00 angefragt
Lixfel hat die Änderungen 2021-05-12 20:54:28 +02:00 genehmigt
Besitzer

Datenbanktabelle muss noch erstellt werden. Das mache ich, damit alle Key-Constraints und Indezes passen.

Datenbanktabelle muss noch erstellt werden. Das mache ich, damit alle Key-Constraints und Indezes passen.
Lixfel hat Commit b71bc98a41 in master 2021-05-14 15:45:59 +02:00 manuell gemerged
Lixfel löschte die Branch BauweltMemberConfig 2021-05-14 15:46:16 +02:00
Dieses Repo ist archiviert. Du kannst Pull-Requests nicht kommentieren.
Keine Beschreibung angegeben.