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

Limit block breaking to team area #311

Zusammengeführt
Lixfel hat 3 Commits von blockBreakLimiter nach master 2021-12-23 12:13:58 +01:00 zusammengeführt
Diskussion 1 Commits 3 Geänderte Dateien 1 +11
1 geänderte Dateien mit 11 neuen und 0 gelöschten Zeilen
Statistiken anzeigen Patch-Datei herunterladen Vergleichs-Datei herunterladen Alle Dateien ausklappen Alle Dateien einklappen

11
FightSystem_Core/src/de/steamwar/fightsystem/listener/TeamArea.java
Datei anzeigen

@ -34,6 +34,7 @@ import org.bukkit.block.Block;
import org.bukkit.entity.Player; import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler; import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener; import org.bukkit.event.Listener;
import org.bukkit.event.block.BlockBreakEvent;
import org.bukkit.event.block.BlockPlaceEvent; import org.bukkit.event.block.BlockPlaceEvent;
import org.bukkit.event.player.PlayerMoveEvent; import org.bukkit.event.player.PlayerMoveEvent;
import org.bukkit.event.player.PlayerQuitEvent; import org.bukkit.event.player.PlayerQuitEvent;
@ -77,6 +78,16 @@ public class TeamArea implements Listener {
FightSystem.getMessage().sendPrefixless("NO_BLOCK_PLACE", player, ChatMessageType.ACTION_BAR); FightSystem.getMessage().sendPrefixless("NO_BLOCK_PLACE", player, ChatMessageType.ACTION_BAR);
} }
@EventHandler
public void blockBreak(BlockBreakEvent event) {
Block block = event.getBlock();
if(Config.BluePasteRegion.getMinY() <= block.getY())
Lixfel markierte diese Unterhaltung als gelöst Veraltet
YoyoNow hat 2021-12-23 12:05:07 +01:00 kommentiert
Veraltet
Review
Link kopieren

Ist dieser Check nicht falsch rum?

Nehmen wir an die PasteRegion minY ist bei 10
und der block der abgebaut wird ist bei 5
dann steht da:

if (10 >= 5) return

Womit der Block zerstörbar ist?
Oder anders rum steht da:

if (5 < 10) return
Ist dieser Check nicht falsch rum? Nehmen wir an die PasteRegion minY ist bei 10 und der block der abgebaut wird ist bei 5 dann steht da: ``` if (10 >= 5) return ``` Womit der Block zerstörbar ist? Oder anders rum steht da: ``` if (5 < 10) return ```
return;
YoyoNow hat 2021-12-23 12:05:29 +01:00 kommentiert
Review
Link kopieren

Ich mag es absolut nicht, dass dieses return in der nächsten Zeile steht, das kann nur zu programmierfehlern führen.

Ich mag es absolut nicht, dass dieses return in der nächsten Zeile steht, das kann nur zu programmierfehlern führen.
Lixfel hat 2021-12-23 12:10:10 +01:00 kommentiert
Review
Link kopieren

Ich mag es absolut nicht, wenn das return in der selben Zeile steht, weil das dazu führt, dass man das return übersieht.

Ich mag es absolut nicht, wenn das return in der selben Zeile steht, weil das dazu führt, dass man das return übersieht.
YoyoNow hat 2021-12-23 12:12:16 +01:00 kommentiert
Review
Link kopieren

ok, ist schon gut. ich habe nur absolut schlechte erfahren damit gemacht die klammern weg zu lassen und es eine zeile tiefer zu schreiben.

ok, ist schon gut. ich habe nur absolut schlechte erfahren damit gemacht die klammern weg zu lassen und es eine zeile tiefer zu schreiben.
Lixfel hat 2021-12-23 12:12:45 +01:00 kommentiert
Review
Link kopieren

Mit leerzeile danach gehts.

Mit leerzeile danach gehts.
event.setCancelled(true);
FightSystem.getMessage().sendPrefixless("NO_BLOCK_BREAK", event.getPlayer(), ChatMessageType.ACTION_BAR);
}
private void checkInTeamRegion(PlayerMoveEvent event, FightTeam team, Player player, FightTeam playerTeam) { private void checkInTeamRegion(PlayerMoveEvent event, FightTeam team, Player player, FightTeam playerTeam) {
boolean spectator = player.getGameMode() == GameMode.SPECTATOR; boolean spectator = player.getGameMode() == GameMode.SPECTATOR;
if(!spectator && playerTeam != null && playerTeam.canPlayerEntern(player)) if(!spectator && playerTeam != null && playerTeam.canPlayerEntern(player))