Solved Creating a File in the Plugin's Datafolder

The plugin:

Code:java

public <main class> plugin;

The constructor:

Code:java

public thisClass(<main class> plugin)
{
    this.plugin = plugin;
}

The rest would be using plugin.getConfig(), plugin.saveConfig(), etc.

 

No, this is a custom file, I'm not using the Bukkit API.

 

though you are using config.yml ;3

 

I wasn't meaning to =3
Do you know how I can fix the NPE though?

 

The File object:

Code:java

private File someFile;

The constructor:

Code:java

public thisClass(<main class> plugin)
{
    someFile = new File(plugin.getDataFolder(), "file.yml");
}

reloadConfig method:

Code:java

public void reloadFile() 
{
    if (!someFile.exists()) 
    { 
        try
        {
            someFile.mkdirs();
            if(!someFile.createNewFile())
            {
                throw new IOException("Failed to create file");
            }
        }
        catch(IOException ex)
        {
            //log.severe("Failed to create file")
            return;
        }
    }
    fileConfiguration = YamlConfiguration.loadConfiguration(someFile);
 
    InputStream defConfigStream = plugin.getResource("file.yml");
    if (defConfigStream != null) 
    {
        YamlConfiguration defConfig = YamlConfiguration.loadConfiguration(defConfigStream);
        fileConfiguration.setDefaults(defConfig);
    }
}

Now keep your methods off the static keyword >:|

 

fireblast709
First of all, I like static
Second, now I'm getting an NPE on the the seventh-last line in what you just wrote, and my file.yml doesn't exist even though at some point I'm calling .createNewFile() on thisFile =/

 

post the stacktrace and current code :3. Also, static is not for convenience, can most of the time be avoided easily, and in your case its very erronous

 

fireblast709

Current Code - http://pastebin.com/zFUXqq7L (NPE on line 14)
Stacktrace - http://pastebin.com/GSem7jH2
By the way, do you have a skype or something? It would probably be easier and faster than on this thread =/

 

You should not hard code the folder it is in. your data folder may not reside in the directory named 'plugins'

EDIT: fix spelling

 

Oh yeah... Odd how there's a folder called config.yml in my server directory... xD

 

wait wait wait... why does the error tell he the JavaPlugin's classloader is null... You ain't calling the method before onEnable() right?

 

I'm calling the method in the onEnable()

Anyways, I messed around with the code some more, and now the file.yml won't generate as a directory or a file, anywhere =/

EDIT: AND there's no stack trace in the config :O

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

 

fire i just realised it actually would because getDataFolder(); still returns an unmade file even if its not there aka the path is in the file variable it just doesnt exist!
im pretty sure thats the case anyway.
but if it was on the plugin variable then no it wouldnt.

 

fireblast709
tommycake50
Just realized something- when I delete the kEssentials folder, it doesn't regenerate when I reload the server :O

EDIT: mkdirs is definitely being called on the folder...
EDIT2: Does anyone have skype? It would really help =/

 

KeybordPiano459 PM'ed you my skype (or pm'ing atm)

 

dont use mkdirs use mkdir btw ;P
oh and getDataFolder() has the path already so use getDataFolder.mkDir();
rather than fussing around with instances.

 

Explain to me, as afaik even if the file at the path does not exist, the File object is not null. The NPE occurs as you try to invoke the .mkdirs

[edit] nvm you found out yourself

 

well if its null explain to me how this method works in one of my plugins.

Show Spoiler Hide Spoiler

Code:java

private void listFileChecks() {
if(!getDataFolder().exists()){
getDataFolder().mkdir();
File fl = new File(getDataFolder(), "logfile.log");
if(fl.exists()){
log = fl;
}else{
try {
fl.createNewFile();
log = fl;
} catch (IOException e) {
e.printStackTrace();
}
}
}else{
File fl = new File(getDataFolder(), "logfile.log");
if(fl.exists()){
log = fl;
}else{
try {
fl.createNewFile();
log = fl;
} catch (IOException e) {
e.printStackTrace();
}
}
}
}

i would love to know.

 

tommycake50
Replace any if (file == null) with if (!file.exists()). fireblast709 and I got it to work that way

 

no no i dont have a problem with that method i was just saying if its null how come it works.