Need help with config EconomyPlugin

Hello,

I am trying to put every new join in the config, but the config gives a error at line 25:

Code:

at me.banjer_hd.survival.factions.Events.PlayerEvents.playerJoinEvent(PlayerEvents.java:25) ~[?:?]
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:1.8.0_66]
        at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source) ~[?:1.8.0_66]
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source) ~[?:1.8.0_66]

My class:

Code:

public class PlayerEvents implements Listener {

    public static Main plugin;

    @EventHandler
    public void playerJoinEvent(PlayerJoinEvent e) {
        Player p = e.getPlayer();
    
        if(existConfig(p.getUniqueId().toString()) == false) {
            p.sendMessage(ChatColor.WHITE + "Welcome!");
>>>>>plugin.getConfig().addDefault("money.players." + p.getUniqueId(), 1000); 
//^ is line 25
            plugin.getConfig().options().copyDefaults(true);
            plugin.saveConfig();
        }
    
    }
    public boolean existConfig(String UUID) {
        try{
            String s = plugin.getConfig().getString("money.players." + UUID);
            if(s.length() == 1) { // this is for the yellow error on s
            
            }
            return true;
            } catch(NullPointerException e){
            return false;
            }
    }

}

Thank you in advance!!

 

@timtower What do you mean?

 

@timtower I am not in the main class... with that line i can use "plugin.getConfig()"

 

@Banjer_HD what have you defined plugin as?

 

@Areoace Why do you ask that? xD
Its called SurvivalFactions

 

Ok I don't think you're getting what I'm saying. You should really get rid of static methods as much as possible and use instances or getters.

You also need to set main, so try make an instance of it. No need for static btw, remove it.

 

You have to set

to be something, it's like a string you can't just do this

and if you print that string it will "Im a string" you have to do it like this

 

@Areoace I have tested this and it works.. So thats not the problem..

 

Static abusing is a very bad habit. @Banjer_HD

 

@Banjer_HD

It most certainly is the problem. It causes a very demonstrable lack of understanding of your own code that results in making mistakes like this.

 

@mythbusterma @Areoace Okay... But what code do i need to do this?

yes it works now Thanks to every1 who helped!
Solution:

Code:

Main plugin;
     
    public PlayerEvents(Main instance) {
    plugin = instance;
    }

Sorry for my bad english i am Dutch

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

 

@Banjer_HD
add 'System.println(plugin);' before the error.