Сканер никогда не закрывается

Question

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

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

import java.util.Scanner; 

public class Main { 

    public static final boolean CHEAT = true; 

    public static void main(String[] args) { 

     Scanner scanner = new Scanner(System.in); 
     int amountOfPlayers; 
     do { 
      System.out.print("Select the amount of players (1/2): "); 
      while (!scanner.hasNextInt()) { 
       System.out.println("That's not a number!"); 
       scanner.next(); // this is important! 
     } 

     amountOfPlayers = scanner.nextInt(); 
     while ((amountOfPlayers <= 0) || (amountOfPlayers > 2)); 
     System.out.println("You've selected " + amountOfPlayers+" player(s)."); 
    } 
} 

Answer 1

Я предполагаю, что вы используете Java 7, таким образом, вы получите предупреждение компилятора, если не закрывать ресурс, который вы должны закрыть сканер обычно в конце концов блока.

Scanner scanner = null; 
try { 
    scanner = new Scanner(System.in); 
    //rest of the code 
} 
finally { 
    if(scanner!=null) 
     scanner.close(); 
} 

Или еще лучше: использовать новый Try with resource statement:

try(Scanner scanner = new Scanner(System.in)){ 
    //rest of your code 
} 

Answer 2

Попробуйте

Scanner scanner = new Scanner(System.in); 
int amountOfPlayers; 
do { 
    System.out.print("Select the amount of players (1/2): "); 
    while (!scanner.hasNextInt()) { 
     System.out.println("That's not a number!"); 
     scanner.next(); // this is important! 
    } 

    amountOfPlayers = scanner.nextInt(); 
} while ((amountOfPlayers <= 0) || (amountOfPlayers > 2)); 
if(scanner != null) { 
    scanner.close(); 
} 
System.out.println("You've selected " + amountOfPlayers+" player(s)."); 

Answer 3

Согласно Javadoc Сканера, она закрывает поток, когда вы называете это близко метод. Вообще говоря, код, создающий ресурс, также отвечает за его закрытие. System.in не был создан вашим кодом, а VM. Поэтому в этом случае безопасно не закрывать сканер, игнорировать предупреждение и добавлять комментарий, почему вы его игнорируете. VM при необходимости позаботится о его закрытии.

(Offtopic: вместо «amount» слово «number» было бы более подходящим для использования для нескольких игроков. Английский не является моим родным языком (я голландский), и я делал то же самое ошибка.)

Answer 4

Вот некоторые лучше использование Java для сканера

try(Scanner sc = new Scanner(System.in)) { 

    //Use sc as you need 

} catch (Exception e) { 

     // handle exception 

}