TCP Chat Server

Question

Я изменяю учебник, который я нашел о том, как создать TCP Chat Server, который принимает несколько клиентов. В конечном итоге я также создам клиентский класс, но пока я тестирую его с TELNET.

Я хочу, чтобы сервер постоянно проверял ввод, чтобы я мог использовать ключевые слова для предварительной работы функций сервера. Так что строковое слово «EXIT» для отключения клиента и строковое слово «Name:» для печати «OK».

Это то, о чем я думал, но это не работает:

 public void run()  
    {
         String line;
         try    
         {
            while(true)   
            {
                if (input.readline("EXIT"))//Should close and remove client
                {
                    clients.remove(this);
                    users.remove(name);
                    break;
                }
                if(input.readline("Name:"))//Should print OK with username
                {
                    System.out.println("OK");
                }
                boradcast(name,line); // method  of outer class - send messages to all
            }// end of while
         } // try
         catch(Exception e) 
         {
           System.out.println(e.getMessage());
         }
    } // end of run()
 }

}

Вот весь класс сервера

// Chat Server runs at port no. 9020
import java.io.*;
import java.util.*;
import java.net.*;
import static java.lang.System.out;

public class  TCPServer 
{
  Vector<String> users = new Vector<String>();
  Vector<HandleClient> clients = new Vector<HandleClient>();

  int PORT = 9020;
  int NumClients = 10;

  public void process() throws Exception  
  {
      ServerSocket server = new ServerSocket(PORT,NumClients);
      out.println("Server Connected...");
      while( true) 
      {
         Socket client = server.accept();
         HandleClient c = new HandleClient(client);
         clients.add(c);
     }  // end of while
  }

  public static void main(String ... args) throws Exception 
  {
      new TCPServer().process();
  } // end of main

  public void boradcast(String user, String message)  
  {
        // send message to all connected users
        for (HandleClient c : clients)
           if (!c.getUserName().equals(user))
           {
              c.sendMessage(user,message);
           }
  }

  class HandleClient extends Thread 
  {
    String name = "";
    BufferedReader input;
    PrintWriter output;

    public HandleClient(Socket client) throws Exception 
    {
          // get input and output streams
         input = new BufferedReader(new InputStreamReader(client.getInputStream())) ;
         output = new PrintWriter (client.getOutputStream(),true);
         output.println("Welcome to Bob's Chat Server!\n");
         // read name
         output.println("Please Enter a User Name: ");
         name  = input.readLine();
         users.add(name); // add to vector
         output.println("Welcome "+name+" we hope you enjoy your chat today");
         start();
    }

    public void sendMessage(String uname,String  msg)  
    {
        output.println( uname + ":" + msg);
    }

    public String getUserName() 
    {  
        return name; 
    }

    public void run()  
    {
         String line;
         try    
         {
            while(true)   
            {
                if (input.readline("EXIT"))
                {
                    clients.remove(this);
                    users.remove(name);
                    break;
                }
                if(input.readline(name))
                {
                    System.out.println("OK");
                }
                boradcast(name,line); // method  of outer class - send messages to all
            }// end of while
         } // try
         catch(Exception e) 
         {
           System.out.println(e.getMessage());
         }
    } // end of run()
  } // end of inner class
} // end of Server

Answer 1

Не зная точно, что вы хотите сделать, когда строка ввода равна текущему имени пользователя, я думаю, это больше, чем вы ищете:

    public void run(){
        try{
            while(true){
                String line = input.readLine();

                if("EXIT".equals(line)){
                    clients.remove(this);
                    users.remove(name);
                    break;
                }else if(name.equals(line)){
                    System.out.println("OK");
                }
                boradcast(name, line); // method  of outer class - send messages to all
            }// end of while
        } // try
        catch(Exception e){
            System.out.println(e.getMessage());
        }
    } // end of run()

Это решает несколько проблем:

• input.readline - это не метод, но input.readLine - и он не принимает никаких параметров. (Это должно было отображаться как ошибка компиляции.)
• Вы никогда не назначали ничего для line строки.
• Вы читали строку несколько раз. Если он не соответствует «EXIT», вы читаете новую строку для сравнения с name - теряя все, что пользователь ввел для предыдущей строки.