IRC логин пропущен - PullRequest
Новая
       11

IRC логин пропущен

0 голосов
/ 16 мая 2018

Мне нужно сделать IRC-клиент, и у меня проблема с логином.Я пытаюсь перебрать ввод с сервера, чтобы найти массаж 004 или 433, в котором говорится об успешном входе в систему или использовании входа в систему.Это мой код: `

import ecs100.*;
import java.io.*;
import java.net.Socket; 
import java.util.*; 



public class ChatClient {
private String server = "irc.ecs.vuw.ac.nz";  
private static final int IRC_PORT = 6667;     



private static String nick; 
private static String name;
private Socket socket;
private Scanner serverIn;
private PrintStream serverOut; 
private String line; 
private String channel; 
// Constructor
public ChatClient (){
    this.setupGUI();

}

public void setupGUI(){ 
    UI.addButton("Connect", this::connect);


}



public void connect(){

    try {

    socket = new Socket(server,IRC_PORT); 
    serverIn = new Scanner(socket.getInputStream());
    serverOut = new PrintStream(socket.getOutputStream());
    UI.println("login");
    this.login();
    UI.println("listening");
    this.listenToServer(); 
}catch(IOException e){UI.println("IO failure "+ e);}

}

private boolean login(){
    String username = UI.askToken("Enter your usercode: ");
    String realname = UI.askString("Enter your real name: ");

    this.nick = username;
    this.name = realname;
    send("NICK " + username);
    send("USER " + username + " 0 unused :" + realname);
    UI.println("listening to server");
   while(this.serverIn.hasNext()){
    this.line = serverIn.nextLine();
    UI.println("SERVER" + line);
    if (line.indexOf("004") >= 0){
        UI.println("true");
        return true;
    } else if(line.indexOf("433") >= 0){
        UI.println("failed to connect, username already in use");
        return false;
    } else {
        UI.println("else");
        return false;
    }
}

}

private void send(String msg){

      serverOut.print(msg + "\r\n");

    serverOut.flush();

}


private void listenToServer() {

    while(this.serverIn.hasNext()){
        String line = serverIn.nextLine();
        UI.println("SERVER" + line);
        if(line.equals("SQUIT")){
            this.closeConnection();
        }
        if(line.equals("PING")){
          String pingMsg = line.split(" ",2)[1];  
          this.send("PONG " + pingMsg);  
        }
    }
}


public void closeConnection(){

    try {
    this.socket.close();
    this.serverIn.close();
    this.serverOut.close();
    UI.println("Closed");
} catch(IOException e){UI.println("IO failure "+ e);}
}

public void joinChannel(){
    String channel = UI.askString("Enter channel name: ");
    send("JOIN " + channel);
}
public void leaveChannel(){
    send("PART " + channel);
}

public static void main(String[] args) {
    new ChatClient();
}

}

Проблема в том, что он продолжает выдавать пропущенную инструкцию возврата в методе входа в систему.Может кто-нибудь, пожалуйста, посмотрите и мой код и посмотреть, что здесь может быть не так?

РЕДАКТИРОВАТЬ: хорошо, поэтому, когда я изменяю оператор if на два if и помещаю return false вне цикла, программа работает.Может кто-нибудь объяснить, почему оператор if / else внутри цикла while не работает?

  private boolean login(){
    String username = UI.askToken("Enter your usercode: ");
    String realname = UI.askString("Enter your real name: ");
    this.nick = username;
    this.name = realname;
    send("NICK " + username);
    send("USER " + username + " 0 unused :" + realname);
    UI.println("listening to server");
   while(this.serverIn.hasNext()){
    this.line = serverIn.nextLine();
    UI.println("SERVER" + line);
    if (line.indexOf("004") >= 0){
        UI.println("true");
        return true;
    } 
    if(line.indexOf("433") >= 0){
        UI.println("failed to connect, username already in use");
        return false;
    } 
   }
   UI.println("else");
   return false;
}
enter code here

1 Ответ

0 голосов
/ 16 мая 2018

Поскольку есть путь к коду, который не приведет к вызову return, и компилятор не может допустить этого для не пустого метода.

Если this.serverIn.hasNext() вернет false, что будет возвращать метод, если у вас нет оператора return в конце? Он должен вернуть boolean в конце концов.

Логика программы может быть написана так, что конкретный путь к коду никогда не берется, но теоретически это возможно, и поэтому компилятор жалуется.

...