Run EventHandler for 30 Seconds

Just for practice, I'm working on a Ninja plugin which will add speed and jump potion effects for 30 seconds when a user types /ninja.

The only trouble is, with a Jump Boost of 10, I lose almost all of my hearts whenever I jump, so I created an EventHandler to prevent all damage, but I have two problems.

1. How can I make the event handler target only the player who typed the command?
2. How can I make the event handler run for only 30 seconds?

My event handler:

Code:JAVA

@EventHandler
public void onPlayerDamage(EntityDamageEvent event) {
    if(godModed.containsKey(event.getEntity())) {
        event.setCancelled(true);
    }
}

Thanks in advance! :3

 

You can unregister individual events after a certain amount of time but it would be pointless because you can't make events individual for players so when you remove the event it will affect all players.
EntityDamageEvent.getHandlerList().unregister() I belive it is, the same for any other event.

However, I suggest you don't use that, I just told you for future reference, instead just leave the event running at all times and do that check... but I suggest you check if it contains entity's name instead of entity and you should store String as map key instead of Entity or Player or whatever you're using because those tend to give you troubles if stored.

If you don't have values to that map you can just use a HashSet, it's like a HashMap but only with keys for quick-check.

 

Thanks for the reply! I decided to go with your idea. If it makes a check, how do I make it only last for 30 seconds?

 

in a plugin i use this

Code:java

 
public static void saveLogoutPlayer(final Player player) {
if(WorldUtils.getZworld() == player.getWorld()){
Bukkit.getServer().getScheduler().scheduleSyncRepeatingTask(Main.plugin, new Runnable(){
int logouttime = 30
public void run() {
if(logouttime != -1){
if(logouttime != 0){
player.sendMessage(ChatColor.RED + "" + logouttime + " seconds until save logout");
 
logouttime--;
}else{
player.kickPlayer("Logged out safe");
}
}
}
}, 0L, 20L);
}
}
 

i think u can change the code a bit to fit ur needs

oops didnt saw digi's idea, now im going to eat lasagna...

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

 

Hmm, actually the Map might be useful, use Map<String, Integer> and store player's name and System.currentTimeMilis() divided by 1000 to store seconds.
Then in the event just get the value and if it's null then it doesn't exist and move on, otherwise compare it against the current System.currentTimeMilis() divided by 1000 and see if the diference is not bigger than 30 seconds, if it is then you should remove the entry because it's a waste of space, otherwise block damage.