Solved Command throwing errors

I'm not quite sure what I've done wrong this time but when I type a command without any arguments on my plugin it throws an error
Error:

Code:

[12:52:08 INFO]: eKember issued server command: /class
[12:52:08 ERROR]: null
org.bukkit.command.CommandException: Unhandled exception executing command 'clas
s' in plugin SpellsAndClasses v1.0
        at org.bukkit.command.PluginCommand.execute(PluginCommand.java:46) ~[spi
got.jar:git-Spigot-1649]
        at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:18
1) ~[spigot.jar:git-Spigot-1649]
        at org.bukkit.craftbukkit.v1_7_R4.CraftServer.dispatchCommand(CraftServe
r.java:767) ~[spigot.jar:git-Spigot-1649]
        at net.minecraft.server.v1_7_R4.PlayerConnection.handleCommand(PlayerCon
nection.java:1043) [spigot.jar:git-Spigot-1649]
        at net.minecraft.server.v1_7_R4.PlayerConnection.a(PlayerConnection.java
:880) [spigot.jar:git-Spigot-1649]
        at net.minecraft.server.v1_7_R4.PacketPlayInChat.a(PacketPlayInChat.java
:28) [spigot.jar:git-Spigot-1649]
        at net.minecraft.server.v1_7_R4.PacketPlayInChat.handle(PacketPlayInChat
.java:65) [spigot.jar:git-Spigot-1649]
        at net.minecraft.server.v1_7_R4.NetworkManager.a(NetworkManager.java:186
) [spigot.jar:git-Spigot-1649]
        at net.minecraft.server.v1_7_R4.ServerConnection.c(ServerConnection.java
:81) [spigot.jar:git-Spigot-1649]
        at net.minecraft.server.v1_7_R4.MinecraftServer.v(MinecraftServer.java:7
34) [spigot.jar:git-Spigot-1649]
        at net.minecraft.server.v1_7_R4.DedicatedServer.v(DedicatedServer.java:2
89) [spigot.jar:git-Spigot-1649]
        at net.minecraft.server.v1_7_R4.MinecraftServer.u(MinecraftServer.java:5
84) [spigot.jar:git-Spigot-1649]
        at net.minecraft.server.v1_7_R4.MinecraftServer.run(MinecraftServer.java
:490) [spigot.jar:git-Spigot-1649]
        at net.minecraft.server.v1_7_R4.ThreadServerApplication.run(SourceFile:6
28) [spigot.jar:git-Spigot-1649]
Caused by: java.lang.ArrayIndexOutOfBoundsException: 0
        at me.kemsly.spells.spells.onCommand(spells.java:205) ~[?:?]
        at org.bukkit.command.PluginCommand.execute(PluginCommand.java:44) ~[spi
got.jar:git-Spigot-1649]
        ... 13 more

Here is the code:

