Save per player configs

Hi all,

So this is my second post for the day (I'll stop soon! ) but I am just making a big plugin and would need some help for that (you guys <3). So I made a function that generates per player yml files, it generates fine and all, but when I try to add something in it and save the file, it is not working.

Code:

Code:

    public static FileConfiguration getPlayerConfig(Player p) {
        FileConfiguration yml = null;
        File file = getPlayerFile(p);
        yml = YamlConfiguration.loadConfiguration(file);
       
        return yml;
    }
   
    public static File getPlayerFile(Player p) {
        return new File(Main.getPlugin().getDataFolder(), "users" + File.separator + p.getUniqueId() + ".yml");
    }
   
    public static void savePlayerConfig(Player p) {
        try {
            getPlayerConfig(p).save(getPlayerFile(p));
            System.out.println("done");
        } catch (IOException e) {
            e.printStackTrace();
        }
    }

Hopefully you guys are seeing a problem

Regards,
Guus Huizen - iAmGuus

 

I can see loads of problems here

 

@Monkey_Swag, how else would I get the methods from other classes?

 

Make it multi-instance - add constructor and in it pass player. (and remove static from functions)

 

Can you show me an example, @mibac138 ?

 

This might help you: http://www.colonelhedgehog.com/tutorials/intermediate/2015/1/1/intermediate-2-custom-configs

 

Basic Java knowledge is a prerequisite to an API written in Java. Static isn't a magic solution to accessing methods across classes, public is.

 

I would say that it is a magic solution to access method from other classes. A dark magic solution. That causes memory leaks and hurts servers.

 

No magic is inherently good or bad. It's how you use it.

 

http://prntscr.com/5zldla
Google says otherwise.

 

Wikipedia is not a valid source!!!

 

Unlike other websites, wikipedia is community oriented and not bias which makes it a great source

 

@Skionz Thank you! Someone that finally agrees with me!

And to answer the OP's post, why don't you just load the file? For example:

Code:

FileConfiguration <file config name, or just whatever you want> = YamlConfiguration.loadConfiguration(<file name, the thing the you made the file with>)

<file config name, the one you used up there>.set(...);

I know its a little messy, but you should just be able to figure it out.

EDIT: Oh also, make sure you save the file every time you use it. You might have to surround it with a try catch.

 

I was quoting every modern teacher ever.

 

So true

 

Totally agree with both of these, wikipedia is oftenly reviewed and fixed, so there is barely any errors