Solved HashMap!!! pls help

Hello, I'm learning to work with the hashmap, but I think everything is correct, but when digit command in game / home, it returns an error.

Code:java

HashMap<String, Location> home = new HashMap<String, Location>();
 
 
    public boolean onCommand(CommandSender sender, Command command,
            String label, String[] args) {
 
        Player p = (Player) sender;
        if(command.getName().equalsIgnoreCase("sethome")){
            home.put(getName(), p.getLocation());
            p.sendMessage("Home marcada");
        }
        else if(command.getName().equalsIgnoreCase("home")){
            Location homeLocation = home.get(p.getName());
            p.teleport(homeLocation);
            p.sendMessage("TELEPORTADO");
        }
        return false;
    }

the error is this

Code:

[18:21:30] [Server thread/ERROR]: null
org.bukkit.command.CommandException: Unhandled exception executing command 'home' in plugin HomelOcation v1.0
    at org.bukkit.command.PluginCommand.execute(PluginCommand.java:46) ~[craftbukkit-1.7.9-R0.2.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:180) ~[craftbukkit-1.7.9-R0.2.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at org.bukkit.craftbukkit.v1_7_R3.CraftServer.dispatchCommand(CraftServer.java:701) ~[craftbukkit-1.7.9-R0.2.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at net.minecraft.server.v1_7_R3.PlayerConnection.handleCommand(PlayerConnection.java:956) [craftbukkit-1.7.9-R0.2.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at net.minecraft.server.v1_7_R3.PlayerConnection.a(PlayerConnection.java:817) [craftbukkit-1.7.9-R0.2.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at net.minecraft.server.v1_7_R3.PacketPlayInChat.a(PacketPlayInChat.java:28) [craftbukkit-1.7.9-R0.2.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at net.minecraft.server.v1_7_R3.PacketPlayInChat.handle(PacketPlayInChat.java:47) [craftbukkit-1.7.9-R0.2.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at net.minecraft.server.v1_7_R3.NetworkManager.a(NetworkManager.java:157) [craftbukkit-1.7.9-R0.2.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at net.minecraft.server.v1_7_R3.ServerConnection.c(SourceFile:134) [craftbukkit-1.7.9-R0.2.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at net.minecraft.server.v1_7_R3.MinecraftServer.v(MinecraftServer.java:667) [craftbukkit-1.7.9-R0.2.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at net.minecraft.server.v1_7_R3.DedicatedServer.v(DedicatedServer.java:260) [craftbukkit-1.7.9-R0.2.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at net.minecraft.server.v1_7_R3.MinecraftServer.u(MinecraftServer.java:558) [craftbukkit-1.7.9-R0.2.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at net.minecraft.server.v1_7_R3.MinecraftServer.run(MinecraftServer.java:469) [craftbukkit-1.7.9-R0.2.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at net.minecraft.server.v1_7_R3.ThreadServerApplication.run(SourceFile:628) [craftbukkit-1.7.9-R0.2.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
Caused by: java.lang.NullPointerException
    at org.bukkit.craftbukkit.v1_7_R3.entity.CraftPlayer.teleport(CraftPlayer.java:484) ~[craftbukkit-1.7.9-R0.2.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at org.bukkit.craftbukkit.v1_7_R3.entity.CraftEntity.teleport(CraftEntity.java:200) ~[craftbukkit-1.7.9-R0.2.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at me.jonasxpx.MainSethome.onCommand(MainSethome.java:31) ~[?:?]
    at org.bukkit.command.PluginCommand.execute(PluginCommand.java:44) ~[craftbukkit-1.7.9-R0.2.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    ... 13 more

 

Could you paste all the code from "MainSetHome" please?

 

Gerov

Code:java

package me.jonasxpx;
 
import java.util.HashMap;
 
import org.bukkit.Location;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.plugin.java.JavaPlugin;
 
public class MainSethome extends JavaPlugin {
 
    @Override
    public void onEnable() {
 
    }
 
    HashMap<String, Location> home = new HashMap<String, Location>();
 
 
    public boolean onCommand(CommandSender sender, Command command,
            String label, String[] args) {
 
        Player p = (Player) sender;
        if(command.getName().equalsIgnoreCase("sethome")){
            home.put(getName(), p.getLocation());
            p.sendMessage("Home marcada");
        }
        else if(command.getName().equalsIgnoreCase("home")){
            Location homeLocation = home.get(p.getName());
            p.teleport(homeLocation);
            p.sendMessage("TELEPORTADO");
        }
        return false;
    }
}
 

 

I think you forgot here what

Code:

home.put(getName(), p.getLocation());

getName() -> p.getName() ?

 

You should check if home.get(p.getName()) is null before trying to teleport a player to it. It could be a player hasn't set his home -> home.get(p.getName()) returns null.

 

Seadragon91


Thanks, that's it. XD

 

JonasXPX
Mark thread as solved,

BTW you arent saving the hashmap -> Server restart -> Homes lost