Code:Java

 
public int resetPrice = 500;
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args)
    {
        if(cmd.getName().equalsIgnoreCase("spell"))
        {
            Player player = (Player) sender;
            if(args.length == 0 || args[0].equalsIgnoreCase(null))
            {
                try {
                    if(this.mysql.getClass(player).equalsIgnoreCase("Warrior"))
                    {
                        player.sendMessage(ChatColor.RED + "[" + ChatColor.GRAY + "Spells" + ChatColor.RED + "]" + ChatColor.AQUA + " You can use the Punch spell.");
                        player.sendMessage(ChatColor.AQUA + "You can buy spells and wands with /spell buy.");
                    }
                    if(this.mysql.getClass(player).equalsIgnoreCase("Assassin"))
                    {
                        player.sendMessage(ChatColor.RED + "[" + ChatColor.GRAY + "Spells" + ChatColor.RED + "]" + ChatColor.AQUA + " You can use the Vanish spell.");
                        player.sendMessage(ChatColor.AQUA + "You can buy spells and wands with /spell buy.");
                    }
                } catch (ClassNotFoundException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
                } catch (SQLException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
                }
            }
            if(args[0].equalsIgnoreCase("buy") && args[1].equalsIgnoreCase(null))
            {
                player.sendMessage(ChatColor.RED + "[" + ChatColor.GRAY + "Spells" + ChatColor.RED + "]" + ChatColor.AQUA + " With /spell buy you can buy in game items including wands, and potions");
                player.sendMessage(ChatColor.AQUA + "You can currently buy a wand, a potion of healing I, or a potion of healing II.");
                player.sendMessage(ChatColor.AQUA + "Use '/spell buy heal one' or '/spell buy heal two' to buy healing potions and '/spell buy wand' to buy a wand.");
            }
            if(args[0].equalsIgnoreCase("buy") && args[1].equalsIgnoreCase("wand"))
            {
                int wandPrice = 350;
                String playername = player.getName();
                if(economy.getBalance(playername)>=wandPrice)
                {
                    EconomyResponse er = economy.withdrawPlayer(playername, wandPrice);
                    if(er.transactionSuccess())
                    {
                        ItemStack wand = new ItemStack(Material.BLAZE_ROD, 1, (short)1);
                        ItemMeta wandMeta = wand.getItemMeta();
                        wandMeta.setDisplayName(ChatColor.GRAY + "Wand");
                        player.getInventory().addItem(wand);
                    }
                    else
                    {
                        player.sendMessage(ChatColor.RED + "[" + ChatColor.GRAY + "Spells" + ChatColor.RED + "]" + ChatColor.AQUA + " Transaction failed!");
                    }
                }
                else
                {
                    player.sendMessage(ChatColor.RED + "[" + ChatColor.GRAY + "Spells" + ChatColor.RED + "]" + ChatColor.AQUA + " You do not have enough money to buy a wand, you need $350!");
                }
            }
            if(args[0].equalsIgnoreCase("buy") && args[1].equalsIgnoreCase("heal"))
            {
                int halfHealPrice = 850;
                int fullHealPrice = 950;
                String playername = player.getName();
                if(args[2].equalsIgnoreCase("one") || args[1].equalsIgnoreCase("I"))
                {
                    if(economy.getBalance(playername)>=halfHealPrice)
                    {
                        EconomyResponse er = economy.withdrawPlayer(playername, halfHealPrice);
                        if(er.transactionSuccess())
                        {
                            Potion healI = new Potion(PotionType.INSTANT_HEAL, 1);
                            ItemStack HealI = healI.toItemStack(1);
                            ItemMeta healIMeta = HealI.getItemMeta();
                            healIMeta.setDisplayName(ChatColor.GOLD + "Heal I");
                            player.getInventory().addItem(HealI);
                        }
                        else
                        {
                            player.sendMessage(ChatColor.RED + "[" + ChatColor.GRAY + "Spells" + ChatColor.RED + "]" + ChatColor.AQUA + " Transaction failed!");
                        }
                    }
                    else
                    {
                        player.sendMessage(ChatColor.RED + "[" + ChatColor.GRAY + "Spells" + ChatColor.RED + "]" + ChatColor.AQUA + " You do not have enough money to buy a half healstone, you need $850!");
                    }
                }
                if(args[2].equalsIgnoreCase("two") || args[1].equalsIgnoreCase("II"))
                {
                    if(economy.getBalance(playername)>=fullHealPrice)
                    {
                        EconomyResponse er = economy.withdrawPlayer(playername, fullHealPrice);
                        if(er.transactionSuccess())
                        {
                            Potion healII = new Potion(PotionType.INSTANT_HEAL, 2);
                            ItemStack HealII = healII.toItemStack(1);
                            ItemMeta healIIMeta = HealII.getItemMeta();
                            healIIMeta.setDisplayName(ChatColor.GOLD + "Heal II");
                            player.getInventory().addItem(HealII);
                        }
                        else
                        {
                            player.sendMessage(ChatColor.RED + "[" + ChatColor.GRAY + "Spells" + ChatColor.RED + "]" + ChatColor.AQUA + " Transaction failed!");
                        }
                    }
                    else
                    {
                        player.sendMessage(ChatColor.RED + "[" + ChatColor.GRAY + "Spells" + ChatColor.RED + "]" + ChatColor.AQUA + " You do not have enough money to buy a wand, you need $950!");
                    }
                }
            }
        }
        if(cmd.getName().equalsIgnoreCase("class"))
        {
            Player player = (Player) sender;
            if(args.length == 0 || args[0].equalsIgnoreCase(null))
            {
                try {
                    player.sendMessage(ChatColor.RED + "[" + ChatColor.GRAY + "Class" + ChatColor.RED + "]" + ChatColor.AQUA + " Your class is currently '" + this.mysql.getClass(player) + "'");
                } catch (ClassNotFoundException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
                } catch (SQLException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
                }
            }
            if(args[0].equalsIgnoreCase("reset"))
            {
                String playername = player.getName();
                try {
                    if(this.mysql.getClass(player).equalsIgnoreCase("Warrior") || this.mysql.getClass(player).equalsIgnoreCase("Assassin") || this.mysql.getClass(player).equalsIgnoreCase("Archer"))
                    {
                        if(economy.getBalance(playername)>=resetPrice)
                        {
                            EconomyResponse er = economy.withdrawPlayer(playername, resetPrice);
                            if(er.transactionSuccess())
                            {
                                this.mysql.resetClass(player);
                                cooldown.remove(player);
                                player.sendMessage(ChatColor.RED + "[" + ChatColor.GRAY + "Class" + ChatColor.RED + "]" + ChatColor.AQUA + " Class reset, you may now choose a different class.");
                            }
                        }
                        else
                        {
                            player.sendMessage(ChatColor.RED + "[" + ChatColor.GRAY + "Class" + ChatColor.RED + "]" + ChatColor.AQUA + " You do not have enough money to reset your class, you require $500!");
                        }
                    }
                    else
                    {
                        player.sendMessage(ChatColor.RED + "[" + ChatColor.GRAY + "Class" + ChatColor.RED + "]" + ChatColor.AQUA + " You are not in a class already, you must be in a class to reset your class.");
                    }
                } catch (ClassNotFoundException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
                } catch (SQLException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
                }
            }
            else
            {
                try {
                    player.sendMessage(ChatColor.RED + "[" + ChatColor.GRAY + "Class" + ChatColor.RED + "]" + ChatColor.AQUA + " Your class is currently '" + this.mysql.getClass(player) + "'");
                } catch (ClassNotFoundException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
                } catch (SQLException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
                }
            }
        }
        return false;
    }
 

