SteamWar/BungeeCore
Archiviert
13
2

Add BauLock #415

Zusammengeführt
Lixfel hat 20 Commits von BauLock nach master 2022-09-14 11:51:49 +02:00 zusammengeführt
5 geänderte Dateien mit 25 neuen und 26 gelöschten Zeilen
Nur Änderungen aus Commit ef6435b176 werden angezeigt - Alle Commits anzeigen

Datei anzeigen

@ -81,11 +81,11 @@ public class BauCommand extends BasicCommand {
break;
case "lock":
setLocked(SteamwarUser.get(p), args[1]);
setLocked(SteamwarUser.get(p), args);
xSpw markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

siehe unten

siehe unten
Veraltet
Review

Hier fliegt eine ArrayIndexOutOfBoundsException, wenn der User einfach nur /bau lock eingibt. Zudem solltest du auf das Argument noch .toUpperCase() ausführen, ansonsten muss der User den Lockzustand immer in Capslock schreiben.

Hier fliegt eine ArrayIndexOutOfBoundsException, wenn der User einfach nur /bau lock eingibt. Zudem solltest du auf das Argument noch .toUpperCase() ausführen, ansonsten muss der User den Lockzustand immer in Capslock schreiben.
break;
case "unlock":
setLocked(SteamwarUser.get(p), "OPEN");
setLocked(SteamwarUser.get(p), new String[]{"", "open"});
xSpw markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

Warum nimmt setLocked nicht genau einen String?

Warum nimmt setLocked nicht genau einen String?
break;
default:
@ -327,22 +327,28 @@ public class BauCommand extends BasicCommand {
}
private static boolean isLocked(SteamwarUser owner, SteamwarUser target) {
BauLockState activeLockState = UserConfig.getUserLockState(owner.getId());
BauLockState activeLockState = getUserLockState(owner.getId());
if(activeLockState == null)
activeLockState = BauLockState.OPEN;
xSpw markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

Hier wird nie auf die Länge des Array geprüft! Wird Fehler werfen.

Hier wird nie auf die Länge des Array geprüft! Wird Fehler werfen.
return activeLockState == BauLockState.NONE || (activeLockState == BauLockState.TEAM && !(Team.get(owner.getId()) == Team.get(target.getId())));
return activeLockState == BauLockState.NOBODY || (activeLockState == BauLockState.TEAM && owner.getTeam() == target.getTeam());
xSpw markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

Hier bekommst du nicht so ganz die Teams zurück ;). Du versuchst hier UserIDs als TeamIDs zu verwenden. Das funktioniert nicht so ganz... Du kannst auch einfach die TeamIDs vergleichen, das wäre dann (SteamWarUser u).getTeam() oder so ähnlich (Kein Team.get nötig, der Vergleich der Teamobjekte mit == würde zudem immer fehlschlagen, weil dabei bei Objekten nur die Objektadresse verglichen wird und in dem Fall 2x zwar das gleiche Team, aber unterschiedliche Objekte zurückgegeben werden würde).

Hier bekommst du nicht so ganz die Teams zurück ;). Du versuchst hier UserIDs als TeamIDs zu verwenden. Das funktioniert nicht so ganz... Du kannst auch einfach die TeamIDs vergleichen, das wäre dann (SteamWarUser u).getTeam() oder so ähnlich (Kein Team.get nötig, der Vergleich der Teamobjekte mit == würde zudem immer fehlschlagen, weil dabei bei Objekten nur die Objektadresse verglichen wird und in dem Fall 2x zwar das gleiche Team, aber unterschiedliche Objekte zurückgegeben werden würde).
}
private static void setLocked(SteamwarUser owner, String arg) {
if(arg == null || BauLockState.valueOf(arg) == null) {
private static final String bauLockConfigName = "baulockstate";
xSpw markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

Werte haben ist eine Sache diese zu nutzen ist eine andere.

Werte haben ist eine Sache diese zu nutzen ist eine andere.
private static void setLocked(SteamwarUser owner, String[] args) {
if(args.length < 2 || BauLockState.valueOf(args[1]) == null) {
Message.send("BAU_LOCKED_OPTIONS", owner.getPlayer());
return;
}
xSpw markierte diese Unterhaltung als gelöst
Review

Keine Individuellen Nachrichten für die einzelnen States?

Keine Individuellen Nachrichten für die einzelnen States?
UserConfig.updateUserConfig(owner.getId(), UserConfig.lockConfigName, arg.toUpperCase());
xSpw markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

Kann man mit einem Ternary operator inlinen

Kann man mit einem Ternary operator inlinen
Message.send("BAU_LOCKED_" + arg, owner.getPlayer());
String lockState = args[1].toUpperCase();
UserConfig.updateUserConfig(owner.getId(), bauLockConfigName, lockState);
Message.send("BAU_LOCKED_" + lockState, owner.getPlayer());
}
private static BauLockState getUserLockState(int userID) {
return BauLockState.valueOf(UserConfig.getConfig(userID, bauLockConfigName));
}
}

Datei anzeigen

@ -2,7 +2,7 @@ package de.steamwar.bungeecore.sql;
public enum BauLockState {
NONE, // Locks the build server for all users
NOBODY, // Locks the build server for all users
xSpw markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

All klingt wie alle können drauf vllt eher zu (none) umbenenen

All klingt wie alle können drauf vllt eher zu (none) umbenenen
Veraltet
Review

Ich fände NOBODY oder NOMEMBERS treffender.

Ich fände NOBODY oder NOMEMBERS treffender.
TEAM, //opens the build server only for every added user which is in the same team as the buildOwner
xSpw markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

Kommentar passt nicht zum namen!

Kommentar passt nicht zum namen!
OPEN //unlocks the build server for all users
xSpw markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

Reihenfolge ist etwas komisch

Reihenfolge ist etwas komisch
}

Datei anzeigen

@ -3,18 +3,11 @@ package de.steamwar.bungeecore.sql;
import java.util.UUID;
public class UserConfig {
public static final String lockConfigName = "baulockstate";
private static final Statement insert = new Statement("INSERT INTO `UserConfig`(`User`, `Config`, `Value`) VALUES (?,?,?)");
private static final Statement update = new Statement("UPDATE `UserConfig` SET `Value`= ? WHERE 'User' = ? AND 'Config' = ?");
xSpw markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

Bitte diesen String hier rausbewegen (z.B. zum Baucommand). Die SQL-Klasse soll generisch bleiben, insbesondere mit dem kommenden CommonDB im Hintergrund.

Bitte diesen String hier rausbewegen (z.B. zum Baucommand). Die SQL-Klasse soll generisch bleiben, insbesondere mit dem kommenden CommonDB im Hintergrund.
private static final Statement select = new Statement("SELECT `Value` FROM `UserConfig` WHERE 'User' = ? AND 'Config' = ?");
private static final Statement delete = new Statement("DELETE FROM `UserConfig` WHERE 'User' = ? AND 'Config' = ?");
public static BauLockState getUserLockState(int userID) {
return BauLockState.valueOf(getConfig(userID, lockConfigName));
}
public static String getConfig(int userID, String config) {
return select.select(rs -> {
if(rs.next())

Datei anzeigen

@ -200,9 +200,9 @@ BAU_ADDMEMBER_ADDED=§aThe player was added to your world.
BAU_ADDMEMBER_ADDED_TARGET=§aYou have been added to the world of §e{0}§a.
xSpw markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

Keine & in Nachrichten. Nutze § bitte.

Keine & in Nachrichten. Nutze § bitte.
Veraltet
Review

Ich würde persönlich das 'have' rausnehmen.

Ich würde persönlich das 'have' rausnehmen.
BAU_TP_USAGE=§8/§7build tp §8[§eplayer§8]
xSpw markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

Keine & in Nachrichten. Nutze § bitte.

Keine & in Nachrichten. Nutze § bitte.
Veraltet
Review

Ich würde persönlich das 'have' rausnehmen.

Ich würde persönlich das 'have' rausnehmen.
BAU_TP_NOALLOWED=§cYou are not allowed to teleport to this player's world.
BAU_LOCKED_NOALLOWED=&cThe buildserver is currently locked.
BAU_LOCKED_OPTIONS=§eBuildserver lock options: §cnone§8, §eteam§8, §aopen§8.
BAU_LOCKED_NONE=§eYou have closed your buildserver to all players.
BAU_LOCKED_NOALLOWED=&cThe build server is currently locked.
xSpw markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

Im englischen musst du build und server trennen (zwei Wörter, betrifft auch die folgenden Zeilen)

Im englischen musst du build und server trennen (zwei Wörter, betrifft auch die folgenden Zeilen)
BAU_LOCKED_OPTIONS=§eBuild server lock options: §cnone§8, §eteam§8, §aopen§8.
BAU_LOCKED_NOBODY=§eYou have closed your buildserver to all players.
BAU_LOCKED_TEAM=§eYou've closed your buildserver to all beside added team members.
BAU_LOCKED_OPEN=§eYou have reopened your build for all added players.
BAU_DELMEMBER_USAGE=§8/§7build delmember §8[§eplayer§8]

Datei anzeigen

@ -38,8 +38,8 @@ HELP_JOIN=§7Trete mit §8/§ejoin §8[§eSpieler§8] §7einem Kampf bei!
HELP_JOIN_HOVER=§eSpieler beitreten
HELP_LOCAL=§7Schreibe mit §8/§elocal §7nur auf dem lokalen Server!
HELP_LOCAL_HOVER=§eLokaler Chat
HELP_BAU_LOCK=§8/§ebuild lock §8- §7Schließe den Bauserver für bestimmte Spieler.
HELP_BAU_LOCK_HOVER=§eSchließe deinen bau
HELP_BAU_LOCK=§8/§ebuild lock §8- §7Sperre dein Bauserver für bestimmte Spieler.
HELP_BAU_LOCK_HOVER=§eSperre deinen bau
xSpw markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

Schließen klingt irgendwie leicht falsch (für mich). Wie wäre es mit sperren?

Schließen klingt irgendwie leicht falsch (für mich). Wie wäre es mit sperren?
HELP_BAU_UNLOCK=§8/§ebuild unlock §8- §7SÖffne den Bauserver für alle vertrauten Spieler.
HELP_BAU_UNLOCK_HOVER=§eÖffne deinen bau
@ -184,10 +184,10 @@ BAU_ADDMEMBER_ADDED=§aDer Spieler wurde zu deiner Welt hinzugefügt.
BAU_ADDMEMBER_ADDED_TARGET=§aDu wurdest zu der Welt von §e{0} §ahinzugefügt.
xSpw markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

Keine & in Nachrichten. Nutze § bitte.

Keine & in Nachrichten. Nutze § bitte.
BAU_TP_USAGE=§8/§7bau tp §8[§eSpieler§8]
xSpw markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

Keine & in Nachrichten. Nutze § bitte.

Keine & in Nachrichten. Nutze § bitte.
BAU_TP_NOALLOWED=§cDu darfst dich nicht auf diese Welt teleportieren.
BAU_LOCKED_NOALLOWED=&cDer Buildserver ist momentan gelocked.
BAU_LOCKED_OPTIONS=§eBauserver schließ optionen: §cnone§8, §eteam§8, §aopen§8.
BAU_LOCKED_NONE=§eDu hast dein Bau für alle Spieler geschlossen.
BAU_LOCKED_TEAM=§eDu hast dein Bau für alle außer hinzugefügte Teammitglieder geschlossen.
BAU_LOCKED_NOALLOWED=&cDer Bauserver ist momentan gesperrt.
xSpw markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

Auf Deutsch ist es der Bauserver

Auf Deutsch ist es der Bauserver
BAU_LOCKED_OPTIONS=§eBauserver sperr optionen: §cnone§8, §eteam§8, §aopen§8.
xSpw markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

Im Deutschen wäre Bauserverschließoptionen ein Wort.

Im Deutschen wäre Bauserverschließoptionen ein Wort.
BAU_LOCKED_NOBODY=§eDu hast dein Bau für alle Spieler geschlossen.
BAU_LOCKED_TEAM=§eDu hast dein Bau für alle außer hinzugefügte Teammitglieder gesperrt.
BAU_LOCKED_OPEN=§eDu hast dein Bau für alle hinzugefügten Spieler wieder geöffnet.
xSpw markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

Das wieder ist ein bisschen doppelt gemoppelt.

Das wieder ist ein bisschen doppelt gemoppelt.
BAU_DELMEMBER_USAGE=§8/§7bau delmember §8[§eSpieler§8]
BAU_DELMEMBER_SELFDEL=§cDu kannst dich nicht selbst entfernen!