Simplest way to color a message?

What's the simplest way to make this command /c [color] [message]???

 

Well....

Code:

String a;
if (args[0].equalsIgnoreCase("red"))
{
    a = ChatColor.RED+"";
}
else if (args[0].equalsIgnoreCase("yellow"))
{
    a = ChatColor.YELLOW+"";
}
//So on and so forth

Then append the message to a, and send it. Also, most plugins are open-source here, make sure you look around to see what other kinds of stuff has what you are looking for and take a snip from them or something.

 

Code:java

String message = ChatColor.valueof(args[0].toUpperCase(java.util.Locale.ENGLISH)).toString().concat(message);

 

Personally, I prefer using § directly rather than Bukkit's ChatColor constants.

 

better:

Code:java

try {
    String message = ChatColor.valueof(args[0].toUpperCase(java.util.Locale.ENGLISH)).toString().concat(message);
    //send message
} catch (IllegalArgumentException e) {
    sender.sendMessage("invalid color!");
}

 

Code:java

public static String replaceColors(String string){
return string.replaceAll("(?i)&([a-k0-9])", "\u00A7$1");
}

Just put your message in the parameter and then you can use color codes with '&'.

 

The local variable message may not have been initialized

 

You should send the message from within the try block, as the variable is only defined for this block, just where I put //send message.

 

This doesnt work

Code:

 try {
                    String message = ChatColor.valueOf(args[0].toUpperCase(java.util.Locale.ENGLISH)).toString().concat(message);
                    Bukkit.getServer().broadcastMessage(message);
                    } catch (IllegalArgumentException e) {
                    sender.sendMessage("invalid color!");
                    }

 

@ferrybig's code was incorrect and I didn't notice it: The 'message' parameter at the end of the line should be another variable, e.g.:

Code:java

StringBuilder b = new StringBuilder(args[1]);
for (int i = 2; i < args.length; i++) {
b.append(" ");
b.append(args[i ]);
}
String message = ChatColor.valueOf(args[0].toUpperCase(java.util.Locale.ENGLISH)).toString() + b.toString();

Make sure that the user typed 2 arguments or more before this code is reached or you'll get an ArrayIndexOutOfBoundsException.

 

Is this correct?

Code:

  if(cmd.getName().equalsIgnoreCase("cb"))
            if(sender.hasPermission("Colors.broadcast")) {
                if(args.length < 2);
                return false;
            }
                StringBuilder b = new StringBuilder(args[1]);
                for (int i = 2; i < args.length; i++) {
                b.append(" ");
                b.append(args[i ]);
                }
                String message = ChatColor.valueOf(args[0].toUpperCase(java.util.Locale.ENGLISH)).toString() + b.toString();
                Bukkit.getServer().broadcastMessage(message);
            return true;
            }
        return false;
        }

 

Correct me if im wrong, but you're missing { at the end of your first if statement. And also, you want to return false and stop the command if they have the permission?

Also with your string builder you don't need to args[1] in the brackets, just
StringBuilder b = new StringBuilder();
will do.

 

PHP:

if(cmd.getName().equalsIgnoreCase("cb")){
                if(args.length < 2);
                return false;
            }
            if(sender.hasPermission("Colors.broadcast")) {
                StringBuilder b = new StringBuilder();
                for (int i = 2; i < args.length; i++) {
                b.append(" ");
                b.append(args[i ]);
                }
                String message = ChatColor.valueOf(args[0].toUpperCase(java.util.Locale.ENGLISH)).toString() + b.toString();
                Bukkit.getServer().broadcastMessage(message);
            return true;
            }
        return false;
        }
        return false;
}

correct right?

 

Well, with your first if statement, you added the open { which is good, but with your second if statement, you have a ; which makes it useless.

Code:java

if (cmd.getName().equalsIgnoreCase("cb")) {
if (args.length < 2) {
return false;
}
 
if (sender.hasPermission("colors.broadcast")) {
String msg;
 
StringBuilder sb = new StringBuilder();
for (int i = 1; i < args.length; i++) {
if (i != 1)
sb.append(' ');
sb.append(args[i]);
}
msg = sb.toString();
 
Bukkit.getServer().broadcastMessage(ChatColor.valueOf(args[0].toUpperCase(java.util.Locale.ENGLISH)).toString().concat(msg));
return true;
}
else {
sender.sendMessage("You don't have permission for that command.");
}
return false;
}[/i]

Try that, im not sure about the chat colour bit, but it's what someone said at the start of the thread, so i used that.
You will soon learn where the { and } go and what they do, but with if statements, if you have a { and } at the end of the if statement, the code inside the if statement will only 'work' if the if statement is fulfilled. The { and } are always in pairs, you will never have an odd one.

 

Code:java

        if (cmd.getName().equalsIgnoreCase("cb")) {
            if (args.length < 2) {
                return false;// this will send the usage defined in the plugin.yml to the player
            }
 
            if (sender.hasPermission("colors.broadcast")) {
 
                String color;
 
                try {
                    color = ""+ChatColor.valueOf(args[0].toUpperCase(java.util.Locale.ENGLISH));
                } catch (IllegalArgumentException e) {
                    sender.sendMessage("invalid colour");
                    return true;
                }
 
                StringBuilder sb = new StringBuilder();
                for (int i = 1; i < args.length; i++) {
                    if (i != 1)
                        sb.append(' ');
                    sb.append(args[ i ]);
                }
 
                Bukkit.getServer().broadcastMessage(color + sb.toString());
 
                return true;
 
            } else {
                sender.sendMessage("You don't have permission to use this command.");
            }
            return true;
        }