Solved Developing a time plugin, need help.

So as the title says, I am developing a time plugin (like to do /time to check time and then /time set day/night) and doing /time does at it should but when I go to set the time with /time set day (day is the only one implemented as its easier to fix one command rather than 2), it gives me an internal error saying "Caused by: java.lang.ArrayIndexOutOfBoundsException: 1
at com.omegagamers.rf2minecraft.SimpleTime.SimpleTime.onCommand(SimpleTi
me.java:27)", I have no idea how to fix this as it is needed I believe. If not, oh well, easily fixed. If somebody can help me, it would be great.


Source Code:

Code:java

package com.omegagamers.rf2minecraft.SimpleTime;
import org.bukkit.ChatColor;
import org.bukkit.plugin.java.JavaPlugin;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.World;
 
public final class SimpleTime extends JavaPlugin {
public boolean onCommand(CommandSender sender, Command cmd, String commandlabel, String[] args) {
    if(sender instanceof Player) {
        if(cmd.getName().equalsIgnoreCase("time")) {
            if(args.length == 0) {
            Player player = (Player) sender; /* Gets the player from the sender */
            String world = player.getWorld().getName(); /* Gets the world the player is in & name of it */
            long time = player.getWorld().getTime(); /* Gets the time of the world at the time of sending command */
            int hours = (int)((time / 1000L + 6L) % 24L); /* Grabs the hours of the world time */
            int minutes = (int)(60L * (time % 1000L) / 1000L); /* Grabs the minutes of the world time */
            if(time < 6000) {
            sender.sendMessage(ChatColor.DARK_PURPLE + "The time in: " + ChatColor.LIGHT_PURPLE + "" + world + ChatColor.DARK_PURPLE + " is " + ChatColor.LIGHT_PURPLE + "" + time + ChatColor.DARK_PURPLE + " ticks / " + "" + hours+":"+minutes + "" + "AM");
        }
            if(time > 6000) {
                sender.sendMessage(ChatColor.DARK_PURPLE + "The time in: " + ChatColor.LIGHT_PURPLE + "" + world + ChatColor.DARK_PURPLE + " is " + ChatColor.LIGHT_PURPLE + "" + time + ChatColor.DARK_PURPLE + " ticks / " + "" + hours+":"+minutes + "" + "PM");
            }
    }
                if(args[1].equalsIgnoreCase("set")) {
                    if(args[2] == null) {
                        sender.sendMessage(ChatColor.DARK_PURPLE + "Please provide a time you wish to set the world to.");
                    }
                    if(args[2] != null) { /* If there is 2 arguments after the normal command */
                        if(args[2].equalsIgnoreCase("day")) { /* If the 2nd argument is "day", do the following. */
                            Player player = (Player) sender; /* Grabs sender's player */
                            World world = player.getWorld(); /* Gets world that player is in */
                            world.setFullTime(0); /* Sets world time to day. */
                                sender.sendMessage(ChatColor.DARK_PURPLE + "Time set."); /* Sends player a message telling them time is set */
 
                        }
 
                    }
                }
            }
    }
return true;
}
}

 

You need to return true at the end of the args.length == 0 command, as when it has finished with that, it carries onto checking args[1] which doesn't exist and throws that exception

And by the way, you should use 'else' instead of 'if (time > 6000)', as if time == 6000, neither of the ifs conditions will be satisfied :<

 

Thanks, I fixed it now.