Scanner n'est jamais fermée

Je suis en train de travailler sur un jeu et je suis tombé sur un petit problème avec mon scanner.
J'ai une fuite de ressources scanner jamais fermé.

Mais je pensais que mon scanner travaillait avant sans la fermer.
Mais maintenant, il n'est pas. Quelqu'un peut m'aider ici?

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)."); 
    }
}
  • Comment pouvez-vous dire que votre scanner ne fonctionne plus? Ce comportement voyez-vous?
InformationsquelleAutor Niek van der Linden | 2013-03-25
  1. 47

    Je suis en supposant que vous êtes à l'aide de java 7, ainsi vous obtenez un avertissement du compilateur, lorsque vous ne fermez pas la ressource, vous devez fermer votre scanner généralement dans un bloc finally.

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

    Ou encore mieux: utiliser la nouvelle Essayez avec déclaration de ressources:

    try(Scanner scanner = new Scanner(System.in)){
    //rest of your code
}
    • Devrait new Scanner(System.in) être entouré avec try-catch bloc?
    • pas nécessairement. mais c'est une bonne pratique pour fermer vos ressources à l'intérieur d'un bloc finally.
    • Merci, vraiment comme Essayer avec la déclaration de ressources, fonctionne comme un charme!
    • btw, je veux juste vous mentionner. c'est juste un avertissement du compilateur que vous obtenez et je suis certainement plus sûr que ce ne serait pas le cas des problèmes avec la façon dont les fonctions de votre scanner. compilateur est juste vous dire que vous devez fermer votre ressource dont vous avez ouvert pour empêcher les fuites . 🙂
    • Merci pour le heads up, le scanner fonctionnait bien, mais je n'aimais pas l'erreur 🙂
    • Pourquoi êtes-vous fermeture System.in par vous-même? C'est le travail de la machine à l'arrêt.
    • Pourquoi avons-nous besoin de fermer le Scanner ? Qu'entend-on par "fuite des ressources" ?
    • Alors pourquoi ne éclipse ne cesse de jeter de l'avertissement ? Il me fait me demander si je fais quelque chose de mal.
    • Fonctionne comme un charme merci.

    InformationsquelleAutor PermGenError
  2. 5

    Selon la Javadoc de Scanner, il ferme le flux lorsque vous appelez la méthode close. En règle générale, le code qui crée une ressource est également responsable pour la refermer. Système.n'était pas instancié par votre code, mais par la VM. Dans ce cas, il est sûr de ne pas fermer le Scanner, ignorer l'avertissement et d'ajouter un commentaire pourquoi vous l'ignorer. La VM va prendre soin de la fermer si nécessaire.

    (Hors-sujet: au lieu de "montant", le mot "nombre" serait plus approprié d'utiliser pour un certain nombre de joueurs. L'anglais n'est pas ma langue maternelle (je suis néerlandais) et j'ai utilisé pour faire exactement la même erreur.)

    InformationsquelleAutor Henno Vermeulen
  3. 1

    Voici quelques une meilleure utilisation de java pour scanner

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

    //Use sc as you need

} catch (Exception e) {

        // handle exception

}
    InformationsquelleAutor Dev4World
  4. 0

    Essayer cette

    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).");
    InformationsquelleAutor tmwanik