Help me with this error

Here is the error i'm getting :

Code:

 [SEVERE] null
org.bukkit.command.CommandException: Unhandled exception executing command 'freeze' in plugin Freeze plugin v1.1
	at org.bukkit.command.PluginCommand.execute(PluginCommand.java:37)
	at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:129)
	at org.bukkit.craftbukkit.CraftServer.dispatchCommand(CraftServer.java:298)
	at net.minecraft.server.NetServerHandler.handleCommand(NetServerHandler.java:711)
	at net.minecraft.server.NetServerHandler.chat(NetServerHandler.java:676)
	at net.minecraft.server.NetServerHandler.a(NetServerHandler.java:669)
	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:84)
	at net.minecraft.server.NetworkListenThread.a(SourceFile:105)
	at net.minecraft.server.MinecraftServer.h(MinecraftServer.java:451)
	at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:361)
	at net.minecraft.server.ThreadServerApplication.run(SourceFile:422)
Caused by: java.lang.ArrayIndexOutOfBoundsException: 0
	at me.beatcomet.Freeze.Frz.onCommand(Frz.java:128)
	at org.bukkit.command.PluginCommand.execute(PluginCommand.java:35)
	... 12 more

This is the error, it says that the error is in line 128

This is what I have in line 128 :

Code:java

if(args[0] == null){

If you need the whole class file just tell me...

 

It's because you are trying to access the element in the args-array at index 0, and you apparently called the command without arguments.
So args is empty and args[0] doesn't exist (thus throws an exception and doesn't return null/undefined like it would in JavaScript or whatever ).

To check if args[0] is given, use "args.length > 0".

 

Here is the whole main class :

Code:java

package me.beatcomet.Freeze;
 
import java.io.File;
import java.util.logging.Logger;
 
import org.bukkit.ChatColor;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.event.Event;
import org.bukkit.plugin.PluginDescriptionFile;
import org.bukkit.plugin.PluginManager;
import org.bukkit.plugin.java.JavaPlugin;
import org.bukkit.util.config.Configuration;
 
public class Frz extends JavaPlugin {
 
	// Creating a new data file
	File configFile = new File("plugins/" + "Freeze" + "/data.ass");
	File cfgfile = new File("plugins/" + "Freeze" + "/status.ass");
 
	Configuration config = new Configuration(configFile);
 
	Configuration cfg = new Configuration(cfgfile);
 
	// getting Minecraft logger to send console messages
	Logger log = Logger.getLogger("Minecraft");
 
	// creating Player Listener reference
	PListener playerListener = new PListener(this);
	EListener entityListener = new EListener(this);
 
	// What happens when the plugin is being disabled
	public void onDisable() {
		// saving the data
		config.save();
 
		// Getting info from plugin.yml
		PluginDescriptionFile PDFile = this.getDescription();
 
		// sending console message saying the plugin has been disabled
 
		log.info(PDFile.getName() + " Version " + PDFile.getVersion()
				+ " is DISABLED!");
	}
 
	public void onEnable() {
		// Sending a console message saying the plugin has been enabled
 
		// Getting info from plugin.yml
		PluginDescriptionFile PDFile = this.getDescription();
 
		log.info(PDFile.getName() + " Version " + PDFile.getVersion()
				+ " is ENABLED!");
 
		// making a new file if a data file dose not exist
		new File("plugins/" + PDFile.getName()).mkdir();
		if (!configFile.exists()) {
			try {
				configFile.createNewFile();
 
			} catch (Exception e) {
				// sending console message in case the data file could not be
				// created
				log.info("[Freeze] Error when creating config file.");
			}
		}
		// Loading data file
		config.load();
 
		new File("plugins/" + PDFile.getName()).mkdir();
		if (!cfgfile.exists()) {
			try {
				cfgfile.createNewFile();
 
			} catch (Exception e) {
				// sending console message in case the data file could not be
				// created
				log.info("[Freeze] Error when creating config file.");
			}
		}
		// Loading data file
		config.load();
 
		// getting the plugin manager to register events
		PluginManager pm = this.getServer().getPluginManager();
 
		// player join event registration
		pm.registerEvent(Event.Type.PLAYER_JOIN, playerListener,
				Event.Priority.Normal, this);
		pm.registerEvent(Event.Type.PLAYER_MOVE, playerListener,
				Event.Priority.Normal, this);
		pm.registerEvent(Event.Type.ENTITY_DAMAGE, entityListener,
				Event.Priority.Normal, this);
	}
 
