runekingdom

I dont like the way this is done :x but nice

here i made this cleaner for you :)

Code: [Select]

if (playerCommand.startsWith("yell")) {
String rank = "[@mag@Player@bla@]";
String Message = playerCommand.substring(4);
switch(c.playerRights) {
case 0:
rank = (c.isDonator >= 0 ? "[@gre@Donator@bla@]" : "[@mag@Player@bla@]");
break;
case 1:
rank = "[@blu@Mod@bla@]";
break;
case 2:
rank = "[@yel@Admin@bla@]";
break;
case 3:
rank = "[@red@Owner@bla@]";
break;
}
if(c.playerName.equals("brandon"))
rank = "[@red@Developer@bla@]["+ c.playerName +"]:";

for (int j = 0; j < Server.playerHandler.players.length; j++) {
if (Server.playerHandler.players[j] != null) {
Client c2 = (Client)Server.playerHandler.players[j];
c2.sendMessage(rank + "[" + c.playerName + "]: " + Message);
}
}
}


It's good to know some people know how to use cases.
Good job, yet it's common sense.
Some people don't even know what's cases.
You might wanna include
case 4:
which is hidden moderator
case 20:
which is hidden owner
That's just to make it fancy.