HashMap Help - Not reading Hashmap Integer

Hello, im new to HashMaps, so what i want, is so that when a player kills 5+ bats, they get a message saying "Well Done, 5+ Bat Deaths!!!", and if they don't have +5 , do <5, they get a message saying "Still Under 5 Bats". Here is the code:

Code:java

package com.arkolly.tester;
 
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.entity.Bat;
 
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.entity.EntityDeathEvent;
import org.bukkit.plugin.java.JavaPlugin;
 
 
import java.util.HashMap;
 
public class main extends JavaPlugin implements Listener {
 
 
 
    @Override
    public void onEnable() {
        Bukkit.getPluginManager().registerEvents(this, this);
    }
 
 
    private HashMap<String, Integer> bats = new HashMap<String, Integer>();
 
    @EventHandler
    public void onbatkill(EntityDeathEvent event){
        Player player = event.getEntity().getKiller();
 
        int batkills = 1;
 
 
            if (event.getEntity() instanceof Bat){
 
            if(bats.get(batkills) < 5){
                batkills++;
                bats.put(player.getName(), batkills);
                player.sendMessage(ChatColor.RED + "Still Under 5 Bats");
 
            }
            else if(bats.get(batkills) >=5){
                batkills++;
                bats.put(player.getName(), batkills);
                player.sendMessage("Well Done, 5+ Bat Deaths!!!");
            }
 
            }
 
 
        }
 
 
 
    }
 

I know that the hashmap data will be wiped when the server restarts Thanks!! HELP ASAP would be nice!

 

Moved to plugin dev.

 

Assuming you are fine with map data being wiped on server restart, I think this code will work.

Code:java

package com.arkolly.tester;
 
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.entity.Bat;
 
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.entity.EntityDeathEvent;
import org.bukkit.plugin.java.JavaPlugin;
 
 
import java.util.HashMap;
 
public class Main extends JavaPlugin implements Listener {
 
 
@Override
public void onEnable() {
Bukkit.getPluginManager().registerEvents(this, this);
}
 
 
private HashMap<String, Integer> bats = new HashMap<String, Integer>();
 
@EventHandler
public void onbatkill(EntityDeathEvent event){
Player player = event.getEntity().getKiller();
 
if(player == null){
return;
}
 
int batkills = bats.containsKey(player.getName()) ? 0 : bats.get(player.getName()); // If the player already has bat kills, load them, otherwise they have none
 
 
if (event.getEntity() instanceof Bat){
 
bats.put(player.getName(), ++batkills); // Increment bat kills by one
 
if(batkills < 5){
player.sendMessage(ChatColor.RED + "Still Under 5 Bats");
 
}
else{
player.sendMessage("Well Done, 5+ Bat Deaths!!!");
 
// Remove the following line of code if you do NOT want to reset bat kills when players get 5
bats.remove(player.getName());
}
 
}
 
 
}
 
 
 
}

As a reminder, assuming you leave the code above unmodified, I believe it will work (not tested) given the following items:

• After a player gets 5 batkills, their batkills will be reset (I showed in code how to change that).
• The message showing their bat kill status, even if you remove the line of code mentioned above, will be shown after every kat kill.
• Bat kills are not persisted across server reloads.
• This system is not guaranteed to work with name changes, as it stores the String names and not the player UUIDs.