Thanks for any help also, please (if you have time) lightly explain what I did wrong (no need to go really in depth).

 

@kember007 Read this. Then try to fix it yourself.

 

What is line 205? The exception is an ArrayOutOfBounds! Try learning how to read the stack trace, will help you when errors occur.

 

Umm, line 205 is just a closing bracket, I don't think that would cause any issue because it has to be there...

 

@kember007 Then you have updated your code since that stack trace. Re-export your plugin. Re-run it in the server. Post the new error log.

 

Are you using the correct formatting? Do you stack two statements on the same line? If you're using Eclipse, do you use Control-Shift-F ?

 

Well the stacktrace now says the error is this line "
if(args[0].equalsIgnoreCase("reset"))
"
see anything wrong?

 

@kember007 Yes. There is nothing in 'args'. You have to add a check to make sure that the String[] isn't empty.

 

I was thinking that aswell but I have this and shouldn't it fix my issue?

Code:

if(args.length == 0 || args[0].equalsIgnoreCase(null))

 

@kember007 No. When you do args[0], if the length isn't at least 1, you will get an error.

 

Okay so to be clear if I enter a command without any args it will return an error? how do I solve this issue?

Ah, I fixed it.
if you where wondering the solution you need to do something like this...

Code:java

 
public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args)
{
        if(cmd.getName().equalsIgnoreCase("enterCommandHere"))
        {
               if(args.length == 0)
                {
                         //do stuff
                         return false; // <-- that was where the error was!
                }
         }
    return false;
}
 

so you can see I forgot to have the return false in the if statement.