java - 如何动态处理聊天程序中的命令？

Question

我的问题更多的是设计问题。我目前正在用 Java 开发一个经典的服务器-客户端聊天程序。在我收到命令之前一切都很好。我认为用户发送命令会很方便，然后服务器会处理这些命令来更改他们的昵称。问题是我想编写灵活的代码，尤其是面向对象的代码。为了避免无休止的 if/else if 语句来知道输入了什么命令，我相信最好为每个从父类(super class) Command 继承的命令创建一个类。然后我可以通过在所有子类中重写的 getCommand() 函数返回特定命令。但这根本不能解决我的问题。服务器仍然需要使用instanceof测试返回了什么命令。动态执行此操作的一种方法是从父类(super class) Command 自动向下转换它，然后调用服务器类中的适当函数。例如:

public void processCommand(CommandNick c) {}
public void processCommand(CommandKick c) {}

但我还没有找到任何合适的方法来做到这一点，即使我找到了，我觉得这里仍然存在设计问题。我相信有一种很好且灵活的方法可以做到这一点，但几天时间不足以让我弄清楚。有任何想法吗？提前致谢! :)

Answer 1

我假设您的服务器以带有发送者和字符串的对象形式接收消息。创建命令类，并在服务器初始化代码中创建 HashMap<String, AbstractCommand>与 String作为 key 和您的AbstractCommand类作为值。您的命令应该扩展此类。注册所有命令，如下所示:

commandRegistry.put("help", new HelpCommandHandler());

我假设命令是一 strip 有 ! 的消息在它之前。因此，当您收到消息时，请检查它是否是命令:

Message message = (Your Message)
String messageBody = message.getBody();
Sender messageSender = message.getSender();

if(messageBody.startsWith("!")) {
    // Split the message after every space
    String[] commandParts = messageBody.split(" ");
    // The first element is the command base, like: !help
    String baseCommand = commandParts[0];
    // Remove the first character from the base, turns !help into help
    baseCommand = baseCommand.substring(1, baseCommand.length());
    // Creates a new array for the arguments. The length is smaller, because we won't copy the command base
    String[] args = new String[commandParts.length - 1];
    // Copy the elements of the commandParts array from index 1 into args from index 0
    if(args.length > 0) {
        System.arraycopy(commandParts, 1, args, 0, commandParts.length - 1);
    }
    // Your parse method
    processCommand(sender, baseCommand, args);
}

public void processCommand(Sender sender, String base, String[] args) {
    if(commandRegistry.containsKey(base)) {
        commandRegistry.get(base).execute(sender, args);
    } else {
        // Handle unknown command
    }
}

public abstract class AbstractCommand {
    public abstract void execute(Sender sender, String[] args);
}

示例实现。我假设您的服务器是单例，您可以使用 Server.get() 获取它的对象。或任何类似的方法。

public class HelpCommandHandler extends AbstractCommand { /* !help */
    @Override
    public void execute(Sender sender, String[] args) {
        sender.sendMessage("You asked for help."); // Your code might not work like this.
    }
}

public class ChangeNickCommandHandler extends AbstractCommand { /* !changenick newNick */
    @Override
    public void execute(Sender sender, String[] args) {
        // I assume you have a List with connected players in your Server class
        String username = sender.getUsername(); // Your code might not work like this
        Server server = Server.get(); // Get Server instance
        server.getUsers().get(username).setNickname(args[0]); // Argument 0. Check if it even exists.
    }
}

// Server class. If it isn't singleton, you can make it one like this:
public class Server {
    private static Server self;
    public static Server init(/* Your args you'd use in a constructor */) { self = new Server(); return get(); }
    public static Server get() { return self; }

    private List<User> users = new List<User>();
    private HashMap<String, AbstractCommand> commandRegitry = new HashMap<>();

    // Make construcor private, use init() instead.
    private Server() {
        commandRegistry.put("help", new HelpCommandHandler());
        commandRegistry.put("changenick", new ChangeNickCommandHandler());
    }

    // Getters
    public List<User> getUsers() {
        return users;
    }

    public HashMap<String, AbstractCommand> getRegistry() {
        return commandRegistry;
    }
}