java - 未调用基于Java接口(interface)的多态性

我正在开发一个程序，该程序将命令作为字符串数据接收，并将其传递给由HashMap定义的正确处理程序。

将命令传递给正确的处理程序的代码如下：

//This is run in its own thread, hence the run() method
@Override
public void run() {
    try {
        //Socket is a Socket object containing the client's connection
        InputStreamReader is = new InputStreamReader(socket.getInputStream());
        //MAX_MESSAGE_SIZE is an integer, specifically 1024
        char[] data = new char[MAX_MESSAGE_SIZE];
        is.read(data, 0, MAX_MESSAGE_SIZE);
        String strData = new String(data);
        //Split on UNIX or Windows type newline
        String[] lines = strData.split("\\r?\\n");
        //First verb determines the command
        String command = (lines[0].split(": "))[0];
        //Re-add the ": " because the HashMap will look for it
        if (handlerMap.containsKey((command + ": "))) {
            System.err.println("Got command: " + command);
            AbstractHandler handler = handlerMap.get(command);
            System.err.println("Passing connection + data to handler...");
            handler.handleData(socket, lines);
            System.err.println("Handler is done!");
        } else {
            System.err.println("Didn't understand command: " + command);
            DataOutputStream outputStream = new DataOutputStream(socket.getOutputStream());
            outputStream.writeBytes(Protocol.UNKNOWN_ERROR);
            outputStream.flush();
            socket.close();
        }
    } catch (IOException e) {
        System.err.println("Caught IOException: " + e.getMessage());
    }
}

HashMap的值部分是实现接口AbstractHandler的对象。 AbstractHandler定义一个方法：handleData(Socket s, String[] lines)。供参考，这是地图的初始化位置：

     public RequestManager(Socket socket) {
    this.socket = socket;
    handlerMap = new HashMap<>();
    handlerMap.put(Protocol.USERNAME, new AuthenticationHandler());
    //Set arg to true b/c it is a set request, not a change request
    handlerMap.put(Protocol.SET_PASSWORD, new ChangePasswordHandler(true));
    handlerMap.put(Protocol.CHANGE_PASSWORD, new ChangePasswordHandler());
    handlerMap.put(Protocol.FORGOT_PASSWORD, new ForgotPasswordHandler());
}

并且对象中的所有handleData方法仅包含以下代码：

@Override
public void handleData(Socket s, String[] lines) {
     clientSocket = s; //clientSocket field in class
     System.err.println("Made it into handler");
}

奇怪的是，显示“将连接+数据传递给处理程序”后，什么都没有发生。我没有看到有关进入处理程序的任何信息，也没有看到处理程序完成的异常或消息。是什么原因造成的？

最佳答案

您测试以查看是否存在处理程序

if (handlerMap.containsKey((command + ": "))) {

但是你尝试得到一个处理程序

AbstractHandler handler = handlerMap.get(command);

因此，如果密钥CommandName:存在，则可能无法通过密钥CommandName来获取它。因此，您在呼叫NullPointerException时将未选中handler.handleData(socket, lines);，并且可运行的对象将死于恐怖的死亡。

看来您需要更改第一个或第二个。假设您说可以看到它显示“正在将连接+数据传递给处理程序...”，我认为您需要将其更改为：

AbstractHandler handler = handlerMap.get(command + ": ");

在处理地图时，如果进行少量的样式更改，就可以防止这种情况现在和将来对您造成影响。 Map.get returns null if the key is not found，因此您可以执行以下操作：

AbstractHandler handler = handlerMap.get(command + ": ");
if (handler != null) {
    /* ... */
    handler.handleData(socket, lines);
    System.err.println("Handler is done!");
} else {
    /* ... */
}