socket send and retrieve

Question

Добрый день, все, что я новичок в java, и я хотел бы знать, может ли кто-нибудь помочь мне с этой проблемой. У меня есть сервер, и он получает информацию от клиента, но оператор if, чтобы проверить значение, которое был принят не работает.

Вот мой код для сервера.

Session(Socket s){ 
     soc = s; 
     try{ 
      br = new BufferedReader(new InputStreamReader(soc.getInputStream())); 

      pw = new PrintWriter(new BufferedOutputStream(soc.getOutputStream()),true); 
      pw.println("Welcome");   
     }catch(IOException ioe){ 
      System.out.println(ioe); 
     } 


     if(runner == null){ 
      runner = new Thread(this); 
      runner.start(); 
     } 
    } 

    public void run(){ 
     while(runner == Thread.currentThread()){ 
      try{ 
       String input = br.readLine().toString(); 
        if(input != null){ 
         String output = Protocol.ProcessInput(input); 
         pw.println(output); 
         System.out.println(input); 


         if(output.equals("Good Bye")){ 
          runner = null; 
          pw.close(); 
          br.close(); 
          soc.close(); 
         } 
       **This if statement doesn't work ↓** 
         if(Protocol.ProcessInput(input).equalsIgnoreCase("tiaan")){ 
          // System.exit(0); 
          System.out.println("Got tiaan!!!"); 
         } 
        } 

      }catch(IOException ie){ 
       System.out.println(ie); 
      } 
      try{ 
       Thread.sleep(10); 
      }catch(InterruptedException ie){ 
       System.out.println(ie); 
      } 
     } 
    } 


} 

class Protocol{ 
    static String ProcessInput(String input){ 
     if(input.equalsIgnoreCase("Hello")){ 
      return "Well hello to you to"; 
     }else{ 
      return "Good bye"; 
     } 
    } 
} 

Answer 1

Ok. Давайте посмотрим на это, если заявление:

if(Protocol.ProcessInput(input).equalsIgnoreCase("tiaan")){ 
    // System.exit(0); 
    System.out.println("Got tiaan!!!"); 
} 

Этот код эквивалентен следующему:

String output = Protocol.ProcessInput(input) 
if(output.equalsIgnoreCase("tiaan")){ 
    // System.exit(0); 
    System.out.println("Got tiaan!!!"); 
} 

Так выход из ProcessInput сравнивается со строкой «Tiaan» и смотрит на ProcessInput показывает, что он никогда не вернет эту строку. Так что, может быть, вы на самом деле хотите сделать что-то еще, например, сравнить вход непосредственно с «Tiaan» или изменить реализацию ProcessInput:

if(input.equalsIgnoreCase("tiaan")){ 
    // System.exit(0); 
    System.out.println("Got tiaan!!!"); 
} 

Заметьте, что вы можете получить NullPointerException, когда вы читаете вход:

//Change this: 
String input = br.readLine().toString(); 
//Into this: 
String input = br.readLine(); 

readLine уже дает вам строку, поэтому вам не нужно toString в конце. Если readLine дает вам значение null, которое оно делает, когда вы достигаете конца потока, тогда вызов toString вызовет исключение NullPointerException. На следующей строке вы действительно проверяете, является ли вход нулевым, но это полезно, но с использованием кода произойдет ошибка до этой проверки.