Solved Don't really know what is wrong... :?

Trouble Code:

Code:java

public boolean onCommand(CommandSender sender, Command command,
            String label, String[] args) {
 
        if (command.getName().equalsIgnoreCase("queue")) {
 
            if (!(sender instanceof Player))
                return false;
 
            Player p = (Player) sender;
 
            if (args.length < 1)
                return false;
 
            World w = Bukkit.getWorld(args[0]);
 
            if (w == null)
                return false;
 
            addToQueue(args[0], p.getName());
            return true;
 
        }
 
        return true;
 
    }
 
    public static void addToQueue(String MapName, String PlayerName) {
 
        if (!plugin.queueMap.containsKey(MapName)) {
 
            List<String> pList = new ArrayList<String>();
            pList.add(PlayerName);
 
            plugin.queueMap.put(MapName, pList);
            return;
 
        }
 
        if (plugin.queueMap.containsKey(MapName)) {
 
            for (Map.Entry<String, List<String>> map : plugin.queueMap
                    .entrySet()) {
 
                if (map.getKey().equalsIgnoreCase(MapName)) {
 
                    if (!map.getValue().contains(PlayerName)) {
 
                        map.getValue().add(PlayerName);
                        QueueScoreboard.update(MapName);
 
                    }
                    return;
 
                }
 
            }
 
        }
 
    }

Lines that are having issues:
"addToQueue(args[0], p.getName());"
"if (!plugin.queueMap.containsKey(MapName))"

I am getting a Null Pointer Exception :/

Please help :>

 

Disturbed_Creator your plugin is null.

 

Define your plugin as:

MainClass plugin;

public currentclassname(Mainclass i) {

plugin = i

}

 

MineDoubleSpace DreTaX

That is not my entire class file
I am asking what is wrong with just that bit of code. I have defined plugin, and implemented this class as a CommandExecutor. What is wrong in the code I showed you?

 

Disturbed_Creator still seems like plugin is null, how did you define the plugin?

 

MineDoubleSpace
It was defined

static League plugin;
public QueueCommand(League plugin){
QueueCommand.plugin = plugin;
}

Also there are no error messages in eclipse.

 

Disturbed_Creator

Ok, so your constructor is:

Code:java

static League plugin;
 
public QueueCommand(League plugin){
QueueCommand.plugin = plugin;
}

I don't know how you got that error, probably because the constructor is set up incorrectly and Eclipse doesn't know it... If your plugin is getting a null pointer there is probably something wrong in your constructor. This is how I use constructors in all of my classes that need them, and it works every time. CREDIT GOES TO spoljo666 FOR THIS METHOD.

Try this as your constructor:

Code:java

 public final PVPEssentials plugin;
 
public ExplosionListener(final PVPEssentials plugin) {
this.plugin = plugin;
} 

Also, in your main class make sure it is defined

as public static League plugin;

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

 

BungeeTheCookie
With this code I am getting the same problems :/

Code:java

package me.disturbed.league.commands;
 
import java.util.ArrayList;
import java.util.List;
import java.util.Map;
 
import me.disturbed.league.League;
import me.disturbed.league.scoreboard.QueueScoreboard;
 
import org.bukkit.Bukkit;
import org.bukkit.World;
import org.bukkit.command.Command;
import org.bukkit.command.CommandExecutor;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
 
public class QueueCommand implements CommandExecutor {
 
    public final League plugin;
 
    public QueueCommand(final League plugin) {
 
        this.plugin = plugin;
 
    }
 
    public boolean onCommand(CommandSender sender, Command command,
            String label, String[] args) {
 
        if (command.getName().equalsIgnoreCase("queue")) {
 
            if (!(sender instanceof Player))
                return false;
 
            Player p = (Player) sender;
 
            if (args.length < 1)
                return false;
 
            World w = Bukkit.getWorld(args[0]);
 
            if (w == null)
                return false;
 
            addToQueue(args[0], p.getName());
            return true;
 
        }
 
        return true;
 
    }
 
    public void addToQueue(String MapName, String PlayerName) {
 
        if (!plugin.queueMap.containsKey(MapName)) {
 
            List<String> pList = new ArrayList<String>();
            pList.add(PlayerName);
 
            plugin.queueMap.put(MapName, pList);
            return;
 
        }
 
        if (plugin.queueMap.containsKey(MapName)) {
 
            for (Map.Entry<String, List<String>> map : plugin.queueMap
                    .entrySet()) {
 
                if (map.getKey().equalsIgnoreCase(MapName)) {
 
                    if (!map.getValue().contains(PlayerName)) {
 
                        map.getValue().add(PlayerName);
                        QueueScoreboard.update(MapName);
 
                    }
                    return;
 
                }
 
            }
 
        }
 
    }
 
}
 

BungeeTheCookie MineDoubleSpace DreTaX
I did some testing and the problem doesn't lie with the plugin variable.

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

 

queueMap may be null

 

Rocoty
queueMap is defined in the Main Class as:

Code:

public HashMap<String, List<String>> queueMap;

 

Disturbed_Creator
You need to do.(You need to do this unless you are defining the hashmap in the onenable or something along the lines of that)
public HashMap<String, List<String>> queueMap = new HashMap<String, List<String>>();

 

xCyanide
Thank you so much.... I have spent an ungodly amount of time on this.. only to find out it was a simple fix... I hate the universe.

 

Disturbed_Creator
Lol