java.util.NoSuchElementException - Scanner lisant les données de l'utilisateur

Question

J'utilise Java pour la première fois, mais j'ai une certaine expérience de C#. Le problème que je rencontre concerne la lecture des données de l'utilisateur à partir de la console.

Je rencontre l'erreur "java.util.NoSuchElementException" avec cette portion de code :

payment = sc.next(); // PromptCustomerPayment function

J'ai deux fonctions qui reçoivent des données de l'utilisateur :

• PromptCustomerQty
• PromptCustomerPayment

Si je n'appelle pas PromptCustomerQty, je n'obtiens pas cette erreur, ce qui m'amène à penser que je fais quelque chose de mal avec le scanner. Vous trouverez ci-dessous mon exemple de code complet. J'apprécie toute aide.

public static void main (String[] args) {   

    // Create a customer
    // Future proofing the possabiltiies of multiple customers
    Customer customer = new Customer("Will");

    // Create object for each Product
    // (Name,Code,Description,Price)
    // Initalize Qty at 0
    Product Computer = new Product("Computer","PC1003","Basic Computer",399.99); 
    Product Monitor = new Product("Monitor","MN1003","LCD Monitor",99.99);
    Product Printer = new Product("Printer","PR1003x","Inkjet Printer",54.23);

    // Define internal variables 
    // ## DONT CHANGE 
    ArrayList<Product> ProductList = new ArrayList<Product>(); // List to store Products
    String formatString = "%-15s %-10s %-20s %-10s %-10s %n"; // Default format for output

    // Add objects to list
    ProductList.add(Computer);
    ProductList.add(Monitor);
    ProductList.add(Printer);

    // Ask users for quantities 
    PromptCustomerQty(customer, ProductList);

    // Ask user for payment method
    PromptCustomerPayment(customer);

    // Create the header
    PrintHeader(customer, formatString);

    // Create Body
    PrintBody(ProductList, formatString);   
}

public static void PromptCustomerQty(Customer customer, ArrayList<Product> ProductList) {
    // Initiate a Scanner
    Scanner scan = new Scanner(System.in);

    // **** VARIABLES ****
    int qty = 0;

    // Greet Customer
    System.out.println("Hello " + customer.getName());

    // Loop through each item and ask for qty desired
    for (Product p : ProductList) {

        do {
        // Ask user for qty
        System.out.println("How many would you like for product: " + p.name);
        System.out.print("> ");

        // Get input and set qty for the object
        qty = scan.nextInt();

        }
        while (qty < 0); // Validation

        p.setQty(qty); // Set qty for object
        qty = 0; // Reset count
    }

    // Cleanup
    scan.close();
}

public static void PromptCustomerPayment (Customer customer) {
    // Initiate Scanner 
    Scanner sc = new Scanner(System.in);

    // Variables
    String payment = "";

    // Prompt User
    do {
    System.out.println("Would you like to pay in full? [Yes/No]");
    System.out.print("> ");

    payment = sc.next();

    } while ((!payment.toLowerCase().equals("yes")) && (!payment.toLowerCase().equals("no")));

    // Check/set result
    if (payment.toLowerCase().equals("yes")) {
        customer.setPaidInFull(true);
    }
    else {
        customer.setPaidInFull(false);
    }

    // Cleanup
    sc.close(); 
}

Answer 1

Cela m'a laissé perplexe pendant un certain temps, mais voici ce que j'ai trouvé en fin de compte.

Lorsque vous appelez, sc.close() en première méthode, il ne ferme pas seulement votre scanner mais aussi votre System.in ainsi que le flux d'entrée. Vous pouvez le vérifier en affichant son état tout en haut de la deuxième méthode sous la forme suivante : :

    System.out.println(System.in.available());

Ainsi, lorsque vous réinstallez, Scanner dans la deuxième méthode, il ne trouve pas d'ouverture. System.in et donc l'exception.

Je doute qu'il y ait une solution pour rouvrir le dossier. System.in parce que :

public void close() throws IOException --> Closes this input stream and releases any system resources associated with this stream. The general contract of close is that it closes the input stream. A closed stream cannot perform input operations and **cannot be reopened.**

La seule bonne solution à votre problème est d'initier la Scanner dans votre méthode principale, passez-le comme argument dans vos deux méthodes, et fermez-le à nouveau dans votre méthode principale, par exemple :

main bloc de code lié à la méthode :

Scanner scanner = new Scanner(System.in);  

// Ask users for quantities 
PromptCustomerQty(customer, ProductList, scanner );

// Ask user for payment method
PromptCustomerPayment(customer, scanner );

//close the scanner 
scanner.close();

Vos méthodes :

 public static void PromptCustomerQty(Customer customer, 
                             ArrayList<Product> ProductList, Scanner scanner) {

    // no more scanner instantiation
    ...
    // no more scanner close
 }

 public static void PromptCustomerPayment (Customer customer, Scanner sc) {

    // no more scanner instantiation
    ...
    // no more scanner close
 }

J'espère que cela vous donnera une idée de la panne et des solutions possibles.

Answer 2

Le problème est le suivant

