SteamWar/BauSystem
Archiviert
13
0

Fixing Loader String Double Activation #159

Manuell gemergt
YoyoNow hat 2 Commits von loader_string_interaction nach master 2021-01-09 20:55:21 +01:00 zusammengeführt
2 geänderte Dateien mit 15 neuen und 13 gelöschten Zeilen

Datei anzeigen

@ -83,5 +83,5 @@ public class Detoloader {
public static final int WOODEN_BUTTON = 30; public static final int WOODEN_BUTTON = 30;
public static final int PRESSURE_PLATE = 20; public static final int PRESSURE_PLATE = 20;
public static final int NOTE_BLOCK = 1; public static final int NOTE_BLOCK = 1;
public static final int TRIPWIRE = 10; public static final int TRIPWIRE = 20;
Review

Warum genau hast du hier oben die Zahl geändert?

Warum genau hast du hier oben die Zahl geändert?
} }

Datei anzeigen

@ -21,6 +21,7 @@ package de.steamwar.bausystem.world;
import de.steamwar.bausystem.BauSystem; import de.steamwar.bausystem.BauSystem;
import de.steamwar.core.Core; import de.steamwar.core.Core;
Review

Könntest du alle Sachen auf das neuere Version System umstellen aus dieser Klasse? Damit dieser Import vllt noch weggeht?

Könntest du alle Sachen auf das neuere Version System umstellen aus dieser Klasse? Damit dieser Import vllt noch weggeht?
Review

Nope, sind zwei separate PR.

Nope, sind zwei separate PR.
Review

Dann mach dafür gleich einen eigenen PR auf

Dann mach dafür gleich einen eigenen PR auf
import de.steamwar.core.VersionedCallable;
import org.bukkit.Bukkit; import org.bukkit.Bukkit;
import org.bukkit.Location; import org.bukkit.Location;
import org.bukkit.Material; import org.bukkit.Material;
@ -64,6 +65,7 @@ public class AutoLoader extends IAutoLoader implements Listener {
private int waitTicks; private int waitTicks;
private ListIterator<LoaderAction> lastAction; private ListIterator<LoaderAction> lastAction;
private boolean setup; private boolean setup;
private Location lastLocation;
private AutoLoader(Player player){ private AutoLoader(Player player){
this.player = player; this.player = player;
@ -163,28 +165,28 @@ public class AutoLoader extends IAutoLoader implements Listener {
if (event.getPlayer().isSneaking()) if (event.getPlayer().isSneaking())
return; return;
if(!setup || !event.getPlayer().equals(player)) if (!setup || !event.getPlayer().equals(player))
return; return;
Detoloader detoloader = null; Detoloader detoloader = VersionedCallable.call(
switch(Core.getVersion()){ new VersionedCallable<>(() -> AutoLoader_12.onPlayerInteractLoader(event), 12),
case 15: new VersionedCallable<>(() -> AutoLoader_15.onPlayerInteractLoader(event), 15));
Veraltet
Review

Was willst du machen, wenn du keinen hast? also nicht Version größer gleich 12

Was willst du machen, wenn du keinen hast? also nicht Version größer gleich 12
Veraltet
Review

Dann hat Spieler Pech gehabt, wir haben nur 1.12 und 1.15 Server, als ist das nicht anzunehmen.

Dann hat Spieler Pech gehabt, wir haben nur 1.12 und 1.15 Server, als ist das nicht anzunehmen.
detoloader = AutoLoader_15.onPlayerInteractLoader(event);
break;
default:
detoloader = AutoLoader_12.onPlayerInteractLoader(event);
} if (detoloader == null || detoloader.getActivation() < 0) return;
Veraltet
Review

Die Lambdas kann man in eine Zeile machen, ist lesbarer, siehe oben

Die Lambdas kann man in eine Zeile machen, ist lesbarer, siehe oben
if(detoloader == null || detoloader.getActivation() < 0) return;
if(detoloader.useActive) if (lastLocation != null && lastLocation.distance(event.getClickedBlock().getLocation()) <= 1) return;
Veraltet
Review

Kann der detoloader wirklich null sein?

Kann der detoloader wirklich null sein?
Veraltet
Review

Theoretisch nein, praktisch Ja

Theoretisch nein, praktisch Ja
if (detoloader.useActive) {
new IAutoLoader.RedstoneActivation(this, event.getClickedBlock().getLocation() new IAutoLoader.RedstoneActivation(this, event.getClickedBlock().getLocation()
Veraltet
Review

Klammern weg! Und hinter dem if ein ' ' space

Klammern weg! Und hinter dem if ein ' ' space
, detoloader.getActivation() == 0 ? getLastActivation() : detoloader.getActivation() , detoloader.getActivation() == 0 ? getLastActivation() : detoloader.getActivation()
Review

Ich würde hier die ',' immer in die Zeile davor schmeißen

Ich würde hier die ',' immer in die Zeile davor schmeißen
Review

Gehört nicht zum PR.

Gehört nicht zum PR.
Review

Dann mach dafür gleich einen eigenen PR auf

Dann mach dafür gleich einen eigenen PR auf
, detoloader.isActive()); , detoloader.isActive());
else } else {
new IAutoLoader.TemporaryActivation(this, event.getClickedBlock().getLocation() new IAutoLoader.TemporaryActivation(this, event.getClickedBlock().getLocation()
Review

Um diesen Block bitte geschweifte Klammern, weil dort mehrere Zeilen (durch die Umbrüche) drinstehen, genauso beim else. Sonst ist das nicht so gut zum lesen. Des weiteren bitte hinter dem if ein ' ' space

Um diesen Block bitte geschweifte Klammern, weil dort mehrere Zeilen (durch die Umbrüche) drinstehen, genauso beim else. Sonst ist das nicht so gut zum lesen. Des weiteren bitte hinter dem if ein ' ' space
, detoloader.getActivation()); , detoloader.getActivation());
Review

Ich würde hier die ‘,’ immer in die Zeile davor schmeißen

Ich würde hier die ‘,’ immer in die Zeile davor schmeißen
Review

Gehört nicht zum PR.

Gehört nicht zum PR.
Review

Dann mach dafür gleich einen eigenen PR auf

Dann mach dafür gleich einen eigenen PR auf
}
print(detoloader.addBack ? "§e" + detoloader.getBlock() + " betätigt" : print(detoloader.addBack ? "§e" + detoloader.getBlock() + " betätigt" :
detoloader.getBlock(), detoloader.addBack); detoloader.getBlock(), detoloader.addBack);
lastLocation = event.getClickedBlock().getLocation();
} }
@EventHandler @EventHandler