NPE while teleporting to a location stored in HashMap

O hai, I'm billofbong.

I have almost finished a plugin, but I have one single NPE holding me back... I'll add a comment on where it happens:

Code:java

package org.awesomecraft.plugins.orphanage;
 
import java.util.HashMap;
import java.util.Map;
import org.bukkit.ChatColor;
import org.bukkit.Location;
import org.bukkit.World;
import org.bukkit.command.Command;
import org.bukkit.command.CommandExecutor;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Orphanage extends JavaPlugin {
    Map<World, Location> orphanage = new HashMap<World, Location>();
    Map<Player, Player> mothers = new HashMap<Player, Player>(); //<Mother, Orphan>
    Map<Player, Boolean> isOrphan = new HashMap<Player, Boolean>(); //<Orphan, isOrphan>
    public void onDisable() {
        System.out.println(this + " is now disabled!");
        try {
            SLAPI.save(mothers, "mothers.map");
        } catch (Exception ex) {
        }
        try {
            SLAPI.save(isOrphan, "orphans.map");
        } catch (Exception ex) {
        }
        try {
            SLAPI.save(orphanage, "orphanage.map");
        }
        catch(Exception ex) {
        }
    }
 
    public void onEnable() {
        getCommand("adopt").setExecutor(new CommandExecutor() {
 
            public boolean onCommand(CommandSender cs, Command cmnd, String alias, String[] args) {
                if(args.length == 1) {
                    if(cs instanceof Player) {
                    Player target = getServer().getPlayer(args[0]);
                    Player sender = (Player) cs;
                    sender.sendMessage(ChatColor.GREEN + "You are now the parent of " + target.getDisplayName() + "!");
                    target.sendMessage(ChatColor.GREEN + "You have been adopted by " + sender.getDisplayName() + "!");
                    mothers.put(sender, target);
                    isOrphan.put(target, Boolean.FALSE);
                    }
                    else{
                        cs.sendMessage(ChatColor.RED + "You must be ingame to use this command.");
                    }
                }
                return true;
            }
        });
        getCommand("abandon").setExecutor(new CommandExecutor() {
 
            public boolean onCommand(CommandSender cs, Command cmnd, String alias, String[] args) {
                if(cs instanceof Player) {
                Player target = getServer().getPlayer(args[0]);
                Player sender = (Player) cs;
                if(mothers.containsValue(target) && mothers.containsKey(sender)) {
                    mothers.put(null, null);
                    isOrphan.put(target, true);
                    target.sendMessage(ChatColor.RED + "You are an orphan again.");
                    sender.sendMessage(ChatColor.RED + "You abandoned you orphan on the streets.");
                } else {
                sender.sendMessage(ChatColor.RED + "You aren't the parent of that person!");
                }
                }
                return true;
        }});
        getCommand("setorphanage").setExecutor(new CommandExecutor() {
 
            public boolean onCommand(CommandSender cs, Command cmnd, String alias, String[] args) {
                if(cs instanceof Player) {
                    Player sender = (Player) cs;
                    orphanage.put(sender.getWorld(), sender.getLocation());
                    sender.sendMessage(ChatColor.GREEN + "Set an orphanage at you current location.");
                }
                else {
                    cs.sendMessage(ChatColor.RED + "You must be ingame to use this command.");
                }
                return true;
            }
        });
        getCommand("orphanage").setExecutor(new CommandExecutor() {
 
            public boolean onCommand(CommandSender cs, Command cmnd, String alias, String[] args) {
                if(cs instanceof Player) {
                    Player sender = (Player) cs;
                    if(orphanage != null) {
                        Location tele = orphanage.get(sender.getWorld()); //right here...
                        sender.teleport(tele);
                    }
                    else {
                        sender.sendMessage(ChatColor.RED + "An orphanage hasn't been set yet.");
                    }
                }
                else {
                  cs.sendMessage(ChatColor.RED + "You must be ingame to use this command.");
                }
                return true;
            }
        });
        try {
            mothers = (Map<Player, Player>)SLAPI.load("mothers.map");
        } catch (Exception ex) {
        }
        try {
            isOrphan = (Map<Player, Boolean>)SLAPI.load("orphans.map");
        } catch (Exception ex) {
        }
        try {
            orphanage = (Map<World, Location>)SLAPI.load("orphanage.map");
        } catch (Exception ex) {
        }
 
        System.out.println(this + " is now enabled!");
    }
}
 

This happens after setting an orphanage, reloading, and then trying to do /orphanage (which teleports you to the orphanage).
Here's the error:

Code:

13:18:24 [SEVERE] null
org.bukkit.command.CommandException: Unhandled exception executing command 'orphanage' in plugin Orphanage v1.0
	at org.bukkit.command.PluginCommand.execute(PluginCommand.java:42)
	at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:165)
	at org.bukkit.craftbukkit.CraftServer.dispatchCommand(CraftServer.java:378)
	at net.minecraft.server.NetServerHandler.handleCommand(NetServerHandler.java:757)
	at net.minecraft.server.NetServerHandler.chat(NetServerHandler.java:722)
	at net.minecraft.server.NetServerHandler.a(NetServerHandler.java:715)
	at net.minecraft.server.Packet3Chat.a(Packet3Chat.java:33)
	at net.minecraft.server.NetworkManager.b(NetworkManager.java:226)
	at net.minecraft.server.NetServerHandler.a(NetServerHandler.java:93)
	at net.minecraft.server.NetworkListenThread.a(SourceFile:108)
	at net.minecraft.server.MinecraftServer.w(MinecraftServer.java:527)
	at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:425)
	at net.minecraft.server.ThreadServerApplication.run(SourceFile:457)
Caused by: java.lang.NullPointerException
	at org.bukkit.craftbukkit.entity.CraftPlayer.teleport(CraftPlayer.java:308)
	at org.bukkit.craftbukkit.entity.CraftEntity.teleport(CraftEntity.java:156)
	at org.awesomecraft.plugins.orphanage.Orphanage$4.onCommand(Orphanage.java:97)
	at org.bukkit.command.PluginCommand.execute(PluginCommand.java:40)
	... 12 more

Many thanks,

billofbong

 

You doing the wrong check... Orphanage is never null, but orphanage.get(sender.getWorld()) is, because this orphanage hasn't been set yet. Either you define tele then check for nullity, or you check for orphanage.containsKey(sender.getWorld)