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 +23 -27
1 geänderte Dateien mit 23 neuen und 27 gelöschten Zeilen
Statistiken anzeigen Patch-Datei herunterladen Vergleichs-Datei herunterladen Alle Dateien ausklappen Alle Dateien einklappen
Nur Änderungen aus Commit 9150a3a4c7 werden angezeigt - Alle Commits anzeigen

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

@ -60,8 +60,8 @@ public class Detonator implements Listener {
WAND.setItemMeta(im); WAND.setItemMeta(im);
} }
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);
} }
@ -73,14 +73,14 @@ public class Detonator implements Listener {
} }
public static void deleteDetonator(Player player) { public static void deleteDetonator(Player player) {
if(players.containsKey(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)); 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() {
for (Detoloader.DetonatorActivation activation: getLocations()) { for (Detoloader.DetonatorActivation activation : getLocations()) {
YoyoNow hat 2021-01-05 16:32:44 +01:00 kommentiert
Review
Link kopieren

Die Zeile kann weg

Die Zeile kann weg
if(activation.activation == -1) { if (activation.activation == -1) {
VersionedRunnable.call(new VersionedRunnable(() -> { VersionedRunnable.call(new VersionedRunnable(() -> {
boolean powered = AutoLoader_12.getLever(activation.location.getBlock()); boolean powered = AutoLoader_12.getLever(activation.location.getBlock());
AutoLoader_12.setRedstone(activation.location, !powered); AutoLoader_12.setRedstone(activation.location, !powered);
@ -92,12 +92,10 @@ public class Detonator implements Listener {
} else { } else {
VersionedRunnable.call(new VersionedRunnable(() -> { VersionedRunnable.call(new VersionedRunnable(() -> {
AutoLoader_12.setRedstone(activation.location, true); AutoLoader_12.setRedstone(activation.location, true);
Bukkit.getScheduler().runTaskLater(BauSystem.getPlugin(), () -> Bukkit.getScheduler().runTaskLater(BauSystem.getPlugin(), () -> AutoLoader_12.setRedstone(activation.location, false), activation.activation);
AutoLoader_12.setRedstone(activation.location, false), activation.activation);
}, 12), new VersionedRunnable(() -> { }, 12), new VersionedRunnable(() -> {
YoyoNow hat 2021-01-05 16:36:02 +01:00 kommentiert
Veraltet
Review
Link kopieren

Lambda bitte in eine Zeile

Lambda bitte in eine Zeile
AutoLoader_15.setRedstone(activation.location, true); AutoLoader_15.setRedstone(activation.location, true);
Bukkit.getScheduler().runTaskLater(BauSystem.getPlugin(), () -> Bukkit.getScheduler().runTaskLater(BauSystem.getPlugin(), () -> AutoLoader_15.setRedstone(activation.location, false), activation.activation);
AutoLoader_15.setRedstone(activation.location, false), activation.activation);
}, 15)); }, 15));
} }
YoyoNow hat 2021-01-05 16:35:55 +01:00 kommentiert
Veraltet
Review
Link kopieren

Lambda bitte in eine Zeile

Lambda bitte in eine Zeile
} }
@ -106,7 +104,7 @@ public class Detonator implements Listener {
@EventHandler @EventHandler
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;
@ -115,12 +113,10 @@ public class Detonator implements Listener {
switch (event.getAction()) { switch (event.getAction()) {
case LEFT_CLICK_BLOCK: case LEFT_CLICK_BLOCK:
Detoloader detoloader = VersionedCallable.call( Detoloader detoloader = VersionedCallable.call(
new VersionedCallable<>(() -> new VersionedCallable<>(() -> AutoLoader_12.onPlayerInteractLoader(event), 12),
AutoLoader_12.onPlayerInteractLoader(event), 12), new VersionedCallable<>(() -> AutoLoader_15.onPlayerInteractLoader(event), 15));
new VersionedCallable<>(() ->
AutoLoader_15.onPlayerInteractLoader(event), 15));
YoyoNow hat 2021-01-05 16:33:31 +01:00 kommentiert
Veraltet
Review
Link kopieren

Auch hier Lambdas in die gleiche Zeile

Auch hier Lambdas in die gleiche Zeile
if(detoloader == null) { if (detoloader == null) {
return; return;
} else if (detoloader.activation == -1) { } else if (detoloader.activation == -1) {
print(detoloader.getBlock(), detoloader.addBack); print(detoloader.getBlock(), detoloader.addBack);
@ -128,25 +124,25 @@ public class Detonator implements Listener {
} }
if(event.getPlayer().isSneaking()) { if (event.getPlayer().isSneaking()) {
if(locs.stream().filter(detonatorActivation -> detonatorActivation.location.equals(event.getClickedBlock().getLocation())).collect(Collectors.toList()).size() == 1) { if (locs.stream().filter(detonatorActivation -> detonatorActivation.location.equals(event.getClickedBlock().getLocation())).collect(Collectors.toList()).size() == 1) {
locs.removeIf(detonatorActivation -> detonatorActivation.location.equals(event.getClickedBlock().getLocation())); locs.removeIf(detonatorActivation -> detonatorActivation.location.equals(event.getClickedBlock().getLocation()));
print(detoloader.addBack ? "§e" + detoloader.getBlock() + " entfernt" : print(detoloader.addBack ? "§e" + detoloader.getBlock() + " entfernt" :
detoloader.getBlock(), detoloader.addBack); detoloader.getBlock(), detoloader.addBack);
}else { } else {
if(detoloader.getActivation() == 0) { if (detoloader.getActivation() == 0) {
locs.add(new Detoloader.DetonatorActivation(event.getClickedBlock().getLocation())); locs.add(new Detoloader.DetonatorActivation(event.getClickedBlock().getLocation()));
}else { } else {
locs.add(new Detoloader.DetonatorActivation(detoloader.getActivation(), event.getClickedBlock().getLocation())); locs.add(new Detoloader.DetonatorActivation(detoloader.getActivation(), event.getClickedBlock().getLocation()));
} }
print(detoloader.addBack ? "§e" + detoloader.getBlock() + " hinzugefügt" : print(detoloader.addBack ? "§e" + detoloader.getBlock() + " hinzugefügt" :
detoloader.getBlock(), detoloader.addBack); detoloader.getBlock(), detoloader.addBack);
} }
}else { } else {
locs.clear(); locs.clear();
if(detoloader.getActivation() == 0) { if (detoloader.getActivation() == 0) {
locs.add(new Detoloader.DetonatorActivation(event.getClickedBlock().getLocation())); locs.add(new Detoloader.DetonatorActivation(event.getClickedBlock().getLocation()));
}else { } else {
locs.add(new Detoloader.DetonatorActivation(detoloader.getActivation(), event.getClickedBlock().getLocation())); locs.add(new Detoloader.DetonatorActivation(detoloader.getActivation(), event.getClickedBlock().getLocation()));
} }
print(detoloader.addBack ? "§e" + detoloader.getBlock() + " getsetzt" : print(detoloader.addBack ? "§e" + detoloader.getBlock() + " getsetzt" :
YoyoNow hat 2021-01-05 19:29:10 +01:00 kommentiert
Review
Link kopieren

Zeilenumbruch weg!

Zeilenumbruch weg!
@ -163,7 +159,7 @@ public class Detonator implements Listener {
@EventHandler @EventHandler
public void onPlayerQuit(PlayerQuitEvent event) { public void onPlayerQuit(PlayerQuitEvent event) {
if(event.getPlayer().equals(player)) if (event.getPlayer().equals(player))
deleteDetonator(player); deleteDetonator(player);
} }
@ -179,8 +175,8 @@ public class Detonator implements Listener {
locs.clear(); locs.clear();
} }
void print(String message, boolean withSize){ void print(String message, boolean withSize) {
YoyoNow hat 2021-01-05 19:29:57 +01:00 kommentiert
Veraltet
Review
Link kopieren

Würdest du bitte hier bitte die Blöcke auch in geschweiften Klammern machen.

Würdest du bitte hier bitte die Blöcke auch in geschweiften Klammern machen.
if(withSize) if (withSize)
getPlayer().spigot().sendMessage(ChatMessageType.ACTION_BAR, TextComponent.fromLegacyText(message + " §8" + getLocations().size())); getPlayer().spigot().sendMessage(ChatMessageType.ACTION_BAR, TextComponent.fromLegacyText(message + " §8" + getLocations().size()));
else else
getPlayer().spigot().sendMessage(ChatMessageType.ACTION_BAR, TextComponent.fromLegacyText(message)); getPlayer().spigot().sendMessage(ChatMessageType.ACTION_BAR, TextComponent.fromLegacyText(message));