	public boolean onCommand(CommandSender sender, Command cmd,
			String commandLabel, String[] args) {
		if (!(sender instanceof Player)) {
			if (commandLabel.equalsIgnoreCase("Freeze")
					|| commandLabel.equalsIgnoreCase("f")) {
				String name = args[0];
				if (args[0] == null) {
					this.log.info("No player name inserted");
				}
				if ((Boolean) this.config.getProperty(name) == false) {
					this.config.setProperty(name, true);
					this.log.info(name + " Has been freezed!");
					this.config.save();
					return true;
				}
				if ((Boolean) this.config.getProperty(name) == true) {
					this.config.setProperty(name, false);
					this.log.info(name + " Has been unfreezed!");
					this.config.save();
					return true;
				} else {
					this.log.info(name + " Does not exist in the data file");
					this.config.setProperty(name, false);
					this.config.save();
					return true;
				}
			}
			// try now
		}
		if (sender instanceof Player) {
			if (sender.isOp()) {
				if (commandLabel.equalsIgnoreCase("Freeze")
						|| commandLabel.equalsIgnoreCase("f")) {
 
					if (args[0] == null) {
						sender.sendMessage(ChatColor.LIGHT_PURPLE + "[Freeze] "
								+ ChatColor.AQUA + " No name inserted!");
					}
 
					String name = args[0];
					if ((Boolean) this.config.getProperty(name) == false) {
						this.config.setProperty(name, true);
						this.config.save();
						sender.sendMessage(ChatColor.LIGHT_PURPLE + "[Freeze] "
								+ ChatColor.AQUA + name + " Has been freezed!");
						return true;
					}
					if ((Boolean) this.config.getProperty(name) == true) {
						this.config.setProperty(name, false);
						this.config.save();
						sender.sendMessage(ChatColor.LIGHT_PURPLE + "[Freeze] "
								+ ChatColor.AQUA + name
								+ " Has been unfreezed!");
						return true;
					} else {
						sender.sendMessage(ChatColor.LIGHT_PURPLE + "[Freeze] "
								+ ChatColor.AQUA + name
								+ " Does not exist in the data file");
						this.config.setProperty(name, false);
						this.config.save();
						return true;
					}
				}
			}
		}
		if (sender instanceof Player && sender.isOp()) {
			if (commandLabel.equalsIgnoreCase("check")) {
				if (args[0] == null) {
					sender.sendMessage(ChatColor.LIGHT_PURPLE + "[Freeze]"
							+ ChatColor.AQUA
							+ "Pleas insert a players name to get status");
				}
				String name = args[0];
				if ((Boolean) this.config.getProperty(name) == true) {
					sender.sendMessage(ChatColor.GOLD + name + ChatColor.RED
							+ " - Freezed");
					return true;
				}
				if ((Boolean) this.config.getProperty(name) == false) {
					sender.sendMessage(ChatColor.GOLD + name + ChatColor.BLUE
							+ " - Not Freezed");
					return true;
				}
			}
		}
		if (sender instanceof Player && sender.isOp()) {
			if (commandLabel.equalsIgnoreCase("status")) {
				String construct = null;
				for (Player p : getServer().getOnlinePlayers()) {
					if ((Boolean) this.config.getProperty(p.getName()) == true) {
						this.cfg.setProperty(p.getName(), ": Freezed");
					}
					if ((Boolean) this.config.getProperty(p.getName()) == false) {
						this.cfg.setProperty(p.getName(), ": Not Freezed");
					}
					construct = ChatColor.GOLD + p.getName() + ChatColor.AQUA
							+ this.cfg.getProperty(p.getName())
							+ ChatColor.BLACK + " | ";
				}
				sender.sendMessage(construct);
 
			}
 
		}
		return true;
 
	}
 
}

It's the main class before I changed it.
It's in line 164 (the problematic if statement)

 

Did you read my reply?

In Java, you can't check if something is set through comparing it to null!

 

My workaround was checking to see

Code:java

if (args.length == 0)

 

The idea was that it checks if there is no argument after the command label
I didn't knew that this is how it works...