spawn protection

if i wanted to code spawn protection where if a player steps off spawn, they lose it even if they go back in,
would i have to use playermoveevent, and what would be the most effecient way of doing it

 

What is "it" that they lose?
Please make an antecedent to your pronoun before using it...
Grammar 101

 

rocket138
uh they lose spawn protection

 

PlayerMoveEvent is the only way I can think of, so there wouldn't really be a more efficient way. Don't forget to listen to PlayerTeleportEvent tho ;3.

Also you could keep a Set<String> with playernames that have protection and should be checked. That Set could be used for the protection itself as well, while you remove them as they leave the protection.

 

fireblast709
ok thanks

is this code right
am i missing anything, guys?

Code:java

ArrayList<String> spawn = new ArrayList<String>();
    @EventHandler
    public void onPlayerJoin(PlayerJoinEvent event) {
        Player player = event.getPlayer();
        if (!player.hasPlayedBefore()){
            spawn.add(player.getName());
        }
    }
    @EventHandler
    public void spawnProtection(PlayerTeleportEvent event){
        Player player = event.getPlayer();
        if (Math.abs(player.getLocation().getBlockX()) <= 20
                && Math.abs(player.getLocation().getBlockZ()) <= 20){
            if (!spawn.contains(player.getName())){
                spawn.add(player.getName());
                player.sendMessage(ChatColor.LIGHT_PURPLE + "You have regained spawn protection.");
            }
        }
        else{
            if (spawn.contains(player.getName())){
                player.sendMessage(ChatColor.LIGHT_PURPLE + "You no longer have spawn protection.");
            }
        }
    }
    @EventHandler
    public void spawnProtection(PlayerMoveEvent event){
        Player player = event.getPlayer();
        if (Math.abs(player.getLocation().getBlockX()) >= 20
            ||    Math.abs(player.getLocation().getBlockZ()) >= 20){
            if (spawn.contains(player.getName())){
                spawn.remove(player.getName());
                player.sendMessage(ChatColor.LIGHT_PURPLE + "You no longer have spawn protection.");
            }
        }
    }

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

 

vtg_the_kid Seems about right on the first glance. Not sure if you need the hasPlayedBefore() check, since they would spawn in spawn anyway.

Also, you might want to integrate the current spawnpoint so you could freely set spawn without worries, but that is optional.

 

fireblast709
would it cause a lot of lag to my server

 

vtg_the_kid it should not

 

fireblast709
is there a more efficient way to do this

 

Not particularly.

 

vtg_the_kid No need to worry about efficiency, especially in a plugin as small as this with minimal scalability.