diff -r f2293e8566f5 -r 4b2c8bedb094 org/sonews/daemon/CommandSelector.java --- a/org/sonews/daemon/CommandSelector.java Thu Aug 20 18:41:21 2009 +0200 +++ b/org/sonews/daemon/CommandSelector.java Mon Aug 24 14:40:37 2009 +0200 @@ -29,28 +29,17 @@ /** * Selects the correct command processing class. * @author Christian Lins + * @since sonews/1.0 */ -class CommandSelector +public class CommandSelector { private static Map instances = new ConcurrentHashMap(); - - public static CommandSelector getInstance() - { - CommandSelector csel = instances.get(Thread.currentThread()); - if(csel == null) - { - csel = new CommandSelector(); - instances.put(Thread.currentThread(), csel); - } - return csel; - } + private static Map> commandClassesMapping + = new ConcurrentHashMap>(); - private Map commandMapping = new HashMap(); - private Command unsupportedCmd = new UnsupportedCommand(); - - private CommandSelector() + static { String[] classes = Resource.getAsString("helpers/commands.list", true).split("\n"); for(String className : classes) @@ -63,13 +52,7 @@ try { - Class clazz = Class.forName(className); - Command cmd = (Command)clazz.newInstance(); - String[] cmdStrs = cmd.getSupportedCommandStrings(); - for(String cmdStr : cmdStrs) - { - this.commandMapping.put(cmdStr, cmd); - } + addCommandHandler(className); } catch(ClassNotFoundException ex) { @@ -86,6 +69,35 @@ } } + public static void addCommandHandler(String className) + throws ClassNotFoundException, InstantiationException, IllegalAccessException + { + Class clazz = Class.forName(className); + Command cmd = (Command)clazz.newInstance(); + String[] cmdStrs = cmd.getSupportedCommandStrings(); + for (String cmdStr : cmdStrs) + { + commandClassesMapping.put(cmdStr, clazz); + } + } + + public static CommandSelector getInstance() + { + CommandSelector csel = instances.get(Thread.currentThread()); + if(csel == null) + { + csel = new CommandSelector(); + instances.put(Thread.currentThread(), csel); + } + return csel; + } + + private Map commandMapping = new HashMap(); + private Command unsupportedCmd = new UnsupportedCommand(); + + private CommandSelector() + {} + public Command get(String commandName) { try @@ -95,16 +107,23 @@ if(cmd == null) { - return this.unsupportedCmd; + Class clazz = commandClassesMapping.get(commandName); + if(clazz == null) + { + cmd = this.unsupportedCmd; + } + else + { + cmd = (Command)clazz.newInstance(); + this.commandMapping.put(commandName, cmd); + } } else if(cmd.isStateful()) { - return cmd.getClass().newInstance(); + cmd = cmd.getClass().newInstance(); } - else - { - return cmd; - } + + return cmd; } catch(Exception ex) {