SteamWar/BauSystem
Archiviert
13
0

TntMode #163

Manuell gemergt
YoyoNow hat 8 Commits von TntMode nach master 2021-01-24 20:19:52 +01:00 zusammengeführt
Besitzer

Closes: #126

Wichtig, man müsste dann noch die sections.yml anpassen aber an sich geht es auch erstmal ohne. Also wir können das ganze auch später einstellen.

Closes: #126 Wichtig, man müsste dann noch die sections.yml anpassen aber an sich geht es auch erstmal ohne. Also wir können das ganze auch später einstellen.
Lixfel hat 2021-01-20 18:02:50 +01:00 Änderungen angefragt
@ -81,6 +81,7 @@ public class BauSystem extends JavaPlugin implements Listener {
getCommand("nightvision").setExecutor(new CommandNV());
getCommand("reset").setExecutor(new CommandReset());
getCommand("speed").setExecutor(new CommandSpeed());
getCommand("buildmode").setExecutor(new CommandBuildMode());
Besitzer

Finde das als Extracommand nicht so gut, da die Funktionalität mit dem TNT-Command kollidiert. Würde den Standard-TNT-Modus zum Buildmode setzen, und standardmäßig zwischen Aus und "Buildmode" (Bessere Wortwahl fehlt mir noch) zu toggeln, nur wenn "an" extra als Argument übergeben wird, in den alten An modus zu gehen.

Finde das als Extracommand nicht so gut, da die Funktionalität mit dem TNT-Command kollidiert. Würde den Standard-TNT-Modus zum Buildmode setzen, und standardmäßig zwischen Aus und "Buildmode" (Bessere Wortwahl fehlt mir noch) zu toggeln, nur wenn "an" extra als Argument übergeben wird, in den alten An modus zu gehen.
@ -0,0 +68,4 @@
if (!blocksDestroyed) {
return;
}
Bukkit.getOnlinePlayers().forEach(player -> player.sendMessage(BauSystem.PREFIX + "§cEs ist etwas explodiert und hätte blöcke zerstört."));
Besitzer

Dafür gibts auch ne Broadcast Funktion. Düfte allerdings spammen, ich empfehle stark, das zu einer ACTION_BAR message zu machen.

Dafür gibts auch ne Broadcast Funktion. Düfte allerdings spammen, ich empfehle stark, das zu einer ACTION_BAR message zu machen.
Autor
Besitzer

Ich würde genau dies nicht machen, da ich nicht möchte, das man nach 5 Sekunden oder so nicht mehr weiß ob es explodiert ist oder so.

Ich würde genau dies nicht machen, da ich nicht möchte, das man nach 5 Sekunden oder so nicht mehr weiß ob es explodiert ist oder so.
Besitzer

Derzeit weißt du es bei "aus" nie. UND: Wenn du dann 100 Messages bekommst, weil .... Auch 10 sind schon nervig.

Derzeit weißt du es bei "aus" nie. UND: Wenn du dann 100 Messages bekommst, weil .... Auch 10 sind schon nervig.
@ -104,4 +105,14 @@ public class BauScoreboard implements Listener {
return "§8/§7" + CommandTPSLimiter.getCurrentTPSLimit();
}
private String tntString() {
Besitzer

Würde das ganze im TNTCommand eher über ein Switch/case mit Enum lösen.

Würde das ganze im TNTCommand eher über ein Switch/case mit Enum lösen.
Autor
Besitzer

Sollte dann nun so sein wie du willst.

Sollte dann nun so sein wie du willst.
Lixfel hat 2021-01-22 07:23:15 +01:00 Änderungen angefragt
@ -32,1 +44,3 @@
return getInstance(CommandTNT.class);
public enum TNTMode {
ON("§aan"),
ONLY_TB("§7nur §eTestblock"),
Besitzer

Nur Testblock ist ja so nicht ganz richtig. Ist ja eher ein Nicht Baubereich (klingt aber beschissen). Evtl. fällt uns da noch was besseres ein.

Nur Testblock ist ja so nicht ganz richtig. Ist ja eher ein Nicht Baubereich (klingt aber beschissen). Evtl. fällt uns da noch was besseres ein.
Autor
Besitzer

Wie würdest du das dann nennen?

Wie würdest du das dann nennen?
@ -46,2 +71,4 @@
}
private String getTestblockEnableMessage() {
return "§aTNT-Schaden beim Testblock aktiviert";
Besitzer

am, nicht beim.

am, nicht beim.
@ -48,0 +75,4 @@
}
private String getDamageMessage() {
return "§cEs ist etwas explodiert und hätte blöcke zerstört";
Besitzer

§cEine Explosion hätte Blöcke im Baubereich zerstört

§cEine Explosion hätte Blöcke im Baubereich zerstört
@ -48,3 +111,2 @@
@EventHandler
public void onExplode(EntityExplodeEvent e) {
e.blockList().clear();
public void onExplode(EntityExplodeEvent event) {
Besitzer

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

Du registrierst den Command nirgends als Listener (weder implements noch registerEvents().
@ -51,0 +113,4 @@
if (tntMode == TNTMode.ON) return;
if (tntMode == TNTMode.OFF) {
event.blockList().clear();
} else {
Besitzer

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.
@ -51,0 +122,4 @@
}
return false;
});
if (!blocksDestroyed) {
Besitzer

Die invertierung macht hier den Code etwas schwerer zu lesen, da dahinter eh nur eine Zeile kommt, kannst du dieses if zurückinvertieren und dann einfach sendToActionBar aufrufen. Dann hast du da weniger Probleme.

Die invertierung macht hier den Code etwas schwerer zu lesen, da dahinter eh nur eine Zeile kommt, kannst du dieses if zurückinvertieren und dann einfach sendToActionBar aufrufen. Dann hast du da weniger Probleme.
@ -51,1 +130,4 @@
}
private void sendToActionBar(String message) {
Bukkit.getOnlinePlayers().forEach(p -> p.spigot().sendMessage(ChatMessageType.ACTION_BAR, TextComponent.fromLegacyText(message)));
Besitzer

Auch mal mit 1.12 kurz kompilieren (sollte funzen, bin mir aber nicht ganz sicher)

Auch mal mit 1.12 kurz kompilieren (sollte funzen, bin mir aber nicht ganz sicher)
Autor
Besitzer

Ja tut. Auch wenn du das gleiche schon in ToggleCommand verwendet hast.

Ja tut. Auch wenn du das gleiche schon in ToggleCommand verwendet hast.
Autor
Besitzer

Habe ich so nun behoben

Habe ich so nun behoben
Lixfel hat die Änderungen 2021-01-24 14:37:52 +01:00 genehmigt
Lixfel hat einen Kommentar hinterlassen
Besitzer

Anpassung aller sections.yml vorher nötig (zumindest 2,3,4)

Anpassung aller sections.yml vorher nötig (zumindest 2,3,4)
Lixfel hat die Änderungen 2021-01-24 20:08:57 +01:00 genehmigt
YoyoNow hat diesen Pull-Request 2021-01-24 20:19:52 +01:00 geschlossen
YoyoNow löschte die Branch TntMode 2021-01-24 20:20:19 +01:00
Dieses Repo ist archiviert. Du kannst Pull-Requests nicht kommentieren.
Keine Beschreibung angegeben.