Make a correct cooldown time?

I was doing thread searching for how to make a cooldown with right-clicking with a specific item, and once you right-click with the item I cannot remove a player from the list, so I cannot use it anymore. Here is some lines of code:

Code:java

@EventHandler
public void onBlockInteract(final PlayerInteractEvent e) {
    Action act = e.getAction();
    final Player p = e.getPlayer();
    Block b = e.getClickedBlock();
 
    if (b != null) {
        Material hand = p.getItemInHand().getType();
        World w = p.getWorld();
 
        boolean enabled = plugin.getConfig().getBoolean("sulphur-explosive", true);
        int explosionSize = plugin.getConfig().getInt("explosion-size", 1);
 
    if (p.hasPermission("mobmeat.sulphur")) {
            if (hand.equals(Material.SULPHUR)) {
                if (act == Action.RIGHT_CLICK_BLOCK) {
                    hash.put(p.getName(), delay);
                        task = Bukkit.getScheduler().scheduleSyncRepeatingTask(plugin, new Runnable() {
 
                        @Override
                        public void run() {
                            if (!cooldown.containsKey(p.getName())) {
                                cooldown.put(p.getName(), delay);
                            }
                            cooldown.put(p.getName(), delay - 1);
                            if (cooldown.get(p.getName()) <= 0) {
                                Bukkit.getScheduler().cancelTask(cooldown.get(p.getName()));
                                cooldown.remove(p.getName());
                            }
                        }
                    }, 0L, 20L);
 
                    if (!cooldown.containsKey(p.getName())) {
                        if (!p.getGameMode().equals(GameMode.CREATIVE))
                            p.getInventory().removeItem(new ItemStack(Material.SULPHUR, 1));
                        else
                            hash.remove(p.getName());
 
                        if (explosionSize == 1) {
                            ...
                            if (explosionSize == 2) {
                                ...
                            }
                        } else {
                            ...
                        }
                    } else
                        ...
                }
            }
        }
    }
}

 

ok so no reason what so ever to make the event final, or the player, act == should be act.equals( pass a final string to the runnable that is the players name instead of making the player final also you should really move the check to see if they have a cooldown to the top of them method, cause right now your runnable is adding them to the cooldown map even if they are already in it

 

I changed that but I still have a problem of it not working correctly

 

updated code and what's not working please?