客户端/服务器设计模式

Question

我是编程新手，我想编写一个简单的客户端服务器系统，其中客户端发送诸如“SAY house”之类的命令，服务器将关键字“SAY”检测为命令并返回关键字后面的任何内容（在这种情况下为“house” ”）。有很多例子如何做到这一点，但我还想通过这个项目学习的是一个很好的设计模式，可以解决这样的问题。我不想只在 Server.java 和 Client.java 等 2 个类中实现所有内容。

所以我从一个 Server 类开始，它在一个端口上启动服务器并等待客户端。当他从客户端收到请求时，Server 类将输入委托给“RequestHandler”类。此类执行 switch-case 部分以检测诸如“Say”或“SHUTDOWN”之类的命令，并通过接口调用相应的方法。

这是一个很好的模式吗？问题是例如服务“SHUTDOWN”。我必须将 Socket 对象一直带到执行命令的方法中。或者也许这种模式不适合升级具有更多功能的服务器？请帮助改进！

谢谢

---

public class Server {
private int port;
private ServerSocket server;

public Server(int port) throws Exception{
    this.port = port;
    try{
          server = new ServerSocket(port);      
     }catch(Exception e){
          System.out.println("Server kann nicht gestartet werden: "+e.getMessage());
          throw e;
     }
}

public Server(int port){
    this.port = port;
}

public void start(){
    Socket client;
    Thread thread;

    while (true) {
        try {
            client = server.accept();
            // Verbindung eingegangen, Objekt erzeugen und in Thread laufen lassen
            System.out.println("Verbindung von "+client.getInetAddress());
            CommandHandler handler = new CommandHandlerImpl(client,new ControllerImpl());
            thread = new Thread(handler);
            thread.start();
        }
        catch (Exception e) {
          System.out.println("Verbindungsfehler: "+e);
        }
      }
}

}

public class CommandHandlerImpl implements CommandHandler{

private final String regex = " ";
private IController controller;
private Socket client;

public CommandHandlerImpl(Socket client, IController controller){
    this.client = client;
    this.controller = controller;
}

@Override
public String getCommand(String abstractString) throws Exception {
    String result;
    String[] arr = abstractString.split(regex);
    String command = arr[0];

    String s = "";
    if(arr.length > 1){
        s = arr[1];
    }
    switch (command) {
    case "CAPITALIZE":
        result = controller.capitalize(s);
        break;
    case "BYE":
        result = controller.sayBye();
        break;
    case "SHUTDOWN":
        result = controller.shutdown(s);
        break;
    default:
        result="Command nicht gefunden!!!";
    }
    return result;
}

@Override
public void run() {
    String line, res;
    try {
        PrintWriter out = new PrintWriter(client.getOutputStream());
        BufferedReader in = new BufferedReader(new InputStreamReader(client.getInputStream()));

        while (true) {
            line = in.readLine();
            System.out.println(line);
            res = null;
            if (line != null){
                res = getCommand(line);
            }
            if (res == null){
                break;
            }
            out.println(res);
            out.flush();
        }
          System.out.println("Verbindung von "+client.getInetAddress()+" beenden.");
          client.close();
    } catch (IOException e) {
        e.printStackTrace();
    }catch(Exception e){
        e.printStackTrace();
    }
}

}

IController 是一个对 String 执行某些操作并返回一个 String 的接口，以便我可以将其发送回客户端。这是好的，或者对于一个好的设计来说应该如何？

Answer 1

此类执行 switch-case 部分以检测诸如“Say”或“SHUTDOWN”之类的命令，并通过接口调用相应的方法。

这是一个很好的模式吗？

通常不应使用 switch 案例，除非所有案例都事先完成。如果我们知道有扩展，就像你的情况一样，我们根本不应该使用 switch-cases。

对于您的情况，您可以根据需要使用命令或策略。如果您的操作集在逻辑上非常相似，请使用Strategy，否则使用Command。

当我读到你的问题时，我认为Command更适合你。

我试图用命令模式实现来概述解决方案，但是理解你的一些代码片段有点困难。通常，只要您的代码对于阅读它的局外人来说不太具有反射性，就应该使用注释。我想你可以通过我提到的来源理解。

此外，如果您想将 String 从客户端映射到所需的 Command 对象，请使用 Java Map。不要使用 Switch-cases 或 if-else 梯子，这肯定会让它变得混乱。如有问题请提出。:))