Неверное удаление пользователя из списка на стороне сервера - PullRequest
1 голос
/ 14 июня 2019

У меня есть небольшой примитивный сервер для изучения и на стороне клиента. Здесь у меня есть часть моего кода сервера:

 public class Connector implements Runnable, SocketListener {

    private Socket socket;
    private ServerSocket serverSocket;
    private List<ServerSideClient> clients = new LinkedList<>();
    private boolean triger;

    public Connector(ServerSocket serverSocket) {
        this.serverSocket = serverSocket;

    }

    @Override
    public void run() {
        while (true) {
            try {
                System.out.println("Waiting for clients..");
                triger = true;
                socket = serverSocket.accept();
                System.out.println("Client connected");
                ServerSideClient client = createClient();
                client.setConnection(true);
                client.startListeningClient();

                clients.add(client);

                new Thread(() -> {
                    socketIsClosed(client);
                }).start();

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

    private ServerSideClient createClient() {
        return new ServerSideClient(socket);
    }


    @Override
    public synchronized void socketIsClosed(ServerSideClient client) {
        while (triger == true) {
            if (client.isConnected() == false) {
                triger = false;
                clients.remove(client);
                System.out.println("Client was removed " + clients.size());
            }
        }
    }
 }

Здесь мы ждем нового Клиента, затем создаем экземпляр клиента и добавляем его в LinkedList. В случае со стороны сервера мы ожидаем информацию от клиента и отправляем ответ в отдельном потоке. Но когда клиент закрывает соединение с сервером, метод socketIsClosed() должен удалить текущую ссылку клиента из коллекции. Но когда клиент отключен, у меня даже нет выхода System.out.println("Client was removed " + clients.size()); из socketIsClosed(ServerSideClient client) метода.

Код клиента:

public class Client {

    private final String HOST = "localhost";
    private final int PORT = 1022;
    private InputStream inputStream;
    private OutputStream outputStream;
    private BufferedReader bufferedReader;
    private Socket socket;

    private boolean connection;

    public Client() throws IOException {
        socket = new Socket();
        socket.connect(new InetSocketAddress(HOST, PORT));
        inputStream = socket.getInputStream();
        outputStream = socket.getOutputStream();
        bufferedReader = new BufferedReader(new InputStreamReader(System.in));
    }


    public static void main(String[] args) {
        Client client = null;
        try {
            client = new Client();
            client.work();
        } catch (IOException e) {
            e.printStackTrace();
        }
    }

    private void work() {
        connection = true;
        listenForConsoleInput();
        receiveAnswerFromServer();
    }

    private void listenForConsoleInput() {
        new Thread(() -> {

            while (connection == true) {
                String requset = null;
                try {

                    requset = bufferedReader.readLine();
                    if (requset.equals(".")) {
                        closeConnection();
                        return;
                    } else {
                        sendRequest(requset);
                    }
                } catch (IOException e) {
                    e.printStackTrace();
                }
            }

        }).start();
    }

    private void sendRequest(String request) {
        try {
            outputStream.write(request.getBytes());
            outputStream.flush();
        } catch (IOException e) {
            e.printStackTrace();
        }
    }

    private void receiveAnswerFromServer() {
        new Thread(() -> {
            while (connection == true) {
                byte[] data = new byte[32 * 1024];
                try {
                    int numberOfBytes = inputStream.read(data);
                    System.out.println("Server>> " + new String(data, 0, numberOfBytes));
                } catch (IOException e) {
                    closeConnection();
                }
            }
        }).start();
    }

    private void closeConnection() {
        try {
            connection = false;
            socket.close();
            inputStream.close();
            outputStream.close();
            bufferedReader.close();
        } catch (IOException e) {
            e.printStackTrace();
        }
    }

}   

socketIsClosed(ServerSideClient client) метод работает в отдельном потоке.

public class ServerSideClient {

    private Socket socket;
    private InputStream in;
    private OutputStream out;

    private boolean connection;
    private int numOfBytes;

    public boolean isConnected() {
        return connection;
    }

    public void setConnection(boolean connection) {
        this.connection = connection;
    }

    public ServerSideClient(Socket socket) {
        this.socket = socket;
        try {
            in = socket.getInputStream();
            out = socket.getOutputStream();
        } catch (IOException e) {
            e.printStackTrace();
        }
    }

    public void startListeningClient() {
        new Thread(() -> {
            listenUsers();
        }).start();
    }

    private void listenUsers() {
        while (connection == true) {
            byte[] data = new byte[32 * 1024];

            readInputFromClient(data);
            if (numOfBytes == -1) {
                try {
                    connection = false;
                    socket.close();
                    in.close();
                    out.close();
                    isConnected();
                } catch (IOException e) {
                    e.printStackTrace();
                }
                System.out.println("Client disconected..");
                return;
            }
            String requestFromClient = new String(data, 0, numOfBytes);
            System.out.println("Client sended>> " + requestFromClient);


            sendResponce(requestFromClient);

        }

    }

    private void readInputFromClient(byte[] data) {
        try {
            numOfBytes = in.read(data);
        } catch (IOException e) {
            e.printStackTrace();
        }
    }

    private void sendResponce(String resp) {
        try {
            out.write(resp.getBytes());
            out.flush();
        } catch (IOException e) {
            e.printStackTrace();
        }
    }
}

Я пытаюсь решить эту проблему с 2 недели, Helllllllp .....

Ответы [ 2 ]

0 голосов
/ 16 июля 2019

Хорошо, я решил эту проблему с библиотекой javaRX. Я просто использовал событие, которое отправляет на сервер состояния наблюдателя. В наблюдаемом классе я создал:

private PublishSubject<Boolean> subject = PublishSubject.create();
    public Observable<Boolean> observable = subject.asObservable();


    public void setConnection(boolean connection) {
        this.connection = connection;
        subject.onNext(this.connection);
    }

Метод setConnection() установить true, если клиент был подключен, и false, если клиент инициализировал отключение.

В классе Observer я инициализировал экземпляр класса Observable и инициализировал подписку:

client = createClient();
client.observable.subscribe(state -> removeClient(state));



public void removeClient(Boolean state) {
        System.out.println("Server state " + state);
        if (state == false) {
            clients.remove(client);
            System.out.println("Client remowed. List size: " + clients.size());
        }
    }

Теперь я всегда знаю о состоянии сервера, и делаю удаление клиента, если последний инициализировал отключение.

0 голосов
/ 14 июня 2019

Мне удалось воспроизвести вашу проблему, и в качестве простого решения этой проблемы можно создать класс SocketClosedListener:

class SocketClosedListener implements Runnable {

     private final ServerSideClient client;
     private List<ServerSideClient> clients;

     public SocketClosedListener(ServerSideClient client, List<ServerSideClient> clients) {
        this.client = client;
         this.clients = clients;
     }

     @Override
     public void run() {
         while (true) {
             if (!client.isConnected()) {
                 clients.remove(client);
                 System.out.println("Client was removed " + clients.size());
                 return;
             }
             try {
                 Thread.sleep(100);
             } catch (InterruptedException e) {
                 e.printStackTrace();
             }
         }
     }
 }

И внутри вашего метода run () в классе Connector у нас есть этот вызов:

@Override
public void run() {
    while (true) {
        try {
            System.out.println("Waiting for clients..");
            triger = true;
            socket = serverSocket.accept();
            System.out.println("Client connected");
            ServerSideClient client = createClient();
            client.setConnection(true);
            client.startListeningClient();

            clients.add(client);

            new Thread(new SocketClosedListener(client, clients)).start();//added

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

Добавлена ​​строка:

new Thread(new SocketClosedListener(client, clients)).start();

Ответственный за поиск клиента при отключении в отдельном потоке.Также задержка в 100 мс, чтобы избежать проверки каждой мс, которая может вызвать проблемы при работе нескольких потоков.

С этим кодом я смог получить это в консоли:

Waiting for clients..
Client sended>> hi
Client disconected..
Client was removed 1
Client disconected..
Client was removed 0
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...