Question

我正在制作一个聊天室项目，服务器接受许多客户端，无论客户端写入什么到达其他客户端等等。不幸的是，服务器接受最多2个客户端，并且在一个客户端写入输入后，它会产生错误。

public class Server2 {

    private static ArrayList<Socket> clients;
    private ServerSocket server;
    DataOutputStream os;
    DataInputStream in;
    Socket s;

    public Server2() throws IOException {
        server = new ServerSocket(5000);
        clients = new ArrayList<Socket>();
        System.out.println("Waiting for connections...");
        runOutput();
    }

    public void addClient() throws IOException {
        s = server.accept();
        clients.add(s);
        System.out.println("A new Client has joined");
    }

    public void runOutput() {

        Thread n = new Thread(new Runnable() {
            public void run() {
                try {
                    addClient();
                } catch (IOException e) {
                    e.printStackTrace();
                }
            }
        });
        n.start(); 

        Thread input = new Thread(new Runnable() {
            public void run() {
                try {
                    addClient();
                    in = new DataInputStream(s.getInputStream());
                    os = new DataOutputStream(s.getOutputStream());
                    String st = in.readLine();
                    System.out.println(st);
                    os.writeBytes(st);
                    for(int i = 0; i < clients.size(); i++)
                    {
                        DataOutputStream oo = new DataOutputStream(clients.get(i).getOutputStream());
                        oo.writeBytes(st);
                    }
                } catch (IOException e) {
                    e.printStackTrace();
                }
            }
        });
        input.start();
    }

    public static void main(String[] args) throws IOException {
        Server2 server = new Server2();
    }
}

和客户端类：

public class Client2 {
    final public static String host = "localhost";
    final public static int port = 5000;
    Socket socket;
    DataInputStream in;
    DataOutputStream ou;
    Scanner chat;
    boolean run;
    String name;

    public Client2(String n) throws IOException {
        name = n ;
        socket = new Socket(host , port);
        System.out.println("Connection Successful");
        run = true;
        runOutput();
    }

    public void runOutput() {
        Thread input = new Thread(new Runnable() {
            public void run() {
                while (run) {
                    try {
                        in = new DataInputStream(socket.getInputStream());
                        String s = in.readLine();
                        System.out.println(s);
                        if(chat.nextLine().compareTo("QUIT") == 0)
                            run = false;
                    } catch (IOException e) {
                        e.printStackTrace();
                    }
                }
            }
        });
        input.start();

        Thread t = new Thread(new Runnable() {
            public void run() {
                while (run) {
                    try {
                        ou = new DataOutputStream(socket.getOutputStream());
                        chat = new Scanner(System.in);
                        ou.writeBytes(name + " says :" + chat.nextLine() + "\n");

                    } catch (IOException e) {
                        e.printStackTrace();
                    }
                }
            }
        });

        t.start(); 
    } 

    public static void main(String[] args) throws IOException {
        Client2 client = new Client2("Ahmed");
    }   
}

Answer 1

你的'n'线程没有循环，这意味着它将运行一次（接受连接），然后死掉。

Thread n = new Thread(new Runnable() {
    public void run() {
        while(true) { //control this with your own boolean, or it will run forever
            try {
                addClient();
            }catch(IOException e) { }
        }
    }
});

别担心，你的帖子会在'ss.accept（）'暂停，因为它会等到接受套接字继续。

Thread thread = new Thread(new Runnable() {
    public void run() {
        while(running) {
             String input;

             while((input = inputstream.readUTF()) != null) {
                  //handle input
             }
         }
    }
});

抱歉，写得很快，没有太多时间。如果它没有帮助，我会回来并将它与你的代码联系起来。

您需要某种方法来连续检索服务器输入流中的输入，然后您可以在其中处理输入。

每次从inputstream.readUTF（）中检索内容时，循环都会循环（当然，isnt为null）。希望这个例子可以帮助你

在实际阅读完代码并进行测试后，我注意到你的结构似乎有些偏差。首先，您服务器中的Thread n负责接受连接（addClient()），但您也会在Thread input中首先调用它？

Thread n正在处理接受连接，因此使用循环，Thread n就可以了。 Thread input正在处理从客户端检索的输入。这是你感到困惑的地方。