Lorsqu'un scanner est fermé, il ferme sa source d'entrée si celle-ci implémente l'interface Closeable.

http://docs.oracle.com/javase/1.5.0/docs/api/java/util/Scanner.html

Ainsi scan.close() ferme System.in .

Pour y remédier, vous pouvez faire

Scanner scan static et ne la ferme pas dans PromptCustomerQty. Le code ci-dessous fonctionne.

public static void main (String[] args) {   

// Create a customer
// Future proofing the possabiltiies of multiple customers
Customer customer = new Customer("Will");

// Create object for each Product
// (Name,Code,Description,Price)
// Initalize Qty at 0
Product Computer = new Product("Computer","PC1003","Basic Computer",399.99); 
Product Monitor = new Product("Monitor","MN1003","LCD Monitor",99.99);
Product Printer = new Product("Printer","PR1003x","Inkjet Printer",54.23);

// Define internal variables 
// ## DONT CHANGE 
ArrayList<Product> ProductList = new ArrayList<Product>(); // List to store Products
String formatString = "%-15s %-10s %-20s %-10s %-10s %n"; // Default format for output

// Add objects to list
ProductList.add(Computer);
ProductList.add(Monitor);
ProductList.add(Printer);

// Ask users for quantities 
PromptCustomerQty(customer, ProductList);

// Ask user for payment method
PromptCustomerPayment(customer);

// Create the header
PrintHeader(customer, formatString);

// Create Body
PrintBody(ProductList, formatString);   
}

static Scanner scan;

public static void PromptCustomerQty(Customer customer, ArrayList<Product> ProductList)               {
// Initiate a Scanner
scan = new Scanner(System.in);

// **** VARIABLES ****
int qty = 0;

// Greet Customer
System.out.println("Hello " + customer.getName());

// Loop through each item and ask for qty desired
for (Product p : ProductList) {

    do {
    // Ask user for qty
    System.out.println("How many would you like for product: " + p.name);
    System.out.print("> ");

    // Get input and set qty for the object
    qty = scan.nextInt();

    }
    while (qty < 0); // Validation

    p.setQty(qty); // Set qty for object
    qty = 0; // Reset count
}

// Cleanup

}

public static void PromptCustomerPayment (Customer customer) {
// Variables
String payment = "";

// Prompt User
do {
System.out.println("Would you like to pay in full? [Yes/No]");
System.out.print("> ");

payment = scan.next();

} while ((!payment.toLowerCase().equals("yes")) && (!payment.toLowerCase().equals("no")));

// Check/set result
if (payment.toLowerCase() == "yes") {
    customer.setPaidInFull(true);
}
else {
    customer.setPaidInFull(false);
}
}

Par ailleurs, vous ne devriez pas utiliser == pour la comparaison des chaînes de caractères, utilisez .equals au lieu de cela.

Answer 3

Vous devez supprimer les lignes de fermeture du scanner : scan.close();

Cela m'est déjà arrivé et c'était la raison.

Answer 4

La raison de l'exception a déjà été expliquée, mais la solution proposée n'est pas vraiment la meilleure.

Vous devez créer une classe qui conserve un scanner comme privé en utilisant le modèle Singleton, ce qui rend ce scanner unique dans votre code.

Vous pouvez ensuite implémenter les méthodes dont vous avez besoin ou créer un getScanner (ce qui n'est pas recommandé) et le contrôler à l'aide d'un booléen privé, quelque chose comme alreadyClosed.

Si vous ne savez pas comment utiliser le modèle Singleton, voici un exemple :

public class Reader {

    private Scanner reader;
    private static Reader singleton = null;
    private boolean alreadyClosed;

    private Reader() {
        alreadyClosed = false;
        reader = new Scanner(System.in);
    }

    public static Reader getInstance() {
        if(singleton == null) {
            singleton = new Reader();
        }
        return singleton;
    }

    public int nextInt() throws AlreadyClosedException {
        if(!alreadyClosed) {
            return reader.nextInt();
        }
        throw new AlreadyClosedException(); //Custom exception
    }

    public double nextDouble() throws AlreadyClosedException {
        if(!alreadyClosed) {
            return reader.nextDouble();
        }
        throw new AlreadyClosedException();
    }

    public String nextLine() throws AlreadyClosedException {
        if(!alreadyClosed) {
            return reader.nextLine();
        }
        throw new AlreadyClosedException();
    }

    public void close() {
        alreadyClosed = true;
        reader.close();
    }   
}

Answer 5

Pour tous ceux qui sont arrivés ici en passant un examen en ligne sur un site comme HackerRank

Vous pouvez recevoir ce message si vous essayez de tester votre code (probablement) parfaitement bon en cliquant sur le bouton pour exécuter main() avec une entrée personnalisée.

Dans ce cas, vous devez cliquer sur l'autre bouton, quelque chose comme "Run Unit Tests". Il est probable que vous ne soyez évalué que sur le fait que le code passe les tests unitaires qu'ils ont écrits - et non sur votre capacité à remanier le code pour réduire le nombre de LOC ou sur votre style de codage.