- How to use StringBuilder properly instead of an excessively long method
- if (args.length == 0) {
- player.sendMessage(ChatColor.DARK_RED + "Usage: /pm <Target> <Message>");
- } else if (args.length > 20) {
- player.sendMessage(ChatColor.DARK_RED + "Error: Maximum of 20 words.");
- } else {
- StringBuilder sb = new StringBuilder();
- sb.append(ChatColor.DARK_AQUA).append("[PM] ");
- sb.append(player.getName() + " > " + targetPlayer.getName());
- sb.append(ChatColor.WHITE);
- for (final String word : args) {
- sb.append(word).append(' ');
- }
- sb.setLength(sb.length - 1);
- targetPlayer.sendMessage(sb.toString());
- }
- for (final String word : args) {
- sb.append(word).append(' ');
- }
- for (int i = 0; i < args.length; i++) {
- sb.append(args[i]).append(' ');
- }
- targetPlayer.sendMessage(ChatColor.DARK_AQUA + "[PM] "
- + player.getName() + " > " + targetPlayer.getName()
- + ChatColor.WHITE + " " + args[0])
- targetPlayer.sendMessage(
- new StringBuilder()
- .append(ChatColor.DARK_AQUA).append("[PM] ")
- .append(player.getName()).append(" > ")
- .append(targetPlayer.getName()).append(ChatColor.WHITE)
- .append(' ').append(args[0]).build());
- switch(args.length){
- case 0: dosomething(); break;
- case 1: doSomethingElse();break;
- default: doSomethingEntirelyDifferent();break;
- }
- if(args.length == 0) {
- player.sendMessage(ChatColor.DARK_RED + "Usage: /pm <Target> <Message>");
- } else if(args.length > 20 && player.getServer().getPlayer(args[0]) != null){
- player.sendMessage(ChatColor.DARK_RED + "Error: Maximum of 20 words.");
- } else {
- StringBuilder argsConcat= new StringBuilder();
- for(int i;i<args.length;i++) {
- argsConcat.append(" "+args[i]);
- }## Heading ##
- targetPlayer.sendMessage(ChatColor.DARK_AQUA + "[PM] " + player.getName() + " > " + targetPlayer.getName() + ChatColor.WHITE + argsConcat.toString());
- }