Solved Creating per Player config in new map

Hi,

Im trying to create a .yml file for every player to easier read data.
I tried alot, but nothing seems to be working. If one could help me / give me a good link to a recourse / tutorial that would be great.

What I need:

• Reloading per player .yml
• Reloading entire PlayerData folder (all .yml's)
• Saving per player .yml file
• Create a folder PlayerData in plugin.getDataFolder() [Solved by myself]
• Getting per player .yml file

Current Code

Code:java

import java.io.File;
import java.io.IOException;
 
import org.bukkit.configuration.file.YamlConfiguration;
import org.bukkit.entity.Player;
 
public class PlayerData {
 
File file;
YamlConfiguration playerData;
 
private Strikerz m;
 
public PlayerData(Strikerz instance) {
m = instance;
}
 
public void saveData(Player p) {
try {
playerData.save(file);
} catch (IOException e) {
e.printStackTrace();
}
}
 
@SuppressWarnings("static-access")
public void reloadData(Player p) {
playerData.loadConfiguration(file);
}
 
@SuppressWarnings("static-access")
public void createData(Player p) {
File dir = new File("PlayerData");
if (!dir.exists()) {
System.out.println("Error: Could not find PlayerData folder");
dir.mkdir();
} else {
file = new File(m.getDataFolder()+ "/" + dir + File.separator, p.getUniqueId().toString()
+ ".yml");
try {
file.createNewFile();
} catch (IOException e) {
e.printStackTrace();
}
playerData = new YamlConfiguration();
playerData.loadConfiguration(file);
}
}
 
@SuppressWarnings("static-access")
public YamlConfiguration getPlayerData(Player p) {
if (file == null) {
createData(p);
}
return playerData.loadConfiguration(file);
}
 
}

Obvious error (I understand the error)

Code:

[15:21:07 WARN]: java.io.IOException: The system cannot find the path specified
[15:21:07 WARN]:        at java.io.WinNTFileSystem.createFileExclusively(Native
Method)
[15:21:07 WARN]:        at java.io.File.createNewFile(Unknown Source)
[15:21:07 WARN]:        at me.dubehh.Main.PlayerData.createData(PlayerData.java:
43)
[15:21:07 WARN]:        at me.dubehh.Listeners.PlayerChange.onJoin(PlayerChange.
java:20)
[15:21:07 WARN]:        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native M
ethod)
[15:21:07 WARN]:        at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown S
ource)
[15:21:07 WARN]:        at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unkno
wn Source)
[15:21:07 WARN]:        at java.lang.reflect.Method.invoke(Unknown Source)
[15:21:07 WARN]:        at org.bukkit.plugin.java.JavaPluginLoader$1.execute(Jav
aPluginLoader.java:292)
[15:21:07 WARN]:        at org.bukkit.plugin.RegisteredListener.callEvent(Regist
eredListener.java:62)
[15:21:07 WARN]:        at org.bukkit.plugin.SimplePluginManager.fireEvent(Simpl
ePluginManager.java:501)
[15:21:07 WARN]:        at org.bukkit.plugin.SimplePluginManager.callEvent(Simpl
ePluginManager.java:486)
[15:21:07 WARN]:        at net.minecraft.server.v1_7_R3.PlayerList.c(PlayerList.
java:251)
[15:21:07 WARN]:        at net.minecraft.server.v1_7_R3.PlayerList.a(PlayerList.
java:138)
[15:21:07 WARN]:        at net.minecraft.server.v1_7_R3.LoginListener.c(LoginLis
tener.java:76)
[15:21:07 WARN]:        at net.minecraft.server.v1_7_R3.LoginListener.a(LoginLis
tener.java:42)
[15:21:07 WARN]:        at net.minecraft.server.v1_7_R3.NetworkManager.a(Network
Manager.java:160)
[15:21:07 WARN]:        at net.minecraft.server.v1_7_R3.ServerConnection.c(Sourc
eFile:134)
[15:21:07 WARN]:        at net.minecraft.server.v1_7_R3.MinecraftServer.v(Minecr
aftServer.java:667)
[15:21:07 WARN]:        at net.minecraft.server.v1_7_R3.DedicatedServer.v(Dedica
tedServer.java:260)
[15:21:07 WARN]:        at net.minecraft.server.v1_7_R3.MinecraftServer.u(Minecr
aftServer.java:558)
[15:21:07 WARN]:        at net.minecraft.server.v1_7_R3.MinecraftServer.run(Mine
craftServer.java:469)
[15:21:07 WARN]:        at net.minecraft.server.v1_7_R3.ThreadServerApplication.
run(SourceFile:628)

 

Unica
You have to create all Folders first before creating the File.
Example:
File folder = new File("plugins/folder/players");
folder.mkdirs();
File file = new File("plugins/folder/players/Moewe1.yml");
file.createNewFile();

 

its easy to fix, also why there is static access warning?

 

