チャットプログラムサーバークラスはメッセージを送信しません

Question

私はこのサーバーコードをクライアントとサーバーのチャットプログラムから持っていますが、実行するとわかりました。たとえば、 "hello to 1"クライアント1に「hello」を送信しますが、文章を書きたい場合は「無効なメッセージ」が表示されます。私はしばらくそれを修正しようとしてきました。メッセージを得ることができる正しい方法を得ることができず、メッセージが送られるべきクライアントの番号が得られます。

public class Serverr implements Runnable {//server implements threading by implementing runnable interface 
Socket csocket; 
static HashMap<Integer,Socket>hm=new HashMap <Integer,Socket>(); //here we store each client socket in hashmap 
static int k=1; 
Serverr(Socket csocket){ 
    this.csocket=csocket;//Server ser=new Server(sock); comes here and assigns it to csocket 
} 

public static void main(String args[]) 
throws Exception{ 
    ServerSocket ssock=new ServerSocket(5000);//Socket on port 5000(same as mentioned inclient class) 
    System.out.println("Listening");//when connected on port 5000 we print listening 
    while(true){ 
     Socket sock=ssock.accept();//accept client socket 
     Serverr ser=new Serverr(sock);//pass socket to constructor of server class 
     new Thread(ser).start();//start thread here 
     hm.put(k,sock);//add the socket into hashmap 
     System.out.println("Connected to client"+k);//print the number of connected client 
     k++; 
    } 


} 

@Override 
public void run() { 
    try{ 
     //once the thread is started we initialize necessary input & output streams here 
     PrintWriter out; 
     BufferedReader in=new BufferedReader(new InputStreamReader(csocket.getInputStream()));//used to read input from the socket 
     String inputLine; 
     String j="sending"; //follow a pattern for sending messages 
     String l="to"; 
     String t="sendtoall";//used to sent message to all clients 
     while((inputLine=in.readLine())!=null)//if the message from connected client !=null 
     { 
      String a[]=inputLine.split(" ");//split the input line using space if a[0] is sendmsg and a[2] is to 
      if(a[0].equals(j) && a[2].equals(l))//we started the server 
      { 
       int id=Integer.parseInt(a[3]);//we will get the number of client here 
       if(hm.containsKey(id)){// we will check hashmap if it contains id 
        Socket ser1=hm.get(id);//here we will get that clients socket from hashmap 

        out=new PrintWriter(ser1.getOutputStream(),true);// we will write to that socket using output stream 
        out.println(a[1]);//we will get that specific message part 
        out.flush();// a[1] is... 
       } 
       else 
       { 
        out=new PrintWriter(csocket.getOutputStream(),true); 
        out.println("user offline");//we print it if hashmap doesnt contain the key value 
        out.flush(); 
       } 
      } 
      else if(a[0].equals(t))//if we want to sent message to all clients at once 
      { 
       for(int h=1;h<hm.size();h++)//loop through hashmap & get those socket value 
       { 
        Socket ser1=hm.get(h); 

        out=new PrintWriter(ser1.getOutputStream(),true);//we will write to that socket here using outputstream 
        out.println(a[1]); 
        out.flush(); 
       } 
      } 
      else{ 
       out=new PrintWriter(csocket.getOutputStream(),true); 
       out.println("Invalid Message");//if the message format doesnt match 
       out.flush(); 
      } 
     } 
    }catch(IOException e){ 
     System.out.println(e); 
    } 
} 
} 

Answer 1

この2行は問題です。

 if(a[0].equals(j) && a[2].equals(l))//we started the server 
     { 
      int id=Integer.parseInt(a[3]);//we will get the number of client here 

"to"とクライアント番号の入力の3番目と4番目の単語を確認しています。しかし、あなたが完全な文を持っているならば、これらは入力の第三と第四の言葉ではありません。代わりに、おそらく2番目と最後の単語をそれぞれチェックすることができます。あなたは

sending Hello my friend to 12345 

ような入力を持っており、正しくフィールドを識別することができます

 if(a[0].equals(j) && a[a.length - 2].equals(l))//we started the server 
     { 
      int id=Integer.parseInt(a[a.length - 1]);//we will get the number of client here 

。

また、それだけで一つの言葉以上のものを送信するようにライン

out.println(a[1]); 

を変更する必要があります。おそらく、これを行うためにループを使用することができます。

for (int word = 1; word < a.length - 2; word++) { 
    out.print(a[word]); 
} 
out.println();