My Hashmap isn't saving to the config

Here is my saving and loading of the hashmap, if you need more, just let me know.

My Code:

Code:java

public void onEnable(){
 
        //Config Stuff...
 
        this.saveDefaultConfig();
        this.reloadConfig();
 
 
        // Load Hashmap
        if(this.getConfig().contains("hashmap.data-storage")){
            this.getConfig().getConfigurationSection("hashmap.data-storage").getValues(false);
        }else{
            this.getConfig().createSection("hashmap.data-storage");
        }
 
 
    }
 
    public void onDisable(){
        // Save Hashmap
        this.getConfig().createSection("hashmap.data-storage", hashmap);
        this.saveConfig();
 
        //Logging Stuff
        Bukkit.getConsoleSender().sendMessage(ChatColor.RED + "[eTokens]" + " " + "Configs all Saved :)");
    }
 
    //TODO: Redeem Enderpearl
 
 
 
 
 
    @EventHandler
    public void onJoin(PlayerJoinEvent event){
    Player MCplayer = event.getPlayer();
    String player = MCplayer.getName();
    boolean playerLoggedBefore = hashmap.containsKey(player);
    if(playerLoggedBefore == true){
        //Player has logged in before
    }else if(playerLoggedBefore == false){
        //Player has logged in for first time!
        hashmap.put(player,0);
    }
    }
 
    //TODO: New enchant command
 
 
 

My config, after reloading, stopping, starting etc.

Code:

#
# eTokens Config  #
hashmap:
  data-storage: {}

Bear in mind, I had invoked the save.

Quick Bump

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

 

MasterDoctor You can only bump within 24 hours

 

MasterDoctor I cant tell you your error, but you could clean up your code
First of all, save getConfig() in a private variable and access it through it, because it is quiet ugly to call a listener again and again... second, you should store the players uuid in the hashmap instead of his name, because since version dont.Ask.Me players can change their name. not throught the client but bukkit can do i think. the uuid is the id which is used to save the data of the player into his file etc...
can you show me the code you are initializing the hashmap?

 

Code:java

hashmap.put(player, 1)

 

MasterDoctor this is how you put content in your map, but somewhere you must initialize it like HashMap<String, Integer> hashmap = new HashMap<String, Integer>(); or something like this. I want to see this because it could be that you're saveing an empty hashmap in your config because your pointer "hashmap" is not pointing on the object you want to
ps thag me or i may wont respond to this thread, this time was pure luck because i was afk for a few minutes and come back right now

 

Shmobi

Code:java

HashMap<Player, Integer> hashmap = new HashMap<Player, Integer>();

 

MasterDoctor The way I made a plugin similar to this didn't require hashmaps. Also, someone correct me if I'm wrong but your 'playerloggedbefore' variable wouldn't work, because after each reload/restart, the hashmap would clear. Anyways, just check if the config contains the player's uuid, if not, create a section in the config with this player's data. After that, just get the int from the config whenever you want to add/remove tokens. No hashmaps needed (You could do it with hashmaps but I find my way easier )

 

MasterDoctor Based on your code, I'm assuming you're pretty new to Java. I'd recommend learning more Java before trying to make plugins, otherwise you just run into problems. I recommend the Oracle tutorials or a Java book

Shmobi Just because you'd rather see fields that method calls, doesn't mean they're ugly. Stop trying to assert your personal preference as fact.

 

MasterDoctor
You realise you can just do:
if(player.hasPlayedBefore()){}

right?

 

Indeed.
MasterDoctor also, storing Player objects in maps/lists of any kind is bad practice. You should use the name (or as of recently, UUID) if you're planning on storing them for any purpose. In this situation though, you could just do what CraftCreeper6 suggested.
https://forums.bukkit.org/threads/common-mistakes.100544/

 

Abs0rbed how exactly is it bad practice? Please elaborate.

 

Abs0rbed I wouldn't call it bad practice. It's more of it can cause memory leaks if you don't remove them.

 

CraftCreeper6 Yeah, I guess you're right about that, though it's not great if you don't remove them and leaks are caused xD
In this case it completely ruins the purpose, because you'd be removing the player object from that hashmap when they quit (or some other time), and then you can't use it for checking to see if they've played before because they're not in the list.

 

Abs0rbed
player.getName(); || player.getUniqueId();

The HashMap would be:
private HashMap<String, Other thingy> = new...
Or:
private HashMap<UUID, Other thingy> = new...

 

That's what I'm trying to say. It's better to do one of those than storing the Player object, because you'll have to remove it to prevent the leak, rendering saving the player object to see if they've played before useless

 

Abs0rbed if you forget to remove them and cause memory leaks... Well that's 100% your issue . And my point was completely against the bad practice part. Of course it wouldn't be practical at all if you only want to track if an UUID has been online before, it's more a small performance boost when you need the Player object when iterating over the said collection.

 

fireblast709
Still only need one extra line of code

 

fireblast709 Point taken, I suppose , though I would still end up using hasPlayedBefore() in this case xD

 

CraftCreeper6 yes but that's still extra code to execute, and extra code to execute means it consumes more time of a tick .

 

fireblast709
Fair enough, fair enough.

 

Yea you're right. But at least you can't say it wouldn't be nicer. Sure it works both ways and it is readable. But at the timepoint he want to access the array (not in this case), he need to do this. i just wanted to say, safeing objects in variables even if it is not needed can prevent from possible errors in the future and can make less work

Never store objects like a full Player in a hashmap. Because if the player leaves the server the object is not deleted, because you're keep pointing at it in your hashmap. you should store the UUID of the player. This id is used to load and safe data of a player and is generated only 1 time per player. use HashMap<UUID, Integer> and get a Player by using Bukkit.getPlayer(uuidOfPlayer)

 

I can indeed say that and will say it now: config.whatever() does not look any nicer than getConfig().whatever()... Again, you may thing the former looks better, but it's personal opinion. I believe the latter looks nicer, and I can be sure (bar overriding) it's the default config. Your opinion is not a fact.

 

AdamQpzm if you really wan't a discussion with good reasons you can have it by contacting me per pm. But i won't keep talking about something this thread isn't about. You're free to contact me whenever you want

 

Shmobi There's no need for further contact. If someone wants to argue that a variable looks better than a method like getConfig() as a fact rather than a personal opinion, I don't know what more I could possible say to explain that it's a personal preference not a fact.

 

AdamQpzm believe me, there is for sure a reason why but it seems like you're no developer who has to watch something like this in his code. i'm working as a developer and i know why i say that there is a reason why you should do it. But i won't start a discussion about it here. if you are interested then contact me, otherwise stay with your opinion