Error if you dont type full command about mobplugin HOT 2 CLOSED

That's an easy one... in MobPlugin.onCommand() method under case "spawn" the first line is String mob = args[1]; But nothing in the previous code ever verified there actually is an args[1] - only args[0] is checked.

from mobplugin.

Super simple fix would be to add:
if (args.length == 1) { commandSender.sendMessage("/mob spawn <mob> <opt:player> - Spawn a mob"); break; }
right at the beginning of case "spawn" to give them a syntax hint and avoid the outofbounds error.

But that fix ignores all the other potential failure cases when a user enters a /mob command. And speaking in more general terms, it's going to be a difficult to read switch case as you add more commands later.

It may be better to move the code block that gives the player the proper syntax for /mob commands into it's own method. Then have the various cases in MobPlugin.onCommand(), including default, call that method if they run into cases where args[0] is not a valid command or args.length is not the correct size for the command.

And you may even consider moving the code for each switch case into their own methods for readability reasons.

from mobplugin.