mrCookieSlime mine-care

Hi, I created the folder + the config.
How would I

• Save the config (for one player only)
• Reload the config (for one player only)
• Get the config (I already have that I believe)

 

Unica
config.save(file);
just load a new config Object from that File
? FileConfiguration config = YamlConfiguration.loadConfiguration(file);

 

Hey,
I tried that. I made this, and now it doesn't even make the folder anymore, what am I doing wrong?

Code:java

 
public class PlayerData {
 
    YamlConfiguration playerData;
 
    private Strikerz m;
 
    public PlayerData(Strikerz instance) {
        m = instance;
    }
 
    public void saveData(Player p) {
        try {
            playerData.save(getPlayerFile(p));
        } catch (IOException e) {
            e.printStackTrace();
        }
    }
 
    @SuppressWarnings("static-access")
    public void reloadData(Player p) {
        playerData.loadConfiguration(getPlayerFile(p));
        saveData(p);
    }
 
    @SuppressWarnings("static-access")
    private void createData(Player p) {
        File dir = new File(m.getDataFolder().getAbsolutePath() + File.separator + "PlayerData");
        if (!dir.exists()) {
            System.out.println("Error: Could not find PlayerData folder");
            System.out.println("Creating one for you.");
            dir.mkdir();
            createData(p);
        } else {
            File file = new File(dir, p.getUniqueId().toString()
                    + ".yml");
            try {
                file.createNewFile();
            } catch (IOException e) {
                e.printStackTrace();
            }
            playerData = new YamlConfiguration();
            playerData.loadConfiguration(file);
            playerData.set("Exp", 0);
            playerData.set("Kills", 0);
            playerData.set("Deaths", 0);
            playerData.set("CruxHits", 0);
            try {
                playerData.save(file);
            } catch (IOException e) {
                e.printStackTrace();
            }
        }
    }
 
    @SuppressWarnings("static-access")
    public YamlConfiguration getPlayerData(Player p) {
        if (getPlayerFile(p).exists()) {
            createData(p);
        }
        return playerData.loadConfiguration(getPlayerFile(p));
    }
 
    private File getPlayerFile(Player p){
        File folder = new File(m.getDataFolder().getAbsolutePath() + File.separator + "PlayerData");
        return new File(folder, p.getUniqueId().toString());
    }
 
}

mrCookieSlime

 

To reload a file you need to reassign the variable
File file = new File(dir, "playername.yml");

 

Unica
As I and 4thegame3 said. Recreate the Objects

 

ups i said variable D:

 

4thegame3
Ups I said Object :O

It is a Variable after all...
Actually, an Object is something else, I just dont know why I always call Variables Objects...

 

mrCookieSlime 4thegame3

I did that right?

Code:java

return new File(folder, p.getUniqueId().toString());
 

So I returned a 'new file'
and I reloaded that?

 

Unica
First of all you probably want it like p.getUniqueId().toString + ".yml" or something along those lines...
Then just create a brand new Config Object of of it.

 

mrCookieSlime

What do you mean with new Config object ?
I have to return a file right?
Could u give me a pseudo code?

 

Unica
File file = new File(path);
FileConfiguration config = YamlConfiguration.loadConfiguration(file);

Use that on reloading the Config, also what do you need a reloading method for anyway??

 

mrCookieSlime

Okay, the folder is creating now.
Now I need to know how to

• Save
• Reload
• Get

The configuration

EDIT:
Ninja'd, let me test

 

Unica
0_0 I already posted you how to save/get/reload the Config before...

Save:

Reload:

Get:

 

mrCookieSlime

Code:java

public class PlayerData {
 
    private Strikerz m;
 
    public PlayerData(Strikerz instance) {
        m = instance;
    }
 
    public void saveData(Player p) {
        File file = new File(getDirectory(), p.getUniqueId().toString()+".yml");
        YamlConfiguration config = YamlConfiguration.loadConfiguration(file);
        try {
            config.save(file);
        } catch (IOException e) {
            e.printStackTrace();
        }
    }
 
    @SuppressWarnings("static-access")
    private void createData(Player p) {
        File file = new File(getDirectory(), p.getUniqueId().toString() + ".yml");
        try {
            file.createNewFile();
        } catch (IOException e) {
            e.printStackTrace();
        }
        YamlConfiguration playerData = new YamlConfiguration();
        playerData.loadConfiguration(file);
        playerData.set("Exp", 0);
        playerData.set("Kills", 0);
        playerData.set("Deaths", 0);
        playerData.set("CruxHits", 0);
        try {
            playerData.save(file);
        } catch (IOException e) {
            e.printStackTrace();
        }
    }
 
    public YamlConfiguration getPlayerData(Player p) {
        File file = new File(getDirectory(), p.getUniqueId().toString()+".yml");
        if(!file.exists()){
            createData(p);
        }
        YamlConfiguration config = YamlConfiguration.loadConfiguration(file);
        return config;
    }
 
