Multiple Args To A Command.

Hey, can someone tell me how I would add another arg to a command and for a different arg to do a different thing.

 

You just use the String[] of args you put in your onCommand. Example:

Code:java

public boolean onCommand(CommandSender sender, Command cmd, String s, String[] args) {
if(cmd.getName().equalsIgnoreCase("AE") {
  if(args[0].equalsIgnoreCase("help") {
// Do stuff
    }
  }
}

 

TheGamesHawk2001 You can also use a switch() { } statement which can be more efficient in some ways.

 

Googlelover1234 es359 Thxx and if I did if args0.equals ignore case"help" below the base command, and I typed /as help it would display the stuff below the arg 0 etc? I'm on a tablet so soz if it doesn't make much sense and I didnt put brackets cus its effort lol

 

TheGamesHawk2001

Yes, for example, if you had the base command "msg" and the args[0] "cheeseburger" you would use /msg cheeseburger. You can keep going with this, like you could have "msg" as the base, args[0] as "cheeseburger" and args[1] as "Panwaffle". That would be /msg cheeseburger PanWaffle.

 

Googlelover1234 Kay, thanks

 

Example using a switch statement.

Code:java

switch(arguments) {
 
    case "arg1":
        //do arg1.
    break;
    case "arg2":
 
        //Do arg2.
    break;
 
    default:
        player.sendMessage("Incorrect arguments, or to many arguments.");
 
}

 

Googlelover1234 When I did

Code:java

package me.TheGamesHawk2001.Commands;
 
import org.bukkit.ChatColor;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Main extends JavaPlugin {
 
    public boolean onCommand(CommandSender sender, Command cmd, String s, String[] args) {
        if(cmd.getName().equalsIgnoreCase("commands")) {
            sender.sendMessage(ChatColor.RED + "You did just /commands");
          if(args[0].equalsIgnoreCase("help")) {
              sender.sendMessage(ChatColor.RED + "You just did /commands help");
              if(args[0].equalsIgnoreCase("list")) {
                  sender.sendMessage(ChatColor.RED + "You just did /commands list");
            }
          }
        }
        return true;
}
}

It ran all of the args when I typed only /commands help etc

 

es359 Do note that Minecraft and Bukkit are (still) compiled with Java 6.
TheGamesHawk2001 you are nesting your commands incorrectly (they shouldn't be nested at all)

 

fireblast709 Help?

 

I've used switch statements in several of my plugins...?

 

TheGamesHawk2001

He literally spelled it out for you..
Your if statements are nested.. If you don't know what that is look at your bracket placement and see if you notice anything.

 

Idk but I tried this:

Code:java

package me.TheGamesHawk2001.FakeOp;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Main extends JavaPlugin {
 
public boolean onCommand(CommandSender sender, Command cmd, String commandLable, String[] args) {
 
if (!(sender instanceof Player)) {
sender.sendMessage( ChatColor.AQUA + "The Console Cant Use This Command");
 
}
 
Player player = (Player) sender;
 
if(cmd.getName().equalsIgnoreCase("FakeOp")) {
if (!(sender instanceof Player)) {
sender.sendMessage(ChatColor.GOLD + "" + ChatColor.BOLD + "You have to be a player to use this command");
return true;
}
if (args.length == 0) {
player.sendMessage(ChatColor.YELLOW + "You cannot fakeop yourself. You have to be op to use it!");
return true;
}else if(args.length == 1);
if(Bukkit.getPlayer(args[0]) == null) {
player.sendMessage(ChatColor.RED + "Player " + ChatColor.ITALIC + args[0] + "" + ChatColor.RESET + "" + ChatColor.RED + "Could not be found or is not online.");
return true;
}else if (args.length == 0) {
if (cmd.getName().equalsIgnoreCase("help"));
sender.sendMessage(ChatColor.RED + "Type /fakeop <player> <reload>");
return true;
}
Player targetPlayer = Bukkit.getPlayer(args[0]);
player.sendMessage(ChatColor.GRAY + "You have FakeOpped " + args[0]);
 
}
return true;
}
}

and when I type /fakeop help, it doesnt register.

bump

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

es359 fireblast709

He's saying that you should compile your plugins against Java 1.6, if you want everyone to be able to use it (personally, I think this is stupid because if you haven't upgraded to Java 7 yet, something is very wrong with you). But using Strings in switch statements is something that only Java 1.7+ supports, and if you compile your plugins against Java 1.7, people still running legacy software won't be able to run your plugin. However I feel they need to upgrade and get over it.

 

TheGamesHawk2001

I think you need to learn Java before working on Bukkit because your problem is a super easy fix.

Your problem in the second code is when the player runs /fakeop it returns true, which means the if statement after that won't be run.. Does that help?

 

Watto But I don't want it to run all the other code when I type /fakeop since that has its own stuff to run when you type /fakeop and other args have their own stuff too.


Edit. This is my new one

Code:java

package me.TheGamesHawk2001.Commands;
 
import org.bukkit.ChatColor;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Main extends JavaPlugin {
 
    public boolean onCommand(CommandSender sender, Command cmd, String s, String[] args) {
        if(cmd.getName().equalsIgnoreCase("commands")) {
            sender.sendMessage(ChatColor.RED + "You did just /commands");
        }
          if(args[0].equalsIgnoreCase("help")) {
              sender.sendMessage(ChatColor.RED + "You just did /commands help");
          }
        return true;
    }
}

Is this right. I actually have no clue what I'm doing with args.

 

No it's not. It will work but it's a terrible way of doing it.

Look at it this way with your new code, you have to ensure that /commands is being typed before you check for the 'help' argument

Just look at it logically, if you have more than one command, and lets say for example that command is '/hello' if that user types /hello help it's going to display that help message.
You need to put the arguments check inside of /commands if statement.

 

Watto so I put it like this?

Code:java

package me.TheGamesHawk2001.Commands;
 
import org.bukkit.ChatColor;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Main extends JavaPlugin {
 
public boolean onCommand(CommandSender sender, Command cmd, String s, String[] args) {
if(cmd.getName().equalsIgnoreCase("commands")) {
sender.sendMessage(ChatColor.RED + "You did just /commands");
 if(args[0].equalsIgnoreCase("help")) {
 sender.sendMessage(ChatColor.RED + "You just did /commands help");
}
 }
return true;
}
}

 

TheGamesHawk2001
That should work fine. Although you will receive both of the messages if you were to execute the "/commands help" command. Also you should check the array length before trying to access an element inside it.

Code:java

if(args.length == 1) { // only 1 argument in the command
    if(args[0] etc) {
 
    }
}

 

Assist How would I make it so when you type /commands it displays a msg. Then /commands member a different and /commands donator for another message?

 

TheGamesHawk2001
You would create new args checks for them, like this:

Code:java

public boolean onCommand(CommandSender sender, Command cmd, String s, String[] args) {
    if (cmd.getName().equalsIgnoreCase("commands")) {
        if (args.length == 0) { // they typed "/commands" only without any args
            sender.sendMessage(ChatColor.RED + "You did just /commands");
        } else if (args.length == 1) {
            if (args[0].equalsIgnoreCase("member")) {
                sender.sendMessage(ChatColor.RED + "You just did /commands member");
            } else if (args[0].equalsIgnoreCase("donator")) {
                sender.sendMessage(ChatColor.RED + "You just did /commands donator");
            }
        }
    }
 
    return false;
}

 

Assist Thanks, What do I do for plugin.yml?

 

TheGamesHawk2001
Make a new section called "commands" and add your command in it.

Code:

commands:
  commands:
    description: a command

It's a bit confusing because your command is called "commands" and the section where you put your commands in is also called "commands".

 

Assist What about the separate arguments. Is that a separate command?

 

@TheGamesHawk200

No, your base command is 'commands' and since they are arguments you don't need to make a separate command.

 

Watto Okay, I have an issue! When ever I type /commands, It runs it fine, but it says /commands in chat when I type it, with any arg. Thanks

Example http://i.imgur.com/f62gpd0.png

 

TheGamesHawk2001

Your onCommand requires a boolean return. The return method is whether or not the command was successful, so set the return to true.

 

Watto

Code:java

package me.TheGamesHawk2001.Commands;
 
import org.bukkit.ChatColor;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Main extends JavaPlugin {
 
public boolean onCommand(CommandSender sender, Command cmd, String s, String[] args) {
 
if (cmd.getName().equalsIgnoreCase("commands")) {
if (args.length == 0) { // they typed "/commands" only without any args
sender.sendMessage(ChatColor.RED + "" + ChatColor.BOLD + "Groups:");
sender.sendMessage(ChatColor.GREEN + "" + ChatColor.BOLD + "Member:");
sender.sendMessage(ChatColor.AQUA + "" + ChatColor.BOLD + "Iron:");
sender.sendMessage(ChatColor.RED + "" + ChatColor.BOLD + "Gold:");
sender.sendMessage(ChatColor.LIGHT_PURPLE + "" + ChatColor.BOLD + "Diamond:");
sender.sendMessage(ChatColor.YELLOW + "" + ChatColor.BOLD + "Emerald:");
sender.sendMessage(ChatColor.WHITE + "" + ChatColor.BOLD + "Chasma:");
 
} else if (args.length == 1) {
if (args[0].equalsIgnoreCase("member")) {
sender.sendMessage(ChatColor.GRAY + "Member has access to the following commands:");
sender.sendMessage(ChatColor.DARK_PURPLE + "Help, Upgrade, Help, Kit, Warp.");
 
} else if (args[0].equalsIgnoreCase("Iron")) {
sender.sendMessage(ChatColor.RED + "You just did /commands donator");
 
} else if (args[0].equalsIgnoreCase("Gold")) {
sender.sendMessage(ChatColor.RED + "You just did /commands donator");
 
} else if (args[0].equalsIgnoreCase("Diamond")) {
sender.sendMessage(ChatColor.RED + "You just did /commands donator");
 
} else if (args[0].equalsIgnoreCase("Emerald")) {
sender.sendMessage(ChatColor.RED + "You just did /commands donator");
 
} else if (args[0].equalsIgnoreCase("Chasma")) {
sender.sendMessage(ChatColor.RED + "You just did /commands donator");
}
}
}
 
return false;
}
}

It does doesn't it? and it's not red..

 

TheGamesHawk2001
Perhaps you have another plugin sending the message?

 

Assist I changed the command to /carrot. Still doesn't work (has /carrot at the bottom)