зачем удалять пользователя?

Вопрос по задаче

Java Multithreading, 6 уровень, 15 лекция

Решен

Зачем удалять только что добавленного пользователя? (п.9)

Пришло время написать главный метод класса Handler, который будет вызывать все вспомогательные методы, написанные ранее.
Реализуем метод void run() в классе Handler.

Он должен:
1. Выводить сообщение, что установлено новое соединение с удаленным адресом, который можно получить с помощью метода getRemoteSocketAddress().
2. Создавать Connection, используя поле socket.
3. Вызывать метод, реализующий рукопожатие с клиентом, сохраняя имя нового клиента.
4. Рассылать всем участникам чата информацию об имени присоединившегося участника (сообщение с типом USER_ADDED).
Подумай, какой метод подойдет для этого лучше всего.
5. Сообщать новому участнику о существующих участниках.
6. Запускать главный цикл обработки сообщений сервером.
7. Обеспечить закрытие соединения при возникновении исключения.
8. Отловить все исключения типа IOException и ClassNotFoundException, вывести в консоль информацию, что произошла ошибка при обмене данными с удаленным адресом.
9. После того как все исключения обработаны, если п.11.3 отработал и возвратил нам имя, мы должны удалить запись для этого имени из connectionMap и разослать всем остальным участникам сообщение с типом USER_REMOVED и сохраненным именем.
10. Последнее, что нужно сделать в методе run() - вывести сообщение, информирующее что соединение с удаленным адресом закрыто.

Наш сервер полностью готов. Попробуй его запустить.

Комментарии (7)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

Алексей ТонкихBackend Developer в Сбербанк

10 февраля 2021, 17:48

Так там нужно реализовать алгоритм жизни нити - создать соединение, получить имя нового пользователя, оповестить всех, потом основной бесконечный цикл serverMainLoop ("общение пользователя"). Это всё обёрнуто в перехват ошибок, которые все методы пробрасывают из себя. А когда основной цикл "общения пользователя" выбросит ошибку (или до него вылетит ошибка, например, соединение с этим пользователем разорвётся), то нужно пользователя удалить, если он был конечно добавлен перед этим успешно.

Илья

Уровень 30

11 февраля 2021, 16:45

Вначале я так и подумал и даже добавил удаление пользователя из мапы в блок catch, но валидатор не принял. Подсмотрел решение валидатора и переделал на:

public void run() {
            ConsoleHelper.writeMessage("Установлено новое соединение с удаленным адресом " +
                    socket.getRemoteSocketAddress());

            String newUser = null;

            try (Connection connection = new Connection(socket)) {

                newUser = serverHandshake(connection);

                notifyUsers(connection, newUser);

                sendBroadcastMessage(new Message(MessageType.USER_ADDED, newUser));

                serverMainLoop(connection, newUser);

            } catch (IOException | ClassNotFoundException e) {
                ConsoleHelper.writeMessage("Произошла ошибка при обмене данными с удаленным сервером, попробуйте снова.");
            }

            if (newUser != null) {
                connectionMap.remove(newUser);
                sendBroadcastMessage(new Message(MessageType.USER_REMOVED, newUser));
            }

            ConsoleHelper.writeMessage("Соединение с удаленным сервером закрыто");
        }

Т.е. получается, что удаление пользователя произойдёт в любом случае.. Или я что-то не так понимаю?

Алексей ТонкихBackend Developer в Сбербанк

11 февраля 2021, 17:33

Да, в конце пользователь должен в любом случае удаляться, если был создан.

Илья

Уровень 30

11 февраля 2021, 18:22

Тогда логика программы такая: 1. присоединился новый пользователь (newUser = serverHandshake(connection);) 2. произошло оповещение всех пользователей об этом событии (notifyUsers(connection, newUser);) 3. отправлено сообщение всем пользователям о присоединении нового (sendBroadcastMessage(new Message(MessageType.USER_ADDED, newUser));) // здесь, кстати тоже не понятно, зачем нужен этот метод, т.к. он по сути дублирует предыдущий.. 4. вновь присоединившийся пользователь отправил сообщение (serverMainLoop(connection, newUser);) 5. пользователь удалился (connectionMap.remove(newUser);) Не понимаю почему пользователь должен удалиться после отправки сообщения?

Алексей ТонкихBackend Developer в Сбербанк

11 февраля 2021, 19:08

решение

2. notifyUsers (находится внутри класса Handler) - выводит присоединившемуся пользователю по его connection список пользователей в чате (выглядит это, как будто они все присоединились, когда новый пользователь подключился). 3. sendBroadcastMessage (находится в Server) - оповещает всех пользователей (кроме нового) по их connection согласно connectionMap, что присоединился новый пользователь. 4. serverMainLoop - содержит бесконечный цикл, который завершится если пользователь сам выйдет (наберет exit) или отвалится. 5. Вот когда закончится serverMainLoop, тогда удаляется пользователь.

Илья

Уровень 30

12 февраля 2021, 15:48

Теперь ясно) Спасибо!

Даниил Александрович

Уровень 35

9 февраля 2021, 20:17

Возможно, чтобы закрепить материал и использовать все методы:

который будет вызывать все вспомогательные методы, написанные ранее.