이 코드를 수정하는 방법?

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; 
} 

동의어는 사용자가 비밀 번호를 추측하는 데 도움이되는 몇 가지 힌트를 사용자에게 제공하면 더 적절합니다.

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; 
    }