Java服务器结构

Question

目前我的java服务器工作正常，将所有客户端存储在一个新线程上，但性能不太好。

我的 MainServer 类

public class MainServer {

    private ServerSocket server;

    public MainServer(int port) {

        try {

            server = new ServerSocket(port);
            System.out.println("Server started on port " + port + "...");
            ServerLoop();

        } catch (IOException e) {

            System.out.println("[ERROR] Server cannot be started on port " + port + "...");
        }
    }

    public void ServerLoop() {

        Thread serverLoop = new Thread(new Runnable() {

            public void run() {

                ServerDispatcher dispatcher = new ServerDispatcher();

                while (true) {

                    try {

                        Socket connectedClient = server.accept();
                        dispatcher.connectedClients.add(connectedClient);
                        System.out.println("Client number - " + dispatcher.connectedClients.size() + " connected...");
                        Thread clientThread = new Thread(new ClientHandler(connectedClient, dispatcher));
                        clientThread.start();

                    } catch (IOException e) {

                        e.printStackTrace();
                    }
                }
            }
        });

        serverLoop.start();
    }
}

因此这部分将为每个客户端启动一个新线程并将客户端添加到调度程序客户端列表中

ClientHandler类

public class ClientHandler implements Runnable {

    private Socket currentClient;
    private ServerDispatcher handler;

    public ClientHandler(Socket s, ServerDispatcher dispatcher) {

        currentClient = s;
        handler = dispatcher;
    }

    public void run() {



        while (true) {


        }


    }
}

这是我的 ServerDispatcher 类

public class ServerDispatcher {

    public ArrayList<Socket> connectedClients;

    public ServerDispatcher() {

        connectedClients = new ArrayList<Socket>();
        CheckClients();
    }

    public void CheckClients() {

        Thread checkClients = new Thread(new Runnable() {

            public void run() {

                while (true) {

                    Integer clientSize = connectedClients.size();

                    if (!clientSize.equals(0)) {

                        for (int i = 0; i < connectedClients.size(); i++) {

                            try {

                                BufferedWriter writer = new BufferedWriter(new OutputStreamWriter(connectedClients.get(i).getOutputStream()));
                                writer.write("1");
                                writer.flush();
                                //System.out.println("Checking");

                            } catch (IOException e) {

                                int clientNumber = i + 1;
                                System.out.println("Client number - " + clientNumber + " disconnected...");
                                connectedClients.remove(i);
                            }
                        }
                    }
                }
            }
        });

        checkClients.start();
    }
}

这会检查所有在线客户端...

问题是该服务器在执行时使用了大约 67% 的 CPU，我该如何改进代码？也许从 ServerDispatcher 中删除所有线程并只保留客户端线程？

还有一个问题:我想在 while 循环上向 X 客户端发送一条消息，但只有一次，我在 google 上查看，似乎 LinkedBlockingQueue 就是我正在寻找的东西，但我找不到任何简单的教程实现..

Answer 1

您的循环只是尽可能快地发送数据，而在无法发送数据时则忙于等待。这势必会使用大量的CPU。一个简单的解决方案是仅定期发送虚拟数据(因为它根本不需要发送)

我建议添加

Thread.sleep(100);

到发送循环，您的 CPU 应该下降到 1% 左右。

您还可以进行许多其他改进，但没有一个比这更重要。