Null Pointer Exception Please help

I am trying to edit a config file to add a player to it but it keeps calling a null pointer exception.

main class:

Code:java

package boardinggamer.Ecahmoney;
 
import java.io.File;
 
import org.bukkit.configuration.Configuration;
import org.bukkit.configuration.file.FileConfiguration;
import org.bukkit.configuration.file.YamlConfiguration;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerJoinEvent;
import org.bukkit.plugin.PluginManager;
import org.bukkit.plugin.java.JavaPlugin;
 
public class ecahmoney extends JavaPlugin implements Listener{
 
MoneyAPI money = MoneyAPI.getInstance();
 
public void onEnable() {
 
PluginManager pm = getServer().getPluginManager();
pm.registerEvents(this, this);
 
createConfigs();
Configuration config = getConfig();
config.addDefault("Money.Name", "Dollor");
config.addDefault("Money.Starting Amount", 25.00);
config.addDefault("Bank.Creation Cost", 10.00);
config.addDefault("Bank.Intrest Rate", 2.50);
File players = new File(getDataFolder(), "players.yml");
FileConfiguration playersConf = YamlConfiguration.loadConfiguration(players);
playersConf.addDefault("Playername.Money", 0.0);
File banks = new File(getDataFolder(), "banks.yml");
FileConfiguration banksConf = YamlConfiguration.loadConfiguration(banks);
banksConf.addDefault("Banksname.Money", 0.0);
banksConf.addDefault("Banksname.Creater", "Playername");
banksConf.addDefault("Banksname.Members", new String[] { "Playername" });
try{
playersConf.options().copyDefaults(true);
playersConf.save(players);
banksConf.options().copyDefaults(true);
banksConf.save(banks);
config.options().copyDefaults(true);
saveConfig();
} catch (Exception e) {}
}
 
public void onDisable() {
 
}
 
public void createConfigs(){
File players = new File(getDataFolder(), "players.yml");
if(players.exists() == false) {
saveResource("players.yml", false);
}
File banks = new File(getDataFolder(), "banks.yml");
if(banks.exists() == false) {
saveResource("banks.yml", false);
}
}
 
@EventHandler
public void joinEvent(PlayerJoinEvent event) {
String name = event.getPlayer().getName();
money.addPlayerToConfig(name);
}
 
}
 

MoneyAPI cass:

Code:java

package boardinggamer.Ecahmoney;
 
import java.io.File;
 
import org.bukkit.configuration.file.FileConfiguration;
import org.bukkit.configuration.file.YamlConfiguration;
 
public class MoneyAPI {
 
private ecahmoney plugin;
private static MoneyAPI singleton;
 
public MoneyAPI(ecahmoney plugin){
this.plugin = plugin;
singleton = this;
}
 
public static final MoneyAPI getInstance(){
return singleton;
}
 
public boolean playerInMoneyConfig(String name){
boolean in = false;
System.out.println("start false");
File players = new File(this.plugin.getDataFolder(), "players.yml");
System.out.println("got file");
if (players.exists()){
System.out.println("file exists");
FileConfiguration pconf = YamlConfiguration.loadConfiguration(players);
 
System.out.println("got conf");
if (pconf.contains(name + ".Money")){
 
System.out.println("not in file");
in = true;
System.out.println("can be added");
}
}
return in;
}
 
public void addPlayerToConfig(String name){
File players = new File(this.plugin.getDataFolder(), "players.yml");
FileConfiguration pconf = YamlConfiguration.loadConfiguration(players);
double start = this.plugin.getConfig().getDouble("Money.Starting Amount");
if (playerInMoneyConfig(name) == false){
pconf.addDefault(name + ".Money", start);
try{
pconf.options().copyDefaults(true);
pconf.save(players);
} catch (Exception e) {}
}
}
 
}
 

Error

Code:bash

c2:05:46 [INFO] creeperzgod[/192.168.1.1:52890] logged in with entity id 8822 at ([world] 255.97760183358474, 66.0, 214.5243049141394)
12:05:46 [SEVERE] Could not pass event PlayerJoinEvent to Ecahmoney v1.0
org.bukkit.event.EventException
at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:341)
at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:62)
at org.bukkit.plugin.SimplePluginManager.fireEvent(SimplePluginManager.java:477)
at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:462)
at net.minecraft.server.ServerConfigurationManagerAbstract.c(ServerConfigurationManagerAbstract.java:153)
at net.minecraft.server.ServerConfigurationManagerAbstract.a(ServerConfigurationManagerAbstract.java:93)
at net.minecraft.server.NetLoginHandler.d(NetLoginHandler.java:132)
at net.minecraft.server.NetLoginHandler.c(NetLoginHandler.java:45)
at net.minecraft.server.DedicatedServerConnectionThread.a(DedicatedServerConnectionThread.java:44)
at net.minecraft.server.DedicatedServerConnection.b(SourceFile:29)
at net.minecraft.server.MinecraftServer.r(MinecraftServer.java:595)
at net.minecraft.server.DedicatedServer.r(DedicatedServer.java:222)
at net.minecraft.server.MinecraftServer.q(MinecraftServer.java:493)
at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:426)
at net.minecraft.server.ThreadServerApplication.run(SourceFile:856)
Caused by: java.lang.NullPointerException
at boardinggamer.Ecahmoney.ecahmoney.joinEvent(ecahmoney.java:65)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:616)
at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:339)
... 14 more
 

 

money is null. You might create a MoneyAPI instance before you call getInstance()

 

Your code makes no sense. A singleton is a class that there can only be one of and this is done by making the constructor private so that you can only access it from within the class. What you should do is either make your constructor private and then do this: MoneyAPI singleton = new MoneyAPI(); in your moneyAPI class or don't make the constructor private and then do this: money = new MoneyAPI(this); in your onEnable() method and then get rid of the MoneyAPI variable in your MoneyAPI class and get rid of the getInstance() method in that class as well. Either one of those should fix your problem but I would suggest the second one so that you don't have to mess with making a variable of your main class since it looks like you would need one in your MoneyAPI class

 

God damn it I forgot to call the MoneyAPI in the onEnable() >_>

Ok I will fix that when I get home, thanks.