每个客户端都应该有自己的InputStream和OutputStream服务器端。类似于你有一个套接字列表（因为你要在服务器端创建多个套接字），你需要多个Streams。为此，我建议创建一个User类，如果有的话，让你的ArrayList为User，而不是Socket。

public class User extends Thread { //create a Thread for each connection to handle data seperately
    DataInputStream in;
    DataInputStream out;

    Socket socket;
    public User(Socket socket) { //creates a stream for the socket that the user connected with
        this.socket = socket;

        initStream(); 
    }

    public void initStream() { //inits streams after socket is initiated (through ss.accept())
        try {
            out = new DataOutputStream(socket.getOutputStream());
            in = new DataInputStream(socket.getInputStream());
        }catch(IOException e) { e.printStackTrace(); }
    }

    public void closeStream() throws IOException { //throws exception since i have a chance to catch it in my run() method
            out.close(); in.close();
            socket.close();
    }

    public void sendMessage(String string) {
        try {
            out.writeUTF(string);
        }catch(IOException e) { e.printStackTrace(); }
    }

    public void run() { //when thread starts, accept/handle data
        String input;

        try {
            while((input = in.readUTF()) != null) {
                //handle input
            }
            closeStream();
        }catch(IOException e) { e.printStackTrace(); }
    }
}

in.readLine()已弃用，这意味着它已过时且不应使用（以避免错误）。现在您已经设置了用于处理数据的User类，现在每次收到连接时都可以创建一个新用户。

public class Server {
    private static boolean running = true;

    public static ArrayList<User> userlist;

    public static synchronized void stop() { //synchronized in-case another thread other than the main thread stops the server
        if(!running) return;
        running = false;
    }

    public static void main(String[] args) {
        /* uses the JVM thread (main thread for applications), so if you dont wanna
        ** dedicate your entire server to accepting connections only (since Users are in new thread)
        ** create a new thread for the code in this void method. */

        try {
            ServerSocket ss = new ServerSocket(8000);

            userlist = new ArrayList<User>();
            User user; //holds the user so we can put it in the list, then start
        while(running) {
            user = new User(ss.accept());

            list.add(user);
            user.start();
        }
        } catch(IOException e) {
            e.printStackTrace(); }
    }
}

希望这能让您更好地了解服务器的运行方式。

至于你的客户......好吧，你应该意识到很多事情：

首先，很大的问题，你正在重新初始化每个循环的DataOutputStream，这将导致流服务器端关闭。要把它从循环中取出来，可能先把它放进去（或者查看我的服务器示例，以便更好地了解处理流初始化）。

其次，名称应保留在服务器上，而不是客户端上。这样，它很容易遍历名称，检查是否采取了等等。

第三，没有网络，但看到你永远不会引用Thread n或Thread input，除了启动它，你应该删除引用变量。（即使我非常确定垃圾收集器拾取本地变量，仍然看起来更好）。例如：

public void method() {
    new Thread(new Runnable() {
        public void run() {
            //code
        }
    }).start();

   new Thread(new Runnable() {
        //run method
   }).start();
}

希望这有助于您了解多线程服务器的工作原理！：）

PS：这是一个非常基本的服务器，实际上不应该用于大规模的应用程序。创建新线程（新的User（））会导致大量开销，因此最好使用ExecutorService或其他类型的线程池服务来处理连接。祝你好运！

Answer 2

我认为你应该重新思考很多服务器代码的逻辑。您应该有一个处理连接客户端的线程，并为它们分配自己的输入线程。然后，您可以在该输入读取线程上拥有一个方法，该方法将遍历客户端向其发送消息。

当我第一次开始编写客户端 - 服务器聊天程序时，我发现this小Oracle文档非常有用。希望它也能帮到你。

您应该特别注意文档的这一部分：

客户端连接请求在端口排队，因此服务器必须按顺序接受连接。但是，服务器可以通过使用线程同时为它们提供服务 - 每个客户端连接一个线程。

这种服务器的基本逻辑流程如下：

while (true) { accept a connection; create a thread to deal with the client; }

Java中的服务器客户端聊天室

2 个答案: