Tracking nearest Player with a compass??

Ok guys, so I am trying to make a plugin kinda like mc-hg's plugin where if you have a compass and rightclick the ground/air the compass points to the nearest player. This is the code I have so far but for some reason it does not work. Thanks for helping!

Code:

    @EventHandler
    public void onCompassTracker(PlayerInteractEvent e){
        Player p = e.getPlayer();
        Player target = e.getPlayer();
        if((e.getAction() == Action.RIGHT_CLICK_AIR || e.getAction() == Action.RIGHT_CLICK_BLOCK) && p.getItemInHand().getType() == Material.COMPASS){
            p.setCompassTarget(target.getLocation());
        }
       
    }

 

Tommy Raids Your target is the player that clicked so you are setting the compass to the player's location when he clicked. You need to get the nearest player and have a scheduler update the location every few ticks to "track" said player.

 

Tommy Raids
What you want to do is player's location and check to see if there are any players within a radius using a for loop/scheduler. Then, you want to get all of the entities in that radius, get the location of the other player and set the compass target to the other player's location.

 

Tommy Raids Here is a method for getting the closest player in a radius but be careful as it can return null is no player is within the chosen radius:

Code:java

    public Player getNearest(Player p, Double range) {
        double distance = Double.POSITIVE_INFINITY; // To make sure the first
                                                    // player checked is closest
        Player target = null;
        for (Entity e : p.getNearbyEntities(range, range, range)) {
            if (!(e instanceof Player))
                continue;
            double distanceto = p.getLocation().distance(e.getLocation());
            if (distanceto > distance)
                continue;
            distance = distanceto;
            target = (Player) e;
        }
        return target;
    }

 

JRL1004 I put it in my event but it doesn't work. It has some errors. Where would I put this code. And no I did not copy and paste.

 

Tommy Raids
Have that code in your PlayerInteractEvent in a scheduler that fires each time a player right clicks a compass.

 

Tommy Raids I did make one mistake, it should be:

Code:java

 public Player getNearest(Player p, Double range) {
double distance = Double.POSITIVE_INFINITY; // To make sure the first
// player checked is closest
Player target = null;
for (Entity e : p.getNearbyEntities(range, range, range)) {
if (!(e instanceof Player))
continue;
if(e == p) continue; //Added this check so you don't target yourself.
double distanceto = p.getLocation().distance(e.getLocation());
if (distanceto > distance)
continue;
distance = distanceto;
target = (Player) e;
}
return target;
}

and what errors are you getting?

 

JRL1004 Where do I put it In my event?

 

Tommy Raids It's not meant to be in an event, I coded it was a standalone method... You would use it like this:

Code:java

// Your method stuff
Player target = getNearest(/* Person you want to use */, /*distance to check around the person */);
//Then you need to make sure there was a player
if(target == null) {
  //Uh-oh, there was not a player in that range
}
else {
  //There was a player so set the compass
}

 

JRL1004 .. It still doesn't work. Whatever Im just a noob anyways. I know your trying your hardest. Sorry.

 

Tommy Raids It's no issue, I like helping others (That sounds like something from a book / movie huh?). Anyways, I need to go soon so I'll try and help you out tomorrow (assuming someone else has not solved the problem).

 

Thanks!It also helps me in my plugin!