The Nearest of Nearby Players?

In my plugin, I want to let players interact with their compass to get the nearest player, and their compass will point to them. I can do player.getNearbyEntities(30, 30, 30) and check if the entity is a player, but if there are multiple players within a 30 block radius of their location, it will send the player multiple messages, and point their compass to multiple locations. How can I get the nearest of nearby players?

 

start with size 0 then work up in the radius till you find a player

 

Or loop through all and keep checking the distance. Keep the nearest everytime.

 

how would you do that?

would this work for that?

Code:

            for (Player player1 : Distance.playersDistance(event.getPlayer()
                    .getLocation(), 32)) {
                if (Distance.exactDistanceFromLocation(player.getWorld(),
                        player.getLocation(), player1.getLocation()) < Distance
                        .exactDistanceFromLocation(player.getWorld(),
                                player.getLocation(), player1.getLocation())) {
                    player.setCompassTarget(player1.getLocation());
                }
            }

Distance class here

Code:

   
    public static double exactDistanceFromLocation(World world, Location loc, Location loc1){
       
        ChunkCoordinates cords = new ChunkCoordinates();
       
        cords.b((int) loc.getX(), (int) loc.getY(), (int) loc.getZ());
       
        double distance = (double) Math.max(Math.abs(loc1.getX() - cords.x), Math.abs(loc1.getZ() - cords.z));
       
        return distance;
       
    }

Code:

   
    public static ArrayList<Player> playersDistance(Location loc, double distance){
       
        ArrayList<Player> players = new ArrayList<Player>();
       
        double i1 = loc.getX() + distance;
        double j1 = loc.getY() + distance;
        double k1 = loc.getZ() + distance;
       
        double i2 = loc.getX() - distance;
        double j2 = loc.getY() - distance;
        double k2 = loc.getZ() - distance;
       
        double i3 = 0;
        double j3 = 0;
        double k3 = 0;
       
        for(Player player : Bukkit.getOnlinePlayers()){
           
            if(player.getWorld().equals(loc.getWorld())){       
           
                i3 = player.getLocation().getX();
                j3 = player.getLocation().getY();
                k3 = player.getLocation().getZ();
               
                if(i3 <= i1 && i3 >= i2 && j3 <= j1 && j3 >= j2 && k3 <= k1 && k3 >= k2){
                   
                    players.add(player);
                   
                }
               
            }
           
        }
       
        return players;
       
    }

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

entity.getLocation().distanceSquared(player.getLocation());
Use distanceSquared because squaring another number before comparing takes up less resources than taking a squareroot (like getDistance() would do)

 

would it work without doing that?
because i just tried it and it didnt work

 

Well its a method for comparing . You would just loop over the entities, and keep overriding the LivingEntity if its closer

 

thats what i thought i was doing ._.

Code:

for (Player player1 : Distance.playersDistance(event.getPlayer()
                    .getLocation(), 32)) {
                if (Distance.exactDistanceFromLocation(player.getWorld(),
                        player.getLocation(), player1.getLocation()) < Distance
                        .exactDistanceFromLocation(player.getWorld(),
                                player.getLocation(), player1.getLocation())) {
                    player.setCompassTarget(player1.getLocation());
                }
            }

 

Code:java

public Player getClosestPlayer(Player player, double radius)
{
    double minimalDistance = Math.pow(radius, 2);
    double curDist;
    Player closest = null;
    for(Player p = Bukkit.getOnlinePlayers())
    {
        if(!p.equals(player))
        {
            curDist = player.getLocation().distanceSquared(p.getLocation());
            if( curDist < minimalDistance)
            {
                minimalDistance = curDist;
                closest = p;
            }
        }
    }
    return closest;
}

Seems legit to me