/broadcast Command Sending Multiple Messages!

Code:java

@Override
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
 
        if (cmd.getName().equalsIgnoreCase("broadcast")){
            if (sender instanceof Player){
                if (sender.hasPermission("se.broadcast")){
                    if (args.length == 0){
                        sender.sendMessage(ChatColor.DARK_PURPLE+ "Whoops! You need to add more arguments for this to work!");
                    }
                    else{
                        if (args.length >= 1){
                            StringBuilder x = new StringBuilder();
                            for (int i = 1; i < args.length; i++) {
                                x.append(args[i] + " ");
                                sender.sendMessage(ChatColor.GOLD+ "Done! Broadcast sent!");
                                Bukkit.broadcastMessage(ChatColor.DARK_RED+ ""+ ChatColor.BOLD+ "ALERT (FROM " + sender.getName()+ "): "+ ChatColor.RED+ "" + ChatColor.ITALIC+ x.toString().trim());
 
                        }
                        }
                    }
                }
                else{
                    //no perm player
                }
            }
            else{
                //console
            }
        }
 
        return false;
    }[/i]

Hello! When I use the code above to send a message, it will not say the first word, ex:

'/broadcast hello im testing this'
sends the message
'ALERT (FROM Silver_Olympus): im testing this'

It also does it like this,

 

Your for loop starts at the args[1], when it should start at args[0]...

 

Tehmaker
Ok, thanks, that fixed it not showing the first word, but do you know how to fix the message showing up three times and done with each one adding a word?

 

You have the messages in the for loop. It sends them 3 times because the loop runs three times. Take them outside the loop.

 

Tehmaker
Maybe im not understanding... Say I did the command

/broadcast hi im testing this command for you

it would send

ALERT (FROM Silver_Olympus): hi
ALERT (FROM Silver_Olympus): hi im
ALERT (FROM Silver_Olympus): hi im testing
ALERT (FROM Silver_Olympus): hi im testing this
ALERT (FROM Silver_Olympus): hi im testing this command
ALERT (FROM Silver_Olympus): hi im testing this command for
ALERT (FROM Silver_Olympus): hi im testing this command for you

 

SilverOlympus

Quite obvious really..

SilverOlympus

Code:java

int x = 1; //Change this
StringBuilder sb = new StringBuilder();
for(int i= x; i<args.length; i++){
sb.append(args).append(" ");
}
String message = sb.toString();
broadcast(message); //Yay
 

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

 

@Unica

When using that code, and sending the message I put in the chat this happens:

 

SilverOlympus append args[i not args

I can't close the bracket because Bukkit makes it italic, but you know what I mean.