From 6e179f96cfda89998349a4063bbd9f92db936f88 Mon Sep 17 00:00:00 2001 From: Tux Date: Sat, 14 Dec 2013 09:34:42 -0500 Subject: [PATCH] Pre-construct 2 of the more common chat messages. --- .../redisbungee/RedisBungeeCommands.java | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/src/main/java/com/imaginarycode/minecraft/redisbungee/RedisBungeeCommands.java b/src/main/java/com/imaginarycode/minecraft/redisbungee/RedisBungeeCommands.java index 82fbcae..cf9307c 100644 --- a/src/main/java/com/imaginarycode/minecraft/redisbungee/RedisBungeeCommands.java +++ b/src/main/java/com/imaginarycode/minecraft/redisbungee/RedisBungeeCommands.java @@ -30,6 +30,11 @@ import java.util.concurrent.TimeUnit; * @since 0.2.3 */ public class RedisBungeeCommands { + private static final BaseComponent[] NO_PLAYER_SPECIFIED = + new ComponentBuilder("").color(ChatColor.RED).append("You must specify a player name.").create(); + private static final BaseComponent[] PLAYER_NOT_FOUND = + new ComponentBuilder("").color(ChatColor.RED).append("No such player found.").create(); + public static class GlistCommand extends Command { protected GlistCommand() { super("glist", "bungeecord.command.list", "redisbungee"); @@ -77,10 +82,10 @@ public class RedisBungeeCommands { sender.sendMessage(new ComponentBuilder("").color(ChatColor.BLUE).append(args[0]).append(" is on ") .append(si.getName()).append(".").create()); } else { - sender.sendMessage(new ComponentBuilder("").color(ChatColor.RED).append("That user is not online.").create()); + sender.sendMessage(PLAYER_NOT_FOUND); } } else { - sender.sendMessage(new ComponentBuilder("").color(ChatColor.RED).append("You must specify a player name.").create()); + sender.sendMessage(NO_PLAYER_SPECIFIED); } } } @@ -103,7 +108,7 @@ public class RedisBungeeCommands { sender.sendMessage(new ComponentBuilder("").color(ChatColor.RED).append(args[0]).append(" has never been online.").create()); } } else { - sender.sendMessage(ChatColor.RED + "You must specify a player name."); + sender.sendMessage(NO_PLAYER_SPECIFIED); } } } @@ -120,10 +125,10 @@ public class RedisBungeeCommands { if (ia != null) { sender.sendMessage(new ComponentBuilder("").color(ChatColor.GREEN).append(args[0]).append(" is connected from ").append(ia.toString()).append(".").create()); } else { - sender.sendMessage(new ComponentBuilder("").color(ChatColor.RED).append("No such player found.").create()); + sender.sendMessage(PLAYER_NOT_FOUND); } } else { - sender.sendMessage(new ComponentBuilder("").color(ChatColor.RED).append("You must specify a player name.").create()); + sender.sendMessage(NO_PLAYER_SPECIFIED); } } }