I have a command: /ish, but if I enter /ish I get information from public void execute and this string:

How to fix it? My code:

package ivansteklow.ishelper.init;

import ivansteklow.isdev.Notifier;
import ivansteklow.ishelper.Refs;
import net.minecraft.command.CommandBase;
import net.minecraft.command.CommandException;
import net.minecraft.command.ICommandSender;
import net.minecraft.entity.player.EntityPlayerMP;
import net.minecraft.server.MinecraftServer;
import net.minecraft.util.text.TextComponentString;
import net.minecraft.world.World;

public class MainCmd extends CommandBase{

	@Override
	public String getName() {
		return "ish";
	}

	@Override
	public String getUsage(ICommandSender sender) {
		
		return "To get more info: /ish help (?)";
	}

	public int getRequiredPermissionLevel()
    {
        return 2;
    }
	
	@Override
	public void execute(MinecraftServer server, ICommandSender sender, String[] args) throws CommandException {
		World world = sender.getEntityWorld();
		if(world.isRemote){
			Notifier.error("Not processing on server side!", Refs.NAME+"/Commands");
		}else{
			
			EntityPlayerMP player = getCommandSenderAsPlayer(sender);
			
			if(!(args.length >= 1)){
				player.sendMessage(new TextComponentString("[ISH] Wrong usage! Usage:"));
				player.sendMessage(new TextComponentString("[ISH] /ish killa (killall) - kill all mobs and drops"));
				player.sendMessage(new TextComponentString("[ISH] /ish heal (healme) - heal player"));
				player.sendMessage(new TextComponentString("[ISH] /ish oheart (oneheart) - set minimal health and hunger"));
			}
			
			if(args[0].equals("killa") || args[0].equals("killall")){
				server.commandManager.executeCommand(server.getServer(), "/kill @e[type=!Player]");
			}else if(args[0].equals("heal") || args[0].equals("healme")){
				player.setHealth(player.getMaxHealth());
				player.getFoodStats().setFoodLevel(20);
			}else if(args[0].equals("oheart") || args[0].equals("oneheart")){
				player.setHealth(1.0f);
				player.getFoodStats().setFoodLevel(1);
			}else if(args[0].equals("help") || args[0].equals("?")){
				player.sendMessage(new TextComponentString("[ISH] Usage:"));
				player.sendMessage(new TextComponentString("[ISH] /ish killa (killall) - kill all mobs and drops"));
				player.sendMessage(new TextComponentString("[ISH] /ish heal (healme) - heal player"));
				player.sendMessage(new TextComponentString("[ISH] /ish oheart (oneheart) - set minimal health and hunger"));
			}else if(args[0].equals("out")){
				if(args[1].equals("0")){
					server.commandManager.executeCommand(server, "/gamerule commandBlockOutput false");
				}else if(args[1].equals("1")){
					server.commandManager.executeCommand(server, "/gamerule commandBlockOutput true");
				}
			}else{
				player.sendMessage(new TextComponentString("[ISH] Wrong usage! Usage:"));
				player.sendMessage(new TextComponentString("[ISH] /ish killa (killall) - kill all mobs and drops"));
				player.sendMessage(new TextComponentString("[ISH] /ish heal (healme) - heal player"));
				player.sendMessage(new TextComponentString("[ISH] /ish oheart (oneheart) - set minimal health and hunger"));
			}
			
		}
	}

}

HELP ME PLS!

Edited June 14, 20178 yr by IvanSteklow
Solved

Show your register code.

That chat message is sent when ICommand#execute throws an exception. The exception is logged when this happens, so look at the log to see what happened.

6 minutes ago, Kokkie said:

Show your register code.

event.registerServerCommand(new MainCmd());

 

2 minutes ago, Choonster said:

That chat message is sent when ICommand#execute throws an exception. The exception is logged when this happens, so look at the log to see what happened.

 

This is log:

java.lang.ArrayIndexOutOfBoundsException: 0
	at ivansteklow.ishelper.init.MainCmd.execute(MainCmd.java:47) ~[MainCmd.class:?]
	at net.minecraft.command.CommandHandler.tryExecute(CommandHandler.java:129) [CommandHandler.class:?]
	at net.minecraft.command.CommandHandler.executeCommand(CommandHandler.java:101) [CommandHandler.class:?]
	at net.minecraft.network.NetHandlerPlayServer.handleSlashCommand(NetHandlerPlayServer.java:946) [NetHandlerPlayServer.class:?]
	at net.minecraft.network.NetHandlerPlayServer.processChatMessage(NetHandlerPlayServer.java:922) [NetHandlerPlayServer.class:?]
	at net.minecraft.network.play.client.CPacketChatMessage.processPacket(CPacketChatMessage.java:47) [CPacketChatMessage.class:?]
	at net.minecraft.network.play.client.CPacketChatMessage.processPacket(CPacketChatMessage.java:8) [CPacketChatMessage.class:?]
	at net.minecraft.network.PacketThreadUtil$1.run(PacketThreadUtil.java:21) [PacketThreadUtil$1.class:?]
	at java.util.concurrent.Executors$RunnableAdapter.call(Unknown Source) [?:1.8.0_101]
	at java.util.concurrent.FutureTask.run(Unknown Source) [?:1.8.0_101]
	at net.minecraft.util.Util.runTask(Util.java:29) [Util.class:?]
	at net.minecraft.server.MinecraftServer.updateTimeLightAndEntities(MinecraftServer.java:754) [MinecraftServer.class:?]
	at net.minecraft.server.MinecraftServer.tick(MinecraftServer.java:699) [MinecraftServer.class:?]
	at net.minecraft.server.integrated.IntegratedServer.tick(IntegratedServer.java:156) [IntegratedServer.class:?]
	at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:548) [MinecraftServer.class:?]
	at java.lang.Thread.run(Unknown Source) [?:1.8.0_101]

 

This log shows me this string, but I can't see any error in code:

if(args[0].equals("killa") || args[0].equals("killall")){

 

1 minute ago, IvanSteklow said:

java.lang.ArrayIndexOutOfBoundsException: 0
	at ivansteklow.ishelper.init.MainCmd.execute(MainCmd.java:47) ~[MainCmd.class:?]

 

 

This log shows me this string, but I can't see any error in code:

if(args[0].equals("killa") || args[0].equals("killall")){

 

 

args was length 0, so attempting to access index 0 threw an ArrayIndexOutOfBoundsException.

 

You're already checking if the length is 0 before printing the help text, you just need to return from the function after doing that instead of trying to access index 0.

11 hours ago, Choonster said:

 

args was length 0, so attempting to access index 0 threw an ArrayIndexOutOfBoundsException.

 

You're already checking if the length is 0 before printing the help text, you just need to return from the function after doing that instead of trying to access index 0.

Thanks!