Solved NPE w/ Stacktrace

Hi there,

Im getting this NullPointerException, which I don't know the answer to.
Stack

Code:

[14:45:41 ERROR]: Error occurred while enabling NumberOne v0.1 (Is it up to date?)
java.lang.NullPointerException
        at me.dubehh.Locations.SpawnLocations.<init>(SpawnLocations.java:18) ~[?:?]
        at me.dubehh.Main.NumberOne.load(NumberOne.java:34) ~[?:?]
        at me.dubehh.Main.NumberOne.onEnable(NumberOne.java:27) ~[?:?]
        at org.bukkit.plugin.java.JavaPlugin.setEnabled(JavaPlugin.java:250) ~[craftbukkit.jar:git-B
ukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
        at org.bukkit.plugin.java.JavaPluginLoader.enablePlugin(JavaPluginLoader.java:350) [craftbuk
kit.jar:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
        at org.bukkit.plugin.SimplePluginManager.enablePlugin(SimplePluginManager.java:384) [craftbu
kkit.jar:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
        at org.bukkit.craftbukkit.v1_7_R1.CraftServer.loadPlugin(CraftServer.java:300) [craftbukkit.
jar:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
        at org.bukkit.craftbukkit.v1_7_R1.CraftServer.enablePlugins(CraftServer.java:282) [craftbukk
it.jar:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
        at org.bukkit.craftbukkit.v1_7_R1.CraftServer.reload(CraftServer.java:632) [craftbukkit.jar:
git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
        at org.bukkit.Bukkit.reload(Bukkit.java:279) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-14-g15b0
4d8-b2991jnks]
        at org.bukkit.command.defaults.ReloadCommand.execute(ReloadCommand.java:23) [craftbukkit.jar
:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
        at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:199) [craftbukkit.jar:
git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
        at org.bukkit.craftbukkit.v1_7_R1.CraftServer.dispatchCommand(CraftServer.java:544) [craftbu
kkit.jar:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
        at org.bukkit.craftbukkit.v1_7_R1.CraftServer.dispatchServerCommand(CraftServer.java:531) [c
raftbukkit.jar:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
        at net.minecraft.server.v1_7_R1.DedicatedServer.aw(DedicatedServer.java:286) [craftbukkit.ja
r:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
        at net.minecraft.server.v1_7_R1.DedicatedServer.u(DedicatedServer.java:251) [craftbukkit.jar
:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
        at net.minecraft.server.v1_7_R1.MinecraftServer.t(MinecraftServer.java:545) [craftbukkit.jar
:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
        at net.minecraft.server.v1_7_R1.MinecraftServer.run(MinecraftServer.java:457) [craftbukkit.j
ar:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
        at net.minecraft.server.v1_7_R1.ThreadServerApplication.run(SourceFile:617) [craftbukkit.jar
:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]

Main:

Code:java

public class NumberOne extends JavaPlugin{
 
    public GameSetup game;
    public SpawnLocations spawnloc;
    public LobbyLocation lobbyloc;
    public ShopLocation shoploc;
    public PointSystem point_system;
 
    public void onEnable(){
        loadMisc();
        load(); //Line 27
        loadEvent();
        loadCmd();
    }
 
 
    public void load(){
        spawnloc = new SpawnLocations(this); //Line 34
        game = new GameSetup(this);
        lobbyloc = new LobbyLocation(this);
        shoploc = new ShopLocation(this);
        point_system = new PointSystem(this);
    }
 
    public void loadEvent(){
        PluginManager pm = Bukkit.getPluginManager();
        pm.registerEvents(new OnJoinServer(this), this);
        pm.registerEvents(new OnLeaveServer(this), this);
        pm.registerEvents(new OnDamage(this), this);
 
    }
 
    public void loadCmd(){
        getCommand("NO").setExecutor(new AdminCommand(this));
    }
 
    public void loadMisc(){
        getConfig().options().copyDefaults(true);
        saveConfig();
    }
 
}

Line 18 in SpawnLocations.jar

Code:java

public class SpawnLocations {
 
    private NumberOne m;
    public SpawnLocations(NumberOne instance){
        m = instance;
    }
    List<String> spawnLoc = m.getConfig().getStringList("SpawnLocations"); //Here
 
    public void saveSpawnLocation(Player p){
        Location loc = p.getLocation();
        String loc_string = Loc2String(loc);
        spawnLoc.add(loc_string);
        m.getConfig().set("SpawnLocations", spawnLoc);
        m.saveConfig();
 
    }
 

 

Dubehh how can this error happen:
at me.dubehh.Locations.SpawnLocations.<init>(SpawnLocations.java:18) ~[?:?]
, if there is no line 18 in SpawnLocations.java?

 

Datdenkikniet

There is a line 18 lol.
I marked it with 'here'.
Remember a class always has those 'import blablabla' en package stuff.

I just copy'd it from public class.

Datdenkikniet

Code:java

package me.dubehh.Locations;
 
import java.util.List;
import java.util.Random;
 
import me.dubehh.Main.NumberOne;
 
import org.bukkit.Bukkit;
import org.bukkit.Location;
import org.bukkit.entity.Player;
 
public class SpawnLocations {
 
    private NumberOne m;
    public SpawnLocations(NumberOne instance){
        m = instance;
    }
    List<String> spawnLoc = m.getConfig().getStringList("SpawnLocations");
 
    public void saveSpawnLocation(Player p){
        Location loc = p.getLocation();
        String loc_string = Loc2String(loc);
        spawnLoc.add(loc_string);
        m.getConfig().set("SpawnLocations", spawnLoc);
        m.saveConfig();
 
    }

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

 

Bump!

 

Dubehh lel, no bumping within 24 hours :>
I think your config/the stringlist SpawnLocations must be null

 

This line
List<String> spawnLoc = m.getConfig().getStringList("SpawnLocations");
is outside of any method.

This means it will be run when the class is created and before the constructor runs. At this point there is no m set so it is null.

Code:java

 
private NumberOne m;
List<String> spawnLoc = m.getConfig().getStringList("SpawnLocations");
 
 
public SpawnLocations(NumberOne instance){
m = instance;
}

That's how it looks another way. To solve it you need to put it in a method that's called after m is set (after m = instance; ).

 

Datdenkikniet
O lol. I went full retard.
I actually defined it w/o checking for a null (ofcourse)
Thanks.

coldandtired
Yup, got it

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