Question

我正在创建一个处理多个客户端的java聊天服务器我使用这个简单的服务器代码

public class Server extends Thread {
    ServerSocket serverSocket = null;
    Socket socket = null;
    private int unique_id;
    ArrayList<Clients> cl;


    public Server(int port) {
        try { 
            serverSocket = new ServerSocket(port);
            cl = new ArrayList<>();
            this.start();
        } catch (Exception e){
            System.out.println("Error 5");
            e.printStackTrace();
        }
    }
    @Override
    public void run(){
        System.out.println("Server Start");
            while (true){
                try {
                    socket = serverSocket.accept();
                    Clients t = new Clients(socket); // add it to thread
                    cl.add(t);
                    t.start();
                    System.out.println("Connected " + String.valueOf(cl.size())); // printed ok 
                }catch (Exception e){
                    System.out.println("Error 4");
                    e.printStackTrace();
                }
            }
    }
    public synchronized void SendToAll(String s){ // this function used by client when one of client socket send a message then server send it to all
        System.out.println("Sended is excuted"); // excuted normal each time i send a message from client but not send to all  
        for (int i = 0; i < cl.size(); i++){
            cl.get(i).WriteToSocket(s);
        }
    }

    public static void main(String args[]){
        int port = 5002;
        Server server = new Server(port); // start server
        //server.run(); // start connections wait for it
    }

    class Clients extends Thread { // class extends thread
        public Socket socket = null;
        DataInputStream input = null; // read input
        DataOutputStream output = null; // read output
        public int myid = 0; // unique id for each client

        public Clients(Socket soc) {
            socket = soc;
            try {
                input = new DataInputStream(socket.getInputStream());
                output = new DataOutputStream(socket.getOutputStream());
                myid = ++unique_id;
                System.out.println("Client Start Thread"); // printed ok !
            } catch (IOException e){
                System.out.println("Error 1");
                e.printStackTrace();
            }
        }
        public void WriteToSocket(String s) { // used to write a message to this socket
            try {
                output.write(s.getBytes());
            }catch (IOException e){
                System.out.println("Error 2");
                e.printStackTrace();
            }
        }
        @Override
        public void run() { // run thread function wait for messages from clients 
            while (true){
                try {
                    String s = input.readLine();
                    if (s.contains("quite")) {
                        socket.close();
                        input.close();
                        output.close();
                        cl.remove(this);
                        this.stop();
                    }
                    if (!s.isEmpty()) {
                        SendToAll(s);// when message come and not empty it use server function to send them to all clients 
                    }
                }catch (IOException e){
                    System.out.println("Error 3");
                    e.printStackTrace();
                }
            }
        } 
    }
}

当客户端连接服务器接受连接并且客户端线程启动时，一切正常但问题是当我从客户端发送消息时它没有收到服务器我用Qt c ++服务器在java中尝试我的客户端应用程序并且它有效吗？那么我做错了什么让服务器无法收到消息？这是我第一次使用java进行网络编程修改
我解决了 NullPointerException 问题是当客户端注销时我没有从ArrayList中删除他的套接字，通过在关闭发送消息之前使客户端解决包含当我看到我从阵列列表中删除了他的套接字
另一个Quetiosn
这里我不知道这条消息是如何发送的那样是System.out.println（）每次客户端发送消息时，SendToAll函数都会打印到屏幕上，但为什么消息不会再次发送给所有客户端？
实际上主要问题是服务器无法将消息发送到阵列列表中的所有客户端后，来自一个客户端的消息问题未解决stell发现
客户端代码类

public class ClientSocket extends Thread {
    public Socket socket = null;
    public DataInputStream input = null;
    public DataOutputStream output = null;
    MainChat chat = null;

    public ClientSocket(String ip, int port,MainChat ch) {
        try {
            socket = new Socket(ip,port);
            input = new DataInputStream(socket.getInputStream());
            output = new DataOutputStream(socket.getOutputStream());
            chat = ch;
            this.start();
        }catch (IOException e){

        }
    }
    @Override
    public void run() {
        while (true){
            try {
                String s = input.readLine();
                if (!s.isEmpty()){
                   chat.WriteToScreen(s.trim());
                }
            }catch (IOException e){

            }
        }
    }
    public void WriteToSocket(String s) throws IOException{
        output.write(s.getBytes());
    }

}

编辑
当我在main中使用此代码时，SendToAll函数将消息发送给所有客户端！为什么当我使用Thread从客户端类中使用它时却没有发送给所有人？

public static void main(String args[]){
    int port = 5002;
    Server server = new Server(port); // start server
    //server.run(); // start connections wait for it
    while (true) {
        String s = in.next();
        server.SendToAll(s + "\n"); // message sended to all client !!
    }
}

Answer 1

问题是readLine读取，直到找到文件结尾的行终止符。这就是为什么它可以与QT C ++中的其他服务器一起使用，但不能与Java服务器一起使用。请看这里： https://docs.oracle.com/javase/7/docs/api/java/io/DataInput.html#readLine() https://docs.oracle.com/javase/7/docs/api/java/io/DataInputStream.html#readLine()

请注意，不推荐使用DataInputStream中的readLine。您应该使用BufferedReader来读取DataInputStream link中指示的一行（使用readLine）。

所以，添加＆＃39; \ n＆＃39;发送到字符串的末尾，它将起作用。

Answer 2

我解决了这个问题，我很抱歉这是我的错，我忘记在sendToAll函数中添加 \ n 所以这就是导致问题的原因所以没有\ n客户端无法解决读取行，因为我在DataInputStream中使用readLine
无论如何，我尝试另一种方法来读取字节而不是readLine它我认为它更好，特别是当你收到UTF-8字符并且之后从字节变为字符串

使用java进行简单的多线程服务器聊天

2 个答案: