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

Fixing Detonator #158

Manuell gemergt
YoyoNow hat 9 Commits von fixing_some_detonator nach master 2021-01-09 20:54:38 +01:00 zusammengeführt
Diskussion 0 Commits 9 Geänderte Dateien 4 +15 -19
2 geänderte Dateien mit 15 neuen und 19 gelöschten Zeilen
Statistiken anzeigen Patch-Datei herunterladen Vergleichs-Datei herunterladen Alle Dateien ausklappen Alle Dateien einklappen
Nur Änderungen aus Commit b1672be384 werden angezeigt - Alle Commits anzeigen

15
BauSystem_15/src/de/steamwar/bausystem/world/AutoLoader_15.java
Datei anzeigen

@ -49,6 +49,7 @@ class AutoLoader_15 {
if (!(data instanceof Powerable)) if (!(data instanceof Powerable))
return false; return false;
Powerable powerable = (Powerable) data; Powerable powerable = (Powerable) data;
powerable.setPowered(active); powerable.setPowered(active);
block.setBlockData(powerable); block.setBlockData(powerable);
@ -59,7 +60,7 @@ class AutoLoader_15 {
block.setBlockData(openable); block.setBlockData(openable);
} }
if (data instanceof Switch){ if (data instanceof Switch) {
Switch swtch = (Switch) data; Switch swtch = (Switch) data;
Block relative; Block relative;
switch(swtch.getFace()){ switch(swtch.getFace()){
@ -74,7 +75,7 @@ class AutoLoader_15 {
break; break;
} }
updateBlock(relative); updateBlock(relative);
}else if(block.getType() == Material.TRIPWIRE) { } else if(block.getType() == Material.TRIPWIRE) {
updateBlock(block); updateBlock(block);
} }
return true; return true;
@ -97,11 +98,7 @@ class AutoLoader_15 {
Material material = block.getType(); Material material = block.getType();
if (material == Material.LEVER) { if (material == Material.LEVER) {
if (powerable.isPowered()) { return new Detoloader("Hebel", 0).setActive(!powerable.isPowered());
return new Detoloader("Hebel", 0).setActive(false);
} else {
return new Detoloader("Hebel", 0).setActive(true);
}
} else if (material == Material.STONE_BUTTON) { } else if (material == Material.STONE_BUTTON) {
return new Detoloader("Knopf", Detoloader.STONE_BUTTON); return new Detoloader("Knopf", Detoloader.STONE_BUTTON);
} else if (material.name().contains("PRESSURE_PLATE")){ } else if (material.name().contains("PRESSURE_PLATE")){
@ -118,6 +115,8 @@ class AutoLoader_15 {
} }
static boolean getLever(Block block) { static boolean getLever(Block block) {
return ((Powerable)block.getBlockData()).isPowered(); if(!(block.getBlockData() instanceof Powerable))
return false;
YoyoNow hat 2021-01-05 16:39:48 +01:00 kommentiert
Review
Link kopieren

Bitte den Zeilenumbruch weg machen

Bitte den Zeilenumbruch weg machen
return ((Powerable) block.getBlockData()).isPowered();
} }
} }

19
BauSystem_Main/src/de/steamwar/bausystem/world/Detonator.java
Datei anzeigen

@ -41,7 +41,7 @@ import java.util.stream.Collectors;
public class Detonator implements Listener { public class Detonator implements Listener {
public static final ItemStack WAND; public static final ItemStack WAND;
private static final Map<Player, Detonator> PLAYERS = new HashMap<>(); private static final Map<Player, Detonator> players = new HashMap<>();
Lixfel hat 2021-01-05 08:59:58 +01:00 kommentiert
Veraltet
Review
Link kopieren

In dem Fall ist die GROßSCHREIBUNG falsch, da es sich hier nicht um eine Konstante handelt. Die Map wird schließlich ständig verändert.

In dem Fall ist die GROßSCHREIBUNG falsch, da es sich hier nicht um eine Konstante handelt. Die Map wird schließlich ständig verändert.
private final Set<Detoloader.DetonatorActivation> locs = new HashSet<>(); private final Set<Detoloader.DetonatorActivation> locs = new HashSet<>();
private final Player player; private final Player player;
@ -61,19 +61,20 @@ public class Detonator implements Listener {
} }
public static Detonator getDetonator(Player player){ public static Detonator getDetonator(Player player){
if(!PLAYERS.containsKey(player)) if(!players.containsKey(player))
return new Detonator(player); return new Detonator(player);
return PLAYERS.get(player); return players.get(player);
} }
public Detonator(Player player) { public Detonator(Player player) {
this.player = player; this.player = player;
Bukkit.getPluginManager().registerEvents(this, BauSystem.getPlugin()); Bukkit.getPluginManager().registerEvents(this, BauSystem.getPlugin());
PLAYERS.put(player, this); players.put(player, this);
} }
public static void deleteDetonator(Player player) { public static void deleteDetonator(Player player) {
HandlerList.unregisterAll(PLAYERS.remove(player)); if(players.containsKey(player))
Lixfel hat 2021-01-05 09:00:26 +01:00 kommentiert
Veraltet
Review
Link kopieren

NPE möglich.

NPE möglich.
HandlerList.unregisterAll(players.remove(player));
YoyoNow hat 2021-01-05 16:37:05 +01:00 kommentiert
Review
Link kopieren

Kannst du das hier auch noch callen, wenn ich den Detonator aus dem Inventar werfe bzw lösche oder geht das nicht? Oder mach es so, dass du nur immer einen detonator hast. Außerdem bitte auch beim leaven des Servers Detonator#deleteDetonator(Player) aufrufen. Da sonst memory leaks aufkommen können.

Kannst du das hier auch noch callen, wenn ich den Detonator aus dem Inventar werfe bzw lösche oder geht das nicht? Oder mach es so, dass du nur immer einen detonator hast. Außerdem bitte auch beim leaven des Servers Detonator#deleteDetonator(Player) aufrufen. Da sonst memory leaks aufkommen können.
Chaoscaot hat 2021-01-05 17:17:47 +01:00 kommentiert
Review
Link kopieren

Wird es schon.

Wird es schon.
YoyoNow hat 2021-01-05 17:20:25 +01:00 kommentiert
Review
Link kopieren

habe ich im code nicht gesehen. Also diese Methode wird nicht direkt aufgerufen oder?

habe ich im code nicht gesehen. Also diese Methode wird nicht direkt aufgerufen oder?
} }
public void execute() { public void execute() {
@ -86,12 +87,7 @@ public class Detonator implements Listener {
YoyoNow hat 2021-01-05 16:33:00 +01:00 kommentiert
Review
Link kopieren

Unnötige Leerzeile

Unnötige Leerzeile
}, 12), new VersionedRunnable(() -> { }, 12), new VersionedRunnable(() -> {
boolean powered; boolean powered;
try { powered = AutoLoader_15.getLever(activation.location.getBlock());
powered = AutoLoader_15.getLever(activation.location.getBlock());
} catch (ClassCastException e) {
locs.remove(activation);
return;
}
AutoLoader_15.setRedstone(activation.location, !powered); AutoLoader_15.setRedstone(activation.location, !powered);
Lixfel hat 2021-01-05 09:01:21 +01:00 kommentiert
Veraltet
Review
Link kopieren

Warum sollte hier überhaupt eine ClassCastException auftreten? Kann das nicht bereits vorher abgefangen werden durch saubere Programmierung?

Warum sollte hier überhaupt eine ClassCastException auftreten? Kann das nicht bereits vorher abgefangen werden durch saubere Programmierung?
Chaoscaot hat 2021-01-05 12:34:43 +01:00 kommentiert
Veraltet
Review
Link kopieren

Diese ClassCastException kommt von einem früheren fix, wo man den Block nach der Registrierung ändern konnte. Der neue Block war dann nicht mehr zu einer Powerable zu Casten, und somit wurde er dann auch entfernt.

Diese ClassCastException kommt von einem früheren fix, wo man den Block nach der Registrierung ändern konnte. Der neue Block war dann nicht mehr zu einer Powerable zu Casten, und somit wurde er dann auch entfernt.
}, 15)); }, 15));
} else { } else {
@ -113,6 +109,7 @@ public class Detonator implements Listener {
public void onPlayerInteract(PlayerInteractEvent event) { public void onPlayerInteract(PlayerInteractEvent event) {
if(!event.getPlayer().equals(player)) if(!event.getPlayer().equals(player))
return; return;
if (event.getItem() == null) return; if (event.getItem() == null) return;
if (event.getItem().isSimilar(WAND)) { if (event.getItem().isSimilar(WAND)) {
event.setCancelled(true); event.setCancelled(true);