SteamWar/SpigotCore
SteamWar/SpigotCore
Archiviert
13
0
Fork 0
Code Issues 4 Pull-Requests 1 Releases Wiki Aktivität

MultiKits #88

Manuell gemergt
Lixfel hat 10 Commits von multi-kit nach master 2021-01-30 16:42:25 +01:00 zusammengeführt
Diskussion 0 Commits 10 Geänderte Dateien 1 +7 -4
1 geänderte Dateien mit 7 neuen und 4 gelöschten Zeilen
Statistiken anzeigen Patch-Datei herunterladen Vergleichs-Datei herunterladen Alle Dateien ausklappen Alle Dateien einklappen
Nur Änderungen aus Commit 5266bb6492 werden angezeigt - Alle Commits anzeigen

11
SpigotCore_Main/src/de/steamwar/sql/PersonalKit.java
Datei anzeigen

@ -61,12 +61,13 @@ public class PersonalKit {
}
}
public static PersonalKit get(int userID, String gamemode, String name) {
public static List<PersonalKit> get(int userID, String gamemode, String name) {
Lixfel hat 2021-01-09 17:49:53 +01:00 kommentiert
Veraltet
Review
Link kopieren

Das sollte nur ein Kit zurückliefern oder null.

Das sollte nur ein Kit zurückliefern oder null.
Chaoscaot hat 2021-01-09 19:19:36 +01:00 kommentiert
Veraltet
Review
Link kopieren

Wie gesagt, Kits können den gleichen Namen haben

Wie gesagt, Kits können den gleichen Namen haben
Lixfel hat 2021-01-10 13:10:09 +01:00 kommentiert
Veraltet
Review
Link kopieren

Mehrere Kits mit gleichem Spieler, GameMode und Namen finde ich nicht sinnvoll, das erlaubt nur Verwirrung.

Mehrere Kits mit gleichem Spieler, GameMode und Namen finde ich nicht sinnvoll, das erlaubt nur Verwirrung.
ResultSet rs = SQL.select("SELECT * FROM PersonalKit WHERE UserID = ? AND GameMode = ? AND Name = ?", userID, gamemode, name);
try {
if(!rs.next())
return null;
return new PersonalKit(rs);
List<PersonalKit> list = new ArrayList<>();
while (rs.next())
list.add(new PersonalKit(rs));
return list;
} catch (SQLException e) {
throw new SecurityException("Failed loading personal kit", e);
}
@ -93,7 +94,9 @@ public class PersonalKit {
list.forEach(kit -> {
if(list.indexOf(kit) >= 1) kit.setInUse(false);
});
list.removeIf(kit -> !kit.isInUse());
}
Lixfel hat 2021-01-24 14:45:44 +01:00 kommentiert
Review
Link kopieren

Ich weiß jetzt nicht ganz den Sinn dieser "NameInUse" Funktion, um Abzufragen, ob dieser Name derzeit verwendet wird, ist das denkbar ungeeignet.

Sinnvoller wäre denke ich ein getInUse(), um das derzeitige Kit in Benutzung zu bekommen. (Es sei denn, du hast da einen sehr speziellen Anwendungsfall)

Ich weiß jetzt nicht ganz den Sinn dieser "NameInUse" Funktion, um Abzufragen, ob dieser Name derzeit verwendet wird, ist das denkbar ungeeignet. Sinnvoller wäre denke ich ein getInUse(), um das derzeitige Kit in Benutzung zu bekommen. (Es sei denn, du hast da einen sehr speziellen Anwendungsfall)
if(list.isEmpty()) return null;
return list.get(0);
} catch (SQLException e) {
throw new SecurityException("Failed loading personal kit", e);