Help with Hashmaps

Hello, i am making a spells plugin, and i'm saving the players current spell to a hashmap, and i was wondering if anyone could help me save the players current spell to the hashmap, i tried making one, but it didn't work

Here's my main class

Code:java

package me.XGravity_GuyX.main;
 
import java.util.HashMap;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.Action;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Main extends JavaPlugin implements Listener{
 
 
    public void onEnable(){
 
        Bukkit.getServer().getPluginManager().registerEvents(this, this);
 
    }
    static int spellCount = Spell.getCount();
 
    static int currentSpell = 1;
 
    public static void main(String[] args) {
 
        for (int i = 0; i < 10; i++) {
            switchSpell();
        }
    }
 
    public static void switchSpell(Player p) {
        currentSpell = currentSpell == spellCount ? 1 : currentSpell + 1;
        Spell s = Spell.getSpell(currentSpell);
        if (s == null)
            return;
        System.out.println("[DH-Spells] Switched to: " + s.getName() + "!");
        HashMap<Player, ?> map = new HashMap<Player, ?>();
        map.get
    }
 
    @EventHandler
    public void onPlayerInteract(PlayerInteractEvent e) {
        Player p = (Player) e.getPlayer();
        if (e.getAction() == Action.RIGHT_CLICK_AIR) {
            if (p.getItemInHand() != null) {
                if (p.getItemInHand().getType() == Material.STICK) {
 
                    switchspell(p);
 
                }
 
 
 
            }
        }
    }
}
 

and here is my spell class

Code:java

package me.XGravity_GuyX.main;
 
import org.bukkit.ChatColor;
 
public enum Spell {
 
    AVADAKEDAVRA(1, ChatColor.ITALIC + "Avada Kedavra"), BOMBARDA(2, ChatColor.ITALIC +  "Bombarda"), BOMBARDAMAXIMA(3, ChatColor.ITALIC +  "Bombarda Maxima"), CONFRINGO(4, ChatColor.ITALIC +  "Confringo"), CONJUNCTIVITIS(5, ChatColor.ITALIC +  "Conjunctivitis"), DIPULSO(6, ChatColor.ITALIC +  "Dipulso"), FLIPENDO(7, ChatColor.ITALIC +  "Flipendo");
    int id;
    String name;
    Spell(int id, String name) {this.id = id; this.name = name;}
    public int getId() {return id;}
    public static int getCount() {return Spell.values().length;}
    public String getName() {return name;}
    public static Spell getSpell(int id) {
        for (Spell s : Spell.values()) {
            if (s.getId()==id) {
                return s;
            }
        }
        return null;
 
    }
 
}
 

if anyone could help me out here, it would be much appreciated

 

Violence010

In the case that you have some piece of information that you need to attach to a player, it's best to make your own custom player class that contains a org.bukkit.entity.Player and his/her custom properties. This is especially important when you plan to expand your plugin and add more data to your custom player class. It will become very burdensome to have to create a Map for every property of each player you add to your program.

In which case:

Code:java

public class CustomPlayer{
 
    private final Player player;
 
    private Spell currentSpell;
 
    public CustomPlayer(Player player) {
 
        this.player = player;
 
        this.currentSpell = null;
    }
 
    public boolean hasSpell() {
 
        return this.currentSpell != null;
    }
 
    public Spell getCurrentSpell() {
 
        return this.currentSpell;
    }
 
    public void setSpell(Spell spell) {
 
        this.currentSpell = spell;
    }
 
    public Player getPlayer() {
 
        return this.player;
    }
 
    @Override
    public boolean equals(Object object) {
 
        if(object instanceof CustomPlayer) {
 
            return ((CustomPlayer) object).getPlayer().equals(this.getPlayer());
        }
 
        return false;
    }
}

 

Violence010

Make a public HashMap and use put(key, value) to add an item, get(key) to get the value with key as key.
Also you should store the players name instead of the player itself to save memory.

Code:java

package me.XGravity_GuyX.main;
 
import java.util.HashMap;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.Action;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Main extends JavaPlugin implements Listener{
 
 
    public void onEnable(){
 
        Bukkit.getServer().getPluginManager().registerEvents(this, this);
 
    }
    static int spellCount = Spell.getCount();
 
    static int currentSpell = 1;
 
    public static void main(String[] args) {
 
        for (int i = 0; i < 10; i++) {
            switchSpell();
        }
    }
 public HashMap<String, Objekt> map = new HashMap<String, Object>();
    public static void switchSpell(Player p) {
        currentSpell = currentSpell == spellCount ? 1 : currentSpell + 1;
        Spell s = Spell.getSpell(currentSpell);
        if (s == null)
            return;
        System.out.println("[DH-Spells] Switched to: " + s.getName() + "!");
 
        map.getValue(keygen);
        map.put(p.getName(), Object);
    }
 
    @EventHandler
    public void onPlayerInteract(PlayerInteractEvent e) {
        Player p = (Player) e.getPlayer();
        if (e.getAction() == Action.RIGHT_CLICK_AIR) {
            if (p.getItemInHand() != null) {
                if (p.getItemInHand().getType() == Material.STICK) {
 
                    switchspell(p);
 
                }
 
 
 
            }
        }
    }
}