我的套接字服务器循环类正确吗？ [关闭]

Question

当客户端的消息类型为“ connect”时，我的服务器出现错误，这很奇怪。

服务器接收方代码：

package threads;

public class ReceiveFromClientThread extends Thread {

ObjectInputStream inFromClient = null;
ServerSocket servs = new ServerSocket();
Socket link = null;
Message m = null;

public void run() {

    try {
        this.servs.servs.accept();
        this.inFromClient = new ObjectInputStream(link.getInputStream());
    } catch (SocketException e3) {
        System.err.println(" Connection reset by peer.");
    } catch (IOException e1) {
        e1.printStackTrace();
    }

    while(true) {   
        try {
            if (link.isConnected()) {
                this.m = (Message) inFromClient.readObject(); //SHOULDN'T THIS THREAD PAUSE HERE?

                if (m.getType().equalsIgnoreCase("normal")) {
                      System.out.println("[" + m.getTimestamp() + "] "  
                          + m.getOwner() + " -> " + m.getText());
                      ServerMain.mc.pushMessageToList(this.m); //Added to message list.
                } else if (m.getType().equalsIgnoreCase("connect")) {
                      System.out.println(m.getOwner() + " connected from "
                          + this.link.getInetAddress()+":"+this.link.getPort());
                } else if (m.getType() == "disconnect") {
                      System.err.println("[" + m.getTimestamp() + "] " 
                          + "[Origin: " + m.getIp() + "] " + m.getOwner() 
                          + " Disconnected." );
                }
            }
        } catch (IOException | ClassNotFoundException e) {
        }
    }
}

public ReceiveFromClientThread(Socket link) {
    this.link = link;   
}
}

客户端连接代码：

ClientMain.sock     = new Socket("127.0.0.1",21215);
ClientMain.oos      = new ObjectOutputStream(sock.getOutputStream());

Message m = new Message();
        m = m.setType("connect");
        m.setOwner("Raphael");
        m.setIp(sock.getInetAddress().getHostAddress());
        m.setPcname(sock.getInetAddress().getCanonicalHostName());
        clientmain.ClientMain.oos.writeObject(m);

它应该可以正常工作，但是会产生无限循环，有时还会出现NullPointerException。 我的代码中有任何错误吗？ 服务器线程不应该在readObject（）行上挂起吗？

Answer 1

从您提交的代码中可以看出，客户端看起来不错，但是服务器存在一些问题。

创建服务器套接字时，必须指定要绑定到的端口（在构造函数中）。 这定义了您在哪个端口上监听并等待客户端。

绑定套接字侦听后，可以调用.accept()接受客户端传入的连接。 .accept()方法将返回一个表示所连接客户端的套接字。 通常，您将在.accept()上循环并在每次返回以处理新客户端（您可能有多个客户端）时生成一个线程。 要与客户端通信，您需要使用返回的套接字。

在您的代码中，您似乎正在使用未绑定的服务器套接字，并且link客户端套接字绝对不是来自.accept() 。

编辑：正如Joachim所指出的，例如，您还需要通过断开循环来处理关闭套接字的情况。

Answer 2

这是完全错误的。 此类代码一次只能正确处理一个客户端。

• 换句话说，调用accept()的结果必须是在其自己的线程中执行的新Runnable ，并且其run()方法构造其自己的所有流并调用在接受的套接字上执行所有I / O的方法。

• 所有这些都不必在该Runnable的构造函数中发生，该构造函数仍是accept循环的线程的一部分。

• 接受的套接字和这些流必须是该Runnable实例成员，而不是服务器类的实例成员。 循环通常看起来像这样：

   while (true) { new Thread(new ConnectionHandler(ss.accept())).start(); } 

  如上所述，其中ConnectionHandler实现Runnable 。

• 请注意，循环不会执行任何其他操作，特别是它不会更改其所属类中的任何实例状态。