Reworking geometrie/area calculation

missilewars-plugin/src/main/java/de/butzlabben/missilewars/configuration/Lobby.java

@@ -20,10 +20,11 @@ package de.butzlabben.missilewars.configuration;
 
 import com.google.gson.annotations.SerializedName;
 import de.butzlabben.missilewars.Logger;
+import de.butzlabben.missilewars.configuration.arena.AreaConfiguration;
 import de.butzlabben.missilewars.configuration.arena.Arena;
 import de.butzlabben.missilewars.game.Arenas;
 import de.butzlabben.missilewars.game.enums.MapChooseProcedure;
-import de.butzlabben.missilewars.util.geometry.Area;
+import de.butzlabben.missilewars.util.geometry.GameArea;
 import de.butzlabben.missilewars.util.serialization.Serializer;
 import lombok.Getter;
 import lombok.RequiredArgsConstructor;
@@ -48,7 +49,7 @@ public class Lobby {
     private String name = "lobby0";
     @SerializedName("display_name") private String displayName = "&eDefault game";
     @SerializedName("auto_load") private boolean autoLoad = true;
-    @SerializedName("world") private String world = Bukkit.getWorlds().get(0).getName();
+    @SerializedName("world") private String world = getDefaultWorld().getName();
     @SerializedName("lobby_time") private int lobbyTime = 60;
     @SerializedName("join_ongoing_game") private boolean joinOngoingGame = false;
     @SerializedName("min_size") private int minSize = 2;
@@ -57,13 +58,14 @@ public class Lobby {
     @SerializedName("team1_color") private String team1Color = "&c";
     @SerializedName("team2_name") private String team2Name = "Team2";
     @SerializedName("team2_color") private String team2Color = "&a";
-    @Setter @SerializedName("spawn_point") private Location spawnPoint = Bukkit.getWorlds().get(0).getSpawnLocation();
-    @Setter @SerializedName("after_game_spawn") private Location afterGameSpawn = Bukkit.getWorlds().get(0).getSpawnLocation();
-    private Area area = Area.defaultAreaAround(Bukkit.getWorlds().get(0).getSpawnLocation());
+    @Setter @SerializedName("spawn_point") private Location spawnPoint = getDefaultWorld().getSpawnLocation();
+    @Setter @SerializedName("after_game_spawn") private Location afterGameSpawn = getDefaultWorld().getSpawnLocation();
+    @SerializedName("area") private AreaConfiguration areaConfig = new AreaConfiguration(-30, 0, -72, 30, 256, 72);
     @SerializedName("map_choose_procedure") private MapChooseProcedure mapChooseProcedure = MapChooseProcedure.FIRST;
     @SerializedName("possible_arenas") private List<String> possibleArenas = new ArrayList<>() {{
         add("arena0");
     }};
+    private transient GameArea gameArea = new GameArea(Bukkit.getWorld(world), areaConfig);
 
     @Setter private transient File file;
 
@@ -75,6 +77,10 @@ public class Lobby {
         }
         return world;
     }
+    
+    private World getDefaultWorld() {
+        return Bukkit.getWorlds().get(0);
+    }
 
     public void checkForWrongArenas() {
         for (String arenaName : possibleArenas) {
@@ -102,11 +108,7 @@ public class Lobby {
         }
     }
 
-    public Location getAreaMinLocation() {
-        return new Location(getBukkitWorld(), area.getMinX(), area.getMinY(), area.getMinZ());
-    }
-
-    public Location getAreaMaxLocation() {
-        return new Location(getBukkitWorld(), area.getMaxX(), area.getMaxY(), area.getMaxZ());
+    public GameArea getArea() {
+        return gameArea;
     }
 }

missilewars-plugin/src/main/java/de/butzlabben/missilewars/configuration/arena/AreaArenaConfiguration.java

@@ -0,0 +1,83 @@
+/*
+ * This file is part of MissileWars (https://github.com/Butzlabben/missilewars).
+ * Copyright (c) 2018-2021 Daniel Nägele.
+ *
+ * MissileWars is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * MissileWars is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with MissileWars.  If not, see <https://www.gnu.org/licenses/>.
+ */
+
+package de.butzlabben.missilewars.configuration.arena;
+
+import com.google.gson.annotations.SerializedName;
+import de.butzlabben.missilewars.game.GameWorld;
+import de.butzlabben.missilewars.util.geometry.ImprovedArea;
+import lombok.Getter;
+import lombok.RequiredArgsConstructor;
+import lombok.ToString;
+import org.bukkit.Location;
+import org.bukkit.World;
+
+@Getter
+@ToString
+@RequiredArgsConstructor
+public class AreaArenaConfiguration {
+    
+    @SerializedName("min_x") private int minX = -50;
+    @SerializedName("min_y") private int minY = 0;
+    @SerializedName("min_z") private int minZ = -50;
+    @SerializedName("max_x") private int maxX = 50;
+    @SerializedName("max_y") private int maxY = 256;
+    @SerializedName("max_z") private int maxZ = 50;
+
+/*    public AreaArenaConfiguration(int minX, int minY, int minZ, int maxX, int maxY, int maxZ) {
+        this.minX = minX;
+        this.minY = minY;
+        this.minZ = minZ;
+        this.maxX = maxX;
+        this.maxY = maxY;
+        this.maxZ = maxZ;
+    }
+
+    public static AreaArenaConfiguration deserialize(Map<String, Object> serialized) {
+        int minX = (int) serialized.get("min_x");
+        int minY = (int) serialized.get("min_y");
+        int minZ = (int) serialized.get("min_z");
+
+        int maxX = (int) serialized.get("max_x");
+        int maxY = (int) serialized.get("max_y");
+        int maxZ = (int) serialized.get("max_z");
+        return new AreaArenaConfiguration(minX, minY, minZ, maxX, maxY, maxZ);
+    }
+    
+    @Override
+    public Map<String, Object> serialize() {
+        Map<String, Object> serialized = new HashMap<>();
+        serialized.put("min_x", minX);
+        serialized.put("min_y", minY);
+        serialized.put("min_z", minZ);
+        serialized.put("max_x", maxX);
+        serialized.put("max_y", maxY);
+        serialized.put("max_z", maxZ);
+        return serialized;
+    }*/
+    
+    public ImprovedArea getArea(GameWorld gameWorld) {
+        World world = gameWorld.getWorld();
+        
+        Location pos1 = new Location(world, minX, minY, minZ);
+        Location pos2 = new Location(world, maxX, maxY, maxZ);
+        
+        return new ImprovedArea(pos1, pos2);
+    }
+    
+}

missilewars-plugin/src/main/java/de/butzlabben/missilewars/configuration/arena/Arena.java

@@ -19,14 +19,11 @@
 package de.butzlabben.missilewars.configuration.arena;
 
 import com.google.gson.annotations.SerializedName;
-import de.butzlabben.missilewars.util.geometry.FlatArea;
-import de.butzlabben.missilewars.util.geometry.Plane;
 import de.butzlabben.missilewars.util.serialization.Serializer;
 import lombok.Getter;
 import lombok.Setter;
 import lombok.ToString;
 import org.bukkit.Location;
-import org.bukkit.util.Vector;
 
 import java.io.File;
 import java.io.IOException;
@@ -56,43 +53,24 @@ public class Arena implements Cloneable {
     @SerializedName("money") private MoneyConfiguration money = new MoneyConfiguration();
     @SerializedName("equipment_interval") private EquipmentIntervalConfiguration interval = new EquipmentIntervalConfiguration();
     @SerializedName("missile") private MissileConfiguration missileConfiguration = new MissileConfiguration();
-    @SerializedName("area") private FlatArea gameArea = new FlatArea(-30, -72, 30, 72);
-    
+    @SerializedName("area") private AreaConfiguration areaConfig = new AreaConfiguration(-30, 0, -72, 30, 256, 72);
+
     @SerializedName("spectator_spawn")
     @Setter
     private Location spectatorSpawn = new Location(null, 0, 100, 0, 90, 0);
-    
+
     @SerializedName("team1_spawn")
     @Setter
     private Location team1Spawn = new Location(null, 0.5, 100, 45.5, 180, 0);
-    
+
     @SerializedName("team2_spawn")
     @Setter
     private Location team2Spawn = new Location(null, 0.5, 100, -45.5, 0, 0);
 
     @Setter private transient File file;
-    
-    public Arena() {
-
-    }
-
-    public Plane getPlane1() {
-        Vector spawn1 = team1Spawn.toVector();
-        Vector normal = team2Spawn.toVector().subtract(spawn1);
-        return new Plane(spawn1, normal);
-    }
 
-    public Plane getPlane2() {
-        Vector spawn2 = team2Spawn.toVector();
-        Vector normal = team1Spawn.toVector().subtract(spawn2);
-        return new Plane(spawn2, normal);
-    }
+    public Arena() {
 
-    public boolean isInBetween(Vector point, Plane plane1, Plane plane2) {
-        double distanceBetween = plane1.distanceSquared(plane2.getSupport());
-        double distance1 = plane1.distanceSquared(point);
-        double distance2 = plane2.distanceSquared(point);
-        return distanceBetween > distance1 + distance2;
     }
 
     @Override

missilewars-plugin/src/main/java/de/butzlabben/missilewars/game/Game.java

@@ -49,6 +49,8 @@ import de.butzlabben.missilewars.listener.game.GameListener;
 import de.butzlabben.missilewars.listener.game.LobbyListener;
 import de.butzlabben.missilewars.player.MWPlayer;
 import de.butzlabben.missilewars.util.PlayerDataProvider;
+import de.butzlabben.missilewars.util.geometry.GameArea;
+import de.butzlabben.missilewars.util.geometry.Geometry;
 import de.butzlabben.missilewars.util.serialization.Serializer;
 import de.butzlabben.missilewars.util.version.VersionUtil;
 import lombok.Getter;
@@ -94,6 +96,7 @@ public class Game {
     private boolean ready = false;
     private boolean restart = false;
     private GameWorld gameWorld;
+    private GameArea gameArea;
     private long timestart;
     private Arena arena;
     private ScoreboardManager scoreboardManager;
@@ -129,9 +132,7 @@ public class Game {
             Logger.ERROR.log("None of the specified arenas match a real arena for the lobby " + lobby.getName());
             return;
         }
-
-        gameWorld = new GameWorld(this, "");
-
+        
         team1 = new Team(lobby.getTeam1Name(), lobby.getTeam1Color(), this);
         team2 = new Team(lobby.getTeam2Name(), lobby.getTeam2Color(), this);
 
@@ -432,29 +433,46 @@ public class Game {
         }
     }
 
+    /**
+     * This method checks if the location is inside in the lobby area.
+     * 
+     * @param location (Location) the location to be checked
+     * @return true, if it's in the lobby area
+     */
     public boolean isInLobbyArea(Location location) {
-        World world = location.getWorld();
-        if (world == null) return false;
-        if (world.getName().equals(lobby.getWorld())) return lobby.getArea().isInArea(location);
-        return false;
+        if (!Geometry.isInsideIn(location, lobby.getArea())) return false;
+        return true;
     }
 
-
+    /**
+     * This method checks if the location is inside in the game area.
+     *
+     * @param location (Location) the location to be checked
+     * @return true, if it's in the game area
+     */
     public boolean isInGameArea(Location location) {
-        if (isInGameWorld(location)) return arena.getGameArea().isInArea(location);
-        return false;
+        if (!Geometry.isInsideIn(location, gameArea)) return false;
+        return true;
     }
 
+    /**
+     * This method checks if the location is in the game world.
+     *
+     * @param location (Location) the location to be checked
+     * @return true, if it's in the game world
+     */
     public boolean isInGameWorld(Location location) {
-        World world = location.getWorld();
-        if (world == null) return false;
-
-        if (gameWorld != null) {
-            return gameWorld.isWorld(world);
-        }
-        return false;
+        if (!Geometry.isInWorld(location, gameArea.getWorld())) return false;
+        return true;
     }
 
+    /**
+     * This (shortcut) method checks if the location is inside in the lobby 
+     * area or inside in the game world.
+     *
+     * @param location (Location) the location to be checked
+     * @return true, if the statement is correct
+     */
     public boolean isIn(Location location) {
         return isInLobbyArea(location) || isInGameWorld(location);
     }
@@ -608,7 +626,7 @@ public class Game {
         // Are missiles only allowed to spawn inside the arena, between the two arena spawn points?
         boolean isOnlyBetweenSpawnPlaceable = this.arena.getMissileConfiguration().isOnlyBetweenSpawnPlaceable();
         if (isOnlyBetweenSpawnPlaceable) {
-            if (!this.arena.isInBetween(player.getLocation().toVector(), this.arena.getPlane1(), this.arena.getPlane2())) {
+            if (!isInGameArea(player.getLocation())) {
                 player.sendMessage(Messages.getMessage("missile_place_deny"));
                 return;
             }
@@ -657,10 +675,9 @@ public class Game {
         }
 
         this.arena = arena.clone();
-
-        // Load world
-        this.gameWorld = new GameWorld(this, this.arena.getTemplateWorld());
+        this.gameWorld = new GameWorld(this, arena.getTemplateWorld());
         this.gameWorld.load();
+        this.gameArea = new GameArea(gameWorld.getWorld(), arena.getAreaConfig());
 
         try {
             Serializer.setWorldAtAllLocations(this.arena, gameWorld.getWorld());

missilewars-plugin/src/main/java/de/butzlabben/missilewars/game/GameWorld.java

@@ -52,13 +52,14 @@ public class GameWorld {
     }
 
     public boolean isWorld(World world) {
-        if (world == null) {
-            return false;
-        }
+        if (world == null) return false;
+
+        if ((worldName == null) || (worldName.isEmpty())) throw new IllegalArgumentException("GameWorld must be loaded first: 'gameWorld.load()'");
         return world.getName().equals(worldName);
     }
 
     public World getWorld() {
+        if ((worldName == null) || (worldName.isEmpty())) throw new IllegalArgumentException("GameWorld must be loaded first: 'gameWorld.load()'");
         return Bukkit.getWorld(worldName);
     }
 

missilewars-plugin/src/main/java/de/butzlabben/missilewars/listener/game/GameListener.java

@@ -28,6 +28,7 @@ import de.butzlabben.missilewars.game.enums.GameResult;
 import de.butzlabben.missilewars.game.misc.RespawnGoldBlock;
 import de.butzlabben.missilewars.game.misc.Shield;
 import de.butzlabben.missilewars.player.MWPlayer;
+import de.butzlabben.missilewars.util.geometry.Geometry;
 import de.butzlabben.missilewars.util.version.VersionUtil;
 import org.bukkit.GameMode;
 import org.bukkit.Location;
@@ -75,13 +76,16 @@ public class GameListener extends GameBoundListener {
         if (event.getChangedType() != VersionUtil.getPortal()) return;
 
         Location location = event.getBlock().getLocation();
-
-        if (getGame().getArena().getPlane1().distance(location.toVector()) > getGame().getArena().getPlane2().distance(location.toVector())) {
-            getGame().getTeam1().setGameResult(GameResult.WIN);
-            getGame().getTeam2().setGameResult(GameResult.LOSE);
+        
+        Team team1 = getGame().getTeam1();
+        Team team2 = getGame().getTeam2();
+        
+        if (Geometry.isCloser(location, team1.getSpawn(), team2.getSpawn())) {
+            team1.setGameResult(GameResult.LOSE);
+            team2.setGameResult(GameResult.WIN);
         } else {
-            getGame().getTeam1().setGameResult(GameResult.LOSE);
-            getGame().getTeam2().setGameResult(GameResult.WIN);
+            team1.setGameResult(GameResult.WIN);
+            team2.setGameResult(GameResult.LOSE);
         }
         
         getGame().sendGameResult();

missilewars-plugin/src/main/java/de/butzlabben/missilewars/util/geometry/Area.java

@@ -1,113 +0,0 @@
-/*
- * This file is part of MissileWars (https://github.com/Butzlabben/missilewars).
- * Copyright (c) 2018-2021 Daniel Nägele.
- *
- * MissileWars is free software: you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * MissileWars is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with MissileWars.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package de.butzlabben.missilewars.util.geometry;
-
-import com.google.gson.annotations.SerializedName;
-import lombok.AllArgsConstructor;
-import lombok.Getter;
-import lombok.ToString;
-import org.bukkit.Location;
-import org.bukkit.configuration.serialization.ConfigurationSerializable;
-
-import java.util.HashMap;
-import java.util.Map;
-
-@ToString
-@AllArgsConstructor
-@Getter
-public class Area implements ConfigurationSerializable {
-
-    @SerializedName("min_x") private int minX;
-    @SerializedName("min_y") private int minY;
-    @SerializedName("min_z") private int minZ;
-    @SerializedName("max_x") private int maxX;
-    @SerializedName("max_y") private int maxY;
-    @SerializedName("max_z") private int maxZ;
-    private transient boolean checked;
-
-    public static Area deserialize(Map<String, Object> serialized) {
-        int minX = (int) serialized.get("min_x");
-        int minY = (int) serialized.get("min_y");
-        int minZ = (int) serialized.get("min_z");
-
-        int maxX = (int) serialized.get("max_x");
-        int maxY = (int) serialized.get("max_y");
-        int maxZ = (int) serialized.get("max_z");
-        return new Area(minX, minY, minZ, maxX, maxY, maxZ, false);
-    }
-
-    public static Area defaultAreaAround(Location location) {
-        return new Area(location.getBlockX() - 20,
-                location.getBlockY() - 20,
-                location.getBlockZ() - 20,
-                location.getBlockX() + 20,
-                location.getBlockY() + 20,
-                location.getBlockZ() + 20, true);
-    }
-
-    public boolean isInArea(double x, double y, double z) {
-        checkValues();
-        return x >= minX && x <= maxX &&
-                y >= minY && y <= maxY &&
-                z >= minZ && z <= maxZ;
-    }
-
-    public boolean isInArea(Location loc) {
-        double x = loc.getX();
-        double y = loc.getY();
-        double z = loc.getZ();
-        return isInArea(x, y, z);
-    }
-
-    void checkValues() {
-        if (checked)
-            return;
-
-        if (minX >= maxX) {
-            int oldMin = minX;
-            this.minX = maxX;
-            this.maxX = oldMin;
-        }
-
-        if (minY >= maxY) {
-            int oldMin = minY;
-            this.minY = maxY;
-            this.maxY = oldMin;
-        }
-
-        if (minZ >= maxZ) {
-            int oldMin = minZ;
-            this.minZ = maxZ;
-            this.maxZ = oldMin;
-        }
-        checked = true;
-    }
-
-    @Override
-    public Map<String, Object> serialize() {
-        Map<String, Object> serialized = new HashMap<>();
-        serialized.put("min_x", minX);
-        serialized.put("min_y", minY);
-        serialized.put("min_z", minZ);
-        serialized.put("max_x", maxX);
-        serialized.put("max_y", maxY);
-        serialized.put("max_z", maxZ);
-        return serialized;
-    }
-}

missilewars-plugin/src/main/java/de/butzlabben/missilewars/util/geometry/FlatArea.java

@@ -1,40 +0,0 @@
-/*
- * This file is part of MissileWars (https://github.com/Butzlabben/missilewars).
- * Copyright (c) 2018-2021 Daniel Nägele.
- *
- * MissileWars is free software: you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * MissileWars is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with MissileWars.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package de.butzlabben.missilewars.util.geometry;
-
-
-import org.bukkit.Location;
-
-public class FlatArea extends Area {
-
-    public FlatArea(int minX, int minZ, int maxX, int maxZ) {
-        super(minX, 0, minZ, maxX, 0, maxZ, false);
-    }
-
-    public boolean isInArea(double x, double z) {
-        return super.isInArea(x, 0, z);
-    }
-
-    @Override
-    public boolean isInArea(Location loc) {
-        long x = Math.round(loc.getX());
-        long z = Math.round(loc.getZ());
-        return isInArea(x, z);
-    }
-}

missilewars-plugin/src/main/java/de/butzlabben/missilewars/util/geometry/GameArea.java

@@ -0,0 +1,146 @@
+/*
+ * This file is part of MissileWars (https://github.com/Butzlabben/missilewars).
+ * Copyright (c) 2018-2021 Daniel Nägele.
+ *
+ * MissileWars is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * MissileWars is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with MissileWars.  If not, see <https://www.gnu.org/licenses/>.
+ */
+
+package de.butzlabben.missilewars.util.geometry;
+
+import de.butzlabben.missilewars.configuration.arena.AreaConfiguration;
+import org.bukkit.Location;
+import org.bukkit.World;
+
+public class GameArea {
+
+    final World world;
+
+    final Location position1, position2;
+
+    long minX, minY, minZ;
+
+    long maxX, maxY, maxZ;
+    Direction direction;
+
+    public GameArea(Location pos1, Location pos2) {
+        
+        if (!Geometry.bothLocInSameWorld(pos1, pos2)) throw new IllegalArgumentException("Defined positions are not in the same world!");
+        if (pos1 == pos2) throw new IllegalArgumentException("The selected positions do not differ.");
+        
+        this.world = pos1.getWorld();
+        
+        this.position1 = pos1;
+        this.position2 = pos2;
+        
+        calculate();
+    }
+    
+    public GameArea(Location center, int offset, int high) {
+        
+        if (offset < 1) throw new IllegalArgumentException("The offset must be higher than 0.");
+        if (high < 2) throw new IllegalArgumentException("The high must be higher than 1.");
+        
+        this.world = center.getWorld();
+        
+        long x1 = center.getBlockX() + offset;
+        long x2 = center.getBlockX() - offset;
+        long z1 = center.getBlockZ() + offset;
+        long z2 = center.getBlockZ() - offset;
+
+        long y1 = center.getBlockY();
+        long y2 = y1 + high;
+
+        this.position1 = new Location(center.getWorld(), x1, y1, z1);
+        this.position2 = new Location(center.getWorld(), x2, y2, z2);
+
+        calculate();
+    }
+    
+    public GameArea(World world, AreaConfiguration areaConfig) {
+        
+        this.world = world;
+        
+        this.position1 = new Location(world, areaConfig.getMinX(), areaConfig.getMinY(), areaConfig.getMinZ());
+        this.position2 = new Location(world, areaConfig.getMaxX(), areaConfig.getMaxY(), areaConfig.getMaxZ());
+
+        if (position1 == position2) throw new IllegalArgumentException("The selected positions do not differ.");
+        
+        calculate();
+    }
+
+    private void calculate() {
+
+        if (position1.getBlockX() < position2.getBlockX()) {
+            minX = position1.getBlockX();
+            maxX = position2.getBlockX();
+        }
+
+        if (position1.getBlockY() < position2.getBlockY()) {
+            minY = position1.getBlockY();
+            maxY = position2.getBlockY();
+        }
+
+        if (position1.getBlockZ() < position2.getBlockZ()) {
+            minZ = position1.getBlockZ();
+            maxZ = position2.getBlockZ();
+        }
+    }
+
+    public World getWorld() {
+        return world;
+    }
+
+    public Location getPosition1() {
+        return position1;
+    }
+
+    public Location getPosition2() {
+        return position2;
+    }
+
+    public Direction getDirection() {
+        return direction;
+    }
+
+    public long getMinX() {
+        return minX;
+    }
+
+    public long getMinY() {
+        return minY;
+    }
+
+    public long getMinZ() {
+        return minZ;
+    }
+
+    public long getMaxX() {
+        return maxX;
+    }
+
+    public long getMaxY() {
+        return maxY;
+    }
+
+    public long getMaxZ() {
+        return maxZ;
+    }
+
+    private enum Direction {
+        NULL,
+        NORTH_SOUTH,
+        EAST_WEST;
+
+    }
+}

missilewars-plugin/src/main/java/de/butzlabben/missilewars/util/geometry/Geometry.java

@@ -0,0 +1,90 @@
+/*
+ * This file is part of MissileWars (https://github.com/Butzlabben/missilewars).
+ * Copyright (c) 2018-2021 Daniel Nägele.
+ *
+ * MissileWars is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * MissileWars is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with MissileWars.  If not, see <https://www.gnu.org/licenses/>.
+ */
+
+package de.butzlabben.missilewars.util.geometry;
+
+import org.bukkit.Location;
+import org.bukkit.World;
+
+public class Geometry {
+
+    /**
+     * This method checks if a location is closer to a target location
+     * than another location.
+     *
+     * @param targetLocation (Location) the start location for the distance measure
+     * @param closerLocation (Location) the closer location
+     * @param furtherAwayLocation (Location) the location that is further away
+     * @return true, if the statement is correct
+     */
+    public static boolean isCloser(Location targetLocation, Location closerLocation, Location furtherAwayLocation) {
+        if (targetLocation.distanceSquared(closerLocation) < targetLocation.distanceSquared(furtherAwayLocation)) return true;
+
+        return false;
+    }
+
+    /**
+     * This method checks if a location is within an arena. The border of the
+     * arena (1 block wide) is still part of the arena (= inside).
+     *
+     * @param targetLocation (Location) the location to be checked
+     * @param area (Location) the arena, which should be around the location
+     * @return true, if the statement is correct
+     */
+    public static boolean isInsideIn(Location targetLocation, GameArea area) {
+
+        if (!Geometry.bothLocInSameWorld(targetLocation, area.position1)) return false;
+
+        if (targetLocation.getBlockX() > area.maxX) return false;
+        if (targetLocation.getBlockX() < area.minX) return false;
+
+        if (targetLocation.getBlockY() > area.maxY) return false;
+        if (targetLocation.getBlockY() < area.minY) return false;
+
+        if (targetLocation.getBlockZ() > area.maxZ) return false;
+        if (targetLocation.getBlockZ() < area.minZ) return false;
+
+        return true;
+    }
+
+    /**
+     * This method checks, if booth locations are in the same world.
+     * 
+     * @param pos1 (Location) location 1
+     * @param pos2 (Location) location 2
+     * @return true, if they are in the same world
+     */
+    public static boolean bothLocInSameWorld(Location pos1, Location pos2) {
+        if ((pos1.getWorld() == null) || (pos2.getWorld() == null)) return false;
+        if (pos1.getWorld().getName().equals(pos2.getWorld().getName())) return true;
+        return false;
+    }
+
+    /**
+     * This method checks if a location is in a specified world.
+     * 
+     * @param targetLocation (Location) the location to be checked
+     * @param world (World) the target world
+     * @return true, if the statement is correct
+     */
+    public static boolean isInWorld(Location targetLocation, World world) {
+        if (targetLocation.getWorld().getName().equals(world.getName())) return true;
+        return false;
+    }
+    
+}

missilewars-plugin/src/main/java/de/butzlabben/missilewars/util/geometry/Line.java

@@ -1,71 +0,0 @@
-/*
- * This file is part of MissileWars (https://github.com/Butzlabben/missilewars).
- * Copyright (c) 2018-2021 Daniel Nägele.
- *
- * MissileWars is free software: you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * MissileWars is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with MissileWars.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package de.butzlabben.missilewars.util.geometry;
-
-import de.butzlabben.missilewars.util.MathUtil;
-import lombok.AllArgsConstructor;
-import lombok.EqualsAndHashCode;
-import lombok.ToString;
-import org.bukkit.util.Vector;
-
-/**
- * A 3-dimensional line in the form:
- * supporter + t*direction
- */
-@AllArgsConstructor
-@ToString
-@EqualsAndHashCode
-public class Line {
-
-    private final Vector support, direction;
-
-    public static Line fromPoints(final Vector point1, final Vector point2) {
-        return new Line(point1.clone(), point2.clone().subtract(point1.clone()));
-    }
-
-    public boolean isIn(final Vector point) {
-        final double tX = this.getTEquationSolved(this.support.getX(), this.direction.getX(), point.getX());
-        final double tY = this.getTEquationSolved(this.support.getY(), this.direction.getY(), point.getY());
-        final double tZ = this.getTEquationSolved(this.support.getZ(), this.direction.getZ(), point.getZ());
-        return MathUtil.closeEnoughEquals(tX, tY) && MathUtil.closeEnoughEquals(tX, tZ) && MathUtil.closeEnoughEquals(tY, tZ);
-    }
-
-    public double distance(final Vector point) {
-        final Vector closestPoint = this.closestPointTo(point);
-        return point.distance(closestPoint);
-    }
-
-    public Vector closestPointTo(final Vector point) {
-        if (this.isIn(point)) return point.clone();
-        final Plane helperPlane = new Plane(point, this.direction);
-        return helperPlane.getBreakThroughPoint(this).get();
-    }
-
-    private double getTEquationSolved(final double supportValue, final double directionValue, final double pointValue) {
-        return (pointValue - supportValue) / directionValue;
-    }
-
-    public Vector getSupport() {
-        return this.support.clone();
-    }
-
-    public Vector getDirection() {
-        return this.direction.clone();
-    }
-}

missilewars-plugin/src/main/java/de/butzlabben/missilewars/util/geometry/Plane.java

@@ -1,70 +0,0 @@
-/*
- * This file is part of MissileWars (https://github.com/Butzlabben/missilewars).
- * Copyright (c) 2018-2021 Daniel Nägele.
- *
- * MissileWars is free software: you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * MissileWars is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with MissileWars.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package de.butzlabben.missilewars.util.geometry;
-
-import de.butzlabben.missilewars.util.MathUtil;
-import lombok.AllArgsConstructor;
-import lombok.Getter;
-import lombok.ToString;
-import org.bukkit.util.Vector;
-
-import java.util.Optional;
-
-@Getter
-@AllArgsConstructor
-@ToString
-public class Plane {
-
-    private final Vector support, normal;
-
-    public boolean isIn(Vector point) {
-        return MathUtil.closeEnoughEquals(point.clone().subtract(support).dot(normal), 0);
-    }
-
-    public Vector closestPointTo(Vector point) {
-        if (isIn(point)) return point.clone();
-
-        Line supportLine = new Line(point, normal);
-        // we can safely get the value, as we know that this plane and the line are not parallel
-        return getBreakThroughPoint(supportLine).get();
-    }
-
-    public double distance(Vector point) {
-        Vector closestPoint = closestPointTo(point);
-        return point.distance(closestPoint);
-    }
-
-    public double distanceSquared(Vector point) {
-        Vector closestPoint = closestPointTo(point);
-        return point.distanceSquared(closestPoint);
-    }
-
-    public Optional<Vector> getBreakThroughPoint(Line line) {
-        if (MathUtil.closeEnoughEquals(line.getDirection().dot(normal), 0)) return Optional.empty();
-        double d = support.dot(normal);
-        double a = normal.getX();
-        double b = normal.getY();
-        double c = normal.getZ();
-        Vector x = line.getSupport();
-        Vector y = line.getDirection();
-        double t = (d - a * x.getX() - b * x.getY() - c * x.getZ()) / (a * y.getX() + b * y.getY() + c * y.getZ());
-        Vector result = line.getSupport().add(line.getDirection().multiply(t)).clone();
-        return Optional.of(result);
-    }
-}