    private File getDirectory(){
        File dir = new File(m.getDataFolder().getAbsolutePath() + File.separator + "PlayerData");
        if(!dir.exists()){
            dir.mkdir();
        }
        return dir;
    }
 
}
 

What am I missing??

 

Unica
A Lot...

First of all your saveData() method is completely useless...
Your basically getting the data from a file and saving the exact same data to that file again ?.?

But createData() and getPlayerData() are just fine. Only saveData() is useless...

 

mrCookieSlime

Code:java

public void saveData(Player p) {
File file = new File(getDirectory(), p.getUniqueId().toString()+".yml");
try {
getPlayerData(p).save(file);
} catch (IOException e) {
e.printStackTrace();
}
}

Im really confused. Sorry if I am being 'dumb', but I've never used this kind of config.
What is the correct form?

 

Unica
You do not need such a saveData() method at all. Those are the Methods you probably want to have:

createData(Player p);
getData(Player p);
setValue(Player p, String key, Object value);
getValue(Player p, String key);
and maybe:
containsValue(Player p, String key);

reloading/saving is just useless. Just save it whenever you change something.

 

mrCookieSlime
The saving doesn't even work.

Code:java

m.player_data.getPlayerData(p).set("Test", "Test");
YamlConfiguration.loadConfiguration(m.player_data.getPlayerFile(p));

Doesn't do anything.

Code:java

public File getPlayerFile(Player p){
return new File(getDirectory(), p.getUniqueId().toString()+".yml"); //Return new File(blabla)   
    }

Why doesn't this save? "reloading/saving is just useless. Just save it whenever you change something."
That's what I am trying to do. For setting values I need to save too

 

Unica
You didn't read what I wrote...

You probably want a setValue() method which includes saving...

EDIT: Here's a Config Class which I'm using for my Plugins:

Code:java

public class Config {
 
    File file;
    FileConfiguration config;
 
    public Config(File configFile) {
        this.file = configFile;
        this.config = YamlConfiguration.loadConfiguration(this.file);
    }
 
    public File getFile() {
        return this.file;
    }
 
    public FileConfiguration getConfiguration() {
        return this.config;
    }
 
    public void setValue(String path, Object value) {
        config.set(path, value);
        try {
            config.save(file);
        } catch (IOException e) {
        }
    }
 
    public boolean setDefaultValue(String path, Object value) {
        if (!contains(path)) {
            config.set(path, value);
            try {
                config.save(file);
                return true;
            } catch (IOException e) {
                return false;
            }
        }
        else {
            return false;
        }
    }
 
    public boolean contains(String path) {
        return config.contains(path);
    }
 
    public Object getValue(String path) {
        return config.get(path);
    }
 
    public ItemStack getItem(String path) {
        return config.getItemStack(path);
    }
 
    public String getRandomStringfromList(String path) {
        return getStringList(path).get(new Random().nextInt(getStringList(path).size()));
    }
 
    public int getRandomIntfromList(String path) {
        return getIntList(path).get(new Random().nextInt(getIntList(path).size()));
    }
 
    public String getString(String path) {
        return config.getString(path);
    }
 
    public int getInt(String path) {
        return config.getInt(path);
    }
 
    public boolean getBoolean(String path) {
        return config.getBoolean(path);
    }
 
    public List<String> getStringList(String path) {
        return config.getStringList(path);
    }
 
    public List<Integer> getIntList(String path) {
        List<Integer> ints = new ArrayList<Integer>();
        for (String string: config.getStringList(path)) {
            ints.add(Integer.parseInt(string));
        }
        return ints;
    }
 
    public void createFile() {
        try {
            this.file.createNewFile();
        } catch (IOException e) {
        }
    }
 
    public Location getLocation(String path) {
        return new Location(
            Bukkit.getWorld(
            config.getString(path + ".world")),
            config.getDouble(path + ".x"),
            config.getDouble(path + ".y"),
            config.getDouble(path + ".z")
        );
    }
 
}

just do Config cfg = new Config(file);
cfg.setValue("Test", "Test");

It will automatically save it for you.

 

mrCookieSlime

I did read what u wrote, my point is. I have to save the data anyway..

Code:java

public void setValue(Player p, String path, Object o){
        getPlayerData(p).set(path, o); //Set values
        File file = new File(getDirectory(), p.getUniqueId().toString()+".yml"); //Get the file from the directory
        try {
            getPlayerData(p).save(file); //save the file
        } catch (IOException e) {
            e.printStackTrace();
        }
 
    }

The file doesn't add up.
If you say the getter is right, than
getPlayerData(p) should return the config, and then I could save the file right?

 

Unica
Correct. Your setValue method should work.

 

mrCookieSlime

Doesn't work.
Nothing get's saved.

mrCookieSlime

I got it to working.
YamlConfiguration#loadConfiguration was needed in order to save everything.
Thanks for your help

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