Как исправить этот код?

Question

public static int guess (int numberOfGuesses, int secretNumber) { 
    Scanner sc = new Scanner(System.in); 

    for(int n = 1; n <= numberOfGuesses; n++){ 
     System.out.println("Guess "+n+" of "+numberOfGuesses+":"); 
     int userNumber = sc.nextInt(); 
     if(userNumber == secretNumber){ 
     return n;  
     } 
     while(userNumber > secretNumber){ 
     System.out.println("The guess is too big"); 
     } 
     while(userNumber < secretNumber){ 
     System.out.println("The guess is too small"); 
     } 
     while(userNumber > 100){ 
     System.out.println("The number "+userNumber+" is not valid. "); 
     } 
    } 
    return -1; 
    } 
} 

Answer 1

Это довольно просто. Вам не нужно выходить из цикла, когда совпадают числа. Вы просто возвращаете n, как вам нужно. Если петля выходит без найденного secretNumber, вы возвращаете -1.

public static int guess (int numberOfGuesses, int secretNumber) { 

    Scanner sc = new Scanner(System.in); 

    for(int n = 1; n <= numberOfGuesses; n++){ 
     System.out.println("Guess "+n+" of "+numberOfGuesses+":"); 
     int userNumber = sc.nextInt(); 

     if(userNumber == secretNumber){ 
      return n;  
     } 
    } 
    return -1; 
} 

BTW, код имеет смысл, если вы дадите пользователю некоторые подсказки, которые помогут им угадать секретный номер.

Answer 2

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

public static int guess(int numberOfGuesses, int secretNumber) { 
     Scanner sc  = new Scanner(System.in); 
     int userNumber = secretNumber/2; 
     boolean found = false; 
     int tries; 
     for (tries = 0 ; tries <= numberOfGuesses && !found ; tries++) { 
      userNumber = sc.nextInt(); 
      found  = secretNumber == userNumber; 
     } 
     sc.close(); 
     return found ? tries : -1; 
    }