Проблема с двумя методами. Диагноз валидатора:
В классе ClientGuiModel должен быть создан корректный геттер для поля allUserNames.
рекомендация: В классе ClientGuiModel некорректно реализован или отсутствует публичный геттер для поля allUserNames.
Метод addUser должен добавлять новое имя пользователя в множество allUserNames.
Вот реализация методов:
public Set<String> getAllUserNames() {
return Collections.unmodifiableSet(getAllUserNames());
}
public void addUser(String newUserName) {
allUserNames.add(newUserName);
}package com.javarush.task.task30.task3008;
import java.io.IOException;
import java.net.ServerSocket;
import java.net.Socket;
import java.util.Map;
import java.util.concurrent.ConcurrentHashMap;
/**
* Created by Rea1 on 05.10.2017.
*/
public class Server {
private static ServerSocket serverSocket;
private static Map<String, Connection> connectionMap = new ConcurrentHashMap<>();
public static void main(String[] args) {
try {
System.out.println("Введите номер порта");
int portNumber = ConsoleHelper.readInt();
serverSocket = new ServerSocket(portNumber);
System.out.printf("Сервер запущен на порту %d\n", portNumber);
while (true) {
Socket socket = serverSocket.accept();
Handler handler = new Handler(socket);
handler.start();
}
} catch (IOException e) {
try {
serverSocket.close();
}
catch (Exception e1) { e1.printStackTrace(); }
}
}
public static void sendBroadcastMessage(Message message) {
for (Map.Entry<String, Connection> entry : connectionMap.entrySet()) {
try {
entry.getValue().send(message);
}
catch (IOException e) {
Message errorMessage = new Message(MessageType.TEXT,"Сообщение не может быть отправлено" );
try {
entry.getValue().send(errorMessage);
} catch (IOException e1) {
e1.printStackTrace();
}
}
}
}
private static class Handler extends Thread {
private Socket socket;
public Handler (Socket socket) {
this.socket = socket;
}
@Override
public void run() {
System.out.println("Установлено новое соединение с удаленным адресом " + socket.getRemoteSocketAddress());
try
{
Connection connection = new Connection(socket);
String name = serverHandshake(connection);
sendBroadcastMessage(new Message(MessageType.USER_ADDED, name));
// for (Map.Entry<String, Connection> entry : connectionMap.entrySet()) {
// entry.getValue().send(new Message(MessageType.USER_ADDED, name));
// }
sendListOfUsers(connection, name);
serverMainLoop(connection, name);
connectionMap.remove(name);
sendBroadcastMessage(new Message(MessageType.USER_REMOVED, name));
}
catch(IOException e1) {
System.out.println("Произошла ошибка при обмене с данными");
}
catch(ClassNotFoundException e2){
System.out.println("Произошла ошибка при обмене с данными");
}
}
private String serverHandshake(Connection connection) throws IOException, ClassNotFoundException {
Message reciveMessage;
while (true ) {
connection.send(new Message(MessageType.NAME_REQUEST));
reciveMessage = connection.receive();
if (reciveMessage.getType() != MessageType.USER_NAME
|| reciveMessage.getData().isEmpty()
||connectionMap.containsKey(reciveMessage.getData())) continue;
else {
connectionMap.put(reciveMessage.getData(), connection);
connection.send(new Message(MessageType.NAME_ACCEPTED));
break;}
}
return reciveMessage.getData();
}
private void sendListOfUsers(Connection connection, String userName) throws IOException {
for (Map.Entry<String, Connection> entry : connectionMap.entrySet()) {
if (entry.getKey()!= userName) connection.send(new Message(MessageType.USER_ADDED, entry.getKey()));
}
}
private void serverMainLoop(Connection connection, String userName) throws IOException, ClassNotFoundException {
while (true) {
Message message = connection.receive();
String textMessage;
if (message.getType() == MessageType.TEXT)
sendBroadcastMessage(new Message(MessageType.TEXT, textMessage = userName + ": " + message.getData()));
// эксперимент
if (message.getType() == MessageType.USER_REMOVED)
break;
//
else System.out.println(" Ошибка. Неверный тип принятого сообщения");
}
}
}
}