Вот реализация метода void run() в классе Client. По логике все должно работать.
public void run(){
// SocketThread thread = client.getSocketThread();
SocketThread thread = getSocketThread();
thread.setDaemon(true);
thread.start();
try {
synchronized (this) {
this.wait();
}
} catch (InterruptedException e) {
e.printStackTrace();
System.out.println("При работе клиента возникла ошибка");
}
if (clientConnected) sendTextMessage("Соединение установлено. Для выхода наберите команду ‘exit’.");
else System.out.println("Произошла ошибка во время работы клиента.");
while(clientConnected) {
String line = ConsoleHelper.readString();
if (line.equals("exit")) break;
if (shouldSendTextFromConsole()) sendTextMessage(line);
}
}
Но валик говорит, что
Если метод shouldSentTextFromConsole возвращает true, должен быть вызван метод sendTextMessage со считанным текстом в качестве параметра.
и дает совет
В случае, если метод shouldSendTextFromConsole возвращает false, метод sendTextMessage не должен быть вызван.
Прошу помощи, как ублажить валик?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 {
int portNumber = ConsoleHelper.readInt();
serverSocket = new ServerSocket(portNumber);
System.out.printf("Сервер запущен на порту %d", 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()));
else System.out.println(" Ошибка. Неверный тип принятого сообщения");
}
}
}
}