Solved Cooldown

Hi, I have made a plugin which adds magic to the world. Every spell needs a cooldown which I have added to the plugin but the cooldown somehow is for every spell in de class. When I use the spell 'noctem' which has a cooldown for 10 minutes or so. All my other spells have a 10min cooldown. How can I fix that?

Code:java

package me.JonathanNLD.Magic;
 
import java.util.ArrayList;
import java.util.List;
 
import org.bukkit.Effect;
import org.bukkit.Material;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.AsyncPlayerChatEvent;
import org.bukkit.scheduler.BukkitRunnable;
 
public class MagicListener implements Listener {
private List<String> cantDoCommand = new ArrayList<>();
 
Main plugin;
public MagicListener(Main plugin){
this.plugin = plugin;
}
 
    @EventHandler
    public void onInteract(AsyncPlayerChatEvent event){
        //AsyncPlayerChatEvent is used for chat commands
        Player player = event.getPlayer();
        final String name = player.getName();
        //This 'gets' the player
        String message = event.getMessage();
        //This replaces event.getMessage with 'message'
        if (message.trim().equalsIgnoreCase("fulmen")){
            if (cantDoCommand.contains(name)){
                event.setCancelled(true);
                return;
            }
            //The chat command + trims the command down to only 'fulmen'
            player.getWorld().strikeLightning(player.getTargetBlock(null, 100).getLocation());
            cantDoCommand.add(name);
            //This activates the lightning
            new BukkitRunnable(){
                @Override
                public void run(){
                    cantDoCommand.remove(name);
                }
            }.runTaskLater(plugin, 60);
        }
    }
 
  @EventHandler
    public void onInteract1(final AsyncPlayerChatEvent event){
 
        if(event.getPlayer().getItemInHand().getType()==Material.BOOK){           
        Player player = event.getPlayer();
        event.getPlayer().getWorld().playEffect(event.getPlayer().getLocation(), Effect.SMOKE, 1);
        final String name = player.getName();
        final String message = event.getMessage();
        if (message.trim().equalsIgnoreCase("Noctem")){
            if (cantDoCommand.contains(name)){
                event.setCancelled(true);
 
                return;
            }
            event.getPlayer().getWorld().setTime(12550);
            cantDoCommand.add(name);
            new BukkitRunnable(){
                @Override
                public void run(){
                    cantDoCommand.remove(name);
                }
            }.runTaskLater(plugin, 12000);
 
        }
 
      }
  }
 
}

Nevermind I got it, I needed 2 seperate lists obivously.

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

 

You need to use a timer per spell per player.
So I guess using a Hashmap with the players name as key and an ArrayList as Value could work.
The arraylist for each player could hold the names of the spells currently on cooldown.
So you only need to check if that spell is on the arraylist to decide if the player may use this spell.

When issuing a spell command, check if the players name is already on the hashmap and if so,
Check if there is an entry for that spell in the corresponding arraylist.
If so, deny the spell usage.
If not, allow the usage.

You have to add the spells name when a spell is cast and delete it when the cooldown timer for this spell is done. (In the run() method of the timer)