Simple Plugin Help Please

Hi I am posting for help about a simple kit plugin I am trying to make (I'm kinda new to this) when I enter /kits it tells me the kits but it also gives me the 2 kits I have made so far. All I want it to do is name the kits so then i could do /kit starter and it would give me kit starter alone please help! Heres the code!

http://pastebin.com/SGScHCtm

 

Well you just do
commandlabel.equalsIgnoreCase("kit")
then check the number of args and if it is bigger than 0 then check what it is and give them the items.
The commandLabel is the command without the args.

 

Thank you but can you give me an example has i have not yet worked with many arguments and am still trying to learning! Thanks in advance!

 

SwiftyMove

EDIT: WELCOME TO BUKKIT!

Code:java

//instead of if(commandlabel
if(cmd.getName().equalsIgnoreCase("kit")) {
if(args.length == 1) { //Check the length of arguments - /kit is argument 0 and starter is argument 1
if(args[0].equalsIgnoreCase("starter")) {
// Do code
 
//Im checking for the args[0] instead of args[1] because that is how Java works, the Bukkit API makes it easier for us because we can do if(args.length == 1) although in java, the 1 would be replaced with 0 

 

I knew something would happen so i made a test project and implemented what you said to do but i probably arranged it wrong sorry to trouble you so much but can you please help again its saying i have an internal error and neither of the commands are working? Thanks in advance! and also if you could how would i extend this for multiple commands like kit god or something?

http://pastebin.com/iqxaCWFC

 

SwiftyMove

Code:java

package me.swiftymove.fun;
 
import java.util.logging.Logger;
 
 
import org.bukkit.Material;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.inventory.ItemStack;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Fun extends JavaPlugin{
        public final Logger logger = Logger.getLogger("Minecraft");
        public static Fun plugin;
 
        @Override
        public void onEnable(){
                this.logger.info("Enabled");
 
        }
 
        @Override
        public void onDisable(){
                this.logger.info("Disabled");
 
        }
 
        public boolean onCommand(CommandSender sender, Command cmd, String commandlabel, String[] args){
                Player player = (Player) sender;
                if(cmd.getName().equalsIgnoreCase("kit"));{
                        player.sendMessage("works");
                        if(args.length == 1) {
                        if(args[0].equalsIgnoreCase("starter")){
                                ItemStack sp = new ItemStack(Material.STONE_PICKAXE, 1);
                                player.getInventory().addItem(sp);
                                return true;
                        }
        return true;
        }
    } 
}

Might be missing a brace. Using my phone so can't tell - this should work. Also you said if(args.length == 1);
Do not make it a statement. Think of it as real English, would you ask someone, if I buy this. It doesn't make sense.

 

Thank you so much i will now have to try and extend this to several commands but also when i type /starter it also comes up with the message sent when i type /kit this might be because it has return set to true but i will look into it thanks again!