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

TntMode #163

Manuell gemergt
YoyoNow hat 8 Commits von TntMode nach master 2021-01-24 20:19:52 +01:00 zusammengeführt
Diskussion 2 Commits 8 Geänderte Dateien 7 +22 -15
2 geänderte Dateien mit 22 neuen und 15 gelöschten Zeilen
Statistiken anzeigen Patch-Datei herunterladen Vergleichs-Datei herunterladen Alle Dateien ausklappen Alle Dateien einklappen
Nur Änderungen aus Commit f887e633ad werden angezeigt - Alle Commits anzeigen

2
BauSystem_Main/src/de/steamwar/bausystem/commands/CommandInfo.java
Datei anzeigen

@ -34,7 +34,7 @@ public class CommandInfo implements CommandExecutor {
@Override @Override
public boolean onCommand(CommandSender sender, Command command, String label, String[] args) { public boolean onCommand(CommandSender sender, Command command, String label, String[] args) {
sender.sendMessage(BauSystem.PREFIX + "Besitzer: §e" + SteamwarUser.get(BauSystem.getOwnerID()).getUserName()); sender.sendMessage(BauSystem.PREFIX + "Besitzer: §e" + SteamwarUser.get(BauSystem.getOwnerID()).getUserName());
sender.sendMessage(BauSystem.PREFIX + "TNT-Schaden: " + (CommandTNT.getTntMode() == CommandTNT.TNTMode.OFF ? "§cAN" : "§aAUS")); sender.sendMessage(BauSystem.PREFIX + "TNT-Schaden: " + CommandTNT.getTntMode().getName());
sender.sendMessage(BauSystem.PREFIX + "Feuerschaden: " + (CommandFire.getInstance().isOn() ? "§aAUS" : "§cAN")); sender.sendMessage(BauSystem.PREFIX + "Feuerschaden: " + (CommandFire.getInstance().isOn() ? "§aAUS" : "§cAN"));
sender.sendMessage(BauSystem.PREFIX + "Eingefroren: " + (CommandFreeze.getInstance().isOn() ? "§aJA" : "§cNEIN")); sender.sendMessage(BauSystem.PREFIX + "Eingefroren: " + (CommandFreeze.getInstance().isOn() ? "§aJA" : "§cNEIN"));

35
BauSystem_Main/src/de/steamwar/bausystem/commands/CommandTNT.java
Datei anzeigen

@ -93,20 +93,27 @@ public class CommandTNT implements CommandExecutor, Listener {
return false; return false;
} }
if (args.length != 0 && (args[0].equalsIgnoreCase("an") || args[0].equalsIgnoreCase("on"))) { if (args.length != 0) {
tntMode = TNTMode.ON; switch (args[0].toLowerCase()) {
sendToActionBar(getEnableMessage()); case "an":
return false; case "on":
} tntMode = TNTMode.ON;
if (args.length != 0 && (args[0].equalsIgnoreCase("aus") || args[0].equalsIgnoreCase("off"))) { sendToActionBar(getEnableMessage());
tntMode = TNTMode.OFF; return false;
sendToActionBar(getDisableMessage()); case "aus":
return false; case "off":
} tntMode = TNTMode.OFF;
if (args.length != 0 && Region.buildAreaEnabled() && (args[0].equalsIgnoreCase("testblock") || args[0].equalsIgnoreCase("tb"))) { sendToActionBar(getDisableMessage());
tntMode = TNTMode.ONLY_TB; return false;
sendToActionBar(getTestblockEnableMessage()); case "testblock":
return false; case "tb":
if (!Region.buildAreaEnabled()) break;
tntMode = TNTMode.ONLY_TB;
sendToActionBar(getTestblockEnableMessage());
Lixfel hat 2021-01-22 07:20:29 +01:00 kommentiert
Review
Link kopieren

Du registrierst den Command nirgends als Listener (weder implements noch registerEvents().

Du registrierst den Command nirgends als Listener (weder implements noch registerEvents().
return false;
default:
break;
}
Lixfel hat 2021-01-22 07:20:56 +01:00 kommentiert
Review
Link kopieren

Ich denke, ein switch-case würde sich besser lesen lassen als dieses If if else gedöns.

Ich denke, ein switch-case würde sich besser lesen lassen als dieses If if else gedöns.
} }
switch (tntMode) { switch (tntMode) {