Make something that lasts an amount of time and cooldown?

Hi. I just started making plugins a few days ago, and I don't know much.
I'm making a kitpvp server, with a custom plugin for it. Here is an example:
there is a kit called ghost that makes you invisible when you hit the ground with wool.
Here's my question:
In this scenario, how would I make it so that the invisibility lasts 30 seconds, then shows the player again, and then make sure that the player can't do it again for 30 seconds?

(Sorry, I'm a noob )

 

1. Give the player a potion effect, invisible, for 30 seconds.
2. Add player to a global list and check whenever the event happens if the player isn't in that list
3. With a scheduler, after 60 seconds, remove him from the list.
Scheduler:
Bukkit.getScheduler().runTaskLater(Globals.plugin, new Runnable() {


@Override

publicvoid run() {
//remove him from list

}}, 60*20); //60 seconds * 20 ticks per second

 

Metal Julien When the player does /kit ghost, it adds their name to a global list of ghost players. When a ghost player hits a block with wool, it hides them from everyone. (using .hideplayer()) I made a scheduler-task thing, GhostUnhideTask:
It undoes the .hidePlayer(), but I can't figure out one thing: with .showPlayer(), you need to specify what player to show. The scheduler is in a different class, so how would I transfer the player that is being hidden in the main plugin class to the GhostUnhideTask class?

I can't make the Player player public, so I don't see how I would be able to access it from the GhostUnhideTask class...

Wait.. would this work, and I wouldn't need the GhostUnhideTask?

Code:java

@EventHandler
    public void onPlayerInteractBlock(PlayerInteractEvent event)
    {
        final Player player = event.getPlayer();
        if(ghostPlayers.contains(player.getName()) && player.getItemInHand().getType() == Material.WOOL && !ghostsOnCooldown.contains(player.getName()));
        {
            for(Player other : getServer().getOnlinePlayers())
            {
                other.hidePlayer(player);
            }
            Bukkit.getScheduler().runTaskLater(this, new Runnable()
            {
                @Override
                public void run()
                {
                    for(Player other : getServer().getOnlinePlayers())
                    {
                        other.showPlayer(player);
                    }
                }
            },60*20);
            player.sendMessage("Starting cooldown...");
            ghostsOnCooldown.add(player.getName());
            Bukkit.getScheduler().runTaskLater(this, new Runnable()
            {
                @Override
                public void run()
                {
                    ghostsOnCooldown.remove(player.getName());
                }
            },60*20);
        }
    }

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

 

EZ https://forums.bukkit.org/threads/easycooldown.128219/

You don't have to use the class but use the idea. Using schedulers is a bad way to do this.

 

mineman451 You add the player after 60 seconds to the list, do it so:

Code:

@EventHandler
    public void onPlayerInteractBlock(PlayerInteractEvent event)
    {
        final Player player = event.getPlayer();
        if(ghostPlayers.contains(player.getName()) && player.getItemInHand().getType() == Material.WOOL && !ghostsOnCooldown.contains(player.getName()));
        {
            for(Player other : getServer().getOnlinePlayers())
            {
                other.hidePlayer(player);
            }
 
            player.sendMessage("You're invisible. Starting cooldown...");
            ghostsOnCooldown.add(player.getName());
 
 
            Bukkit.getScheduler().runTaskLater(this, new Runnable()
            {
                @Override
                public void run()
                {
                    ghostsOnCooldown.remove(player.getName());
 
                    for(Player other : getServer().getOnlinePlayers())
                    {
                        other.showPlayer(player);
                    }
                }
            },60*20);
        }
    }
 

I hope that should work

 

That would put the cooldown into no use, wouldn;t it? you have it so that it starts the cooldown once the player becomes invisible, and it removes him/her from the cooldown before it even un-hides them.
PS
Does },60*20);
result in waiting 60 seconds? If it does, it would keep the player invisible for twice the amount of time it should.

 

mineman451 So do a bukkittask 30*20 inwhat you unhide the player, and in that bukkittask do another bukkittask 30*20 inwhere you remove him from list?