Question

我正在处理一种库存程序，该程序处理从文件中读取项目列表并且我遇到了一些问题。

这是我到目前为止完成的代码：

import java.util.Scanner; 
import java.io.*; 

public class Store { 
    public static void main(String[] args) throws Exception {
        Book[] books = readInventory();

        for (Book book : books) {
            System.out.printf("ISBN: %s, Price: %f, Copies: %d%n",
            book.getISBN(), book.getPrice(), book.getCopies()); 
        }
    }

    public static Book[] readInventory() throws Exception {
        Book[] books = new Book[15];
        java.io.File file = new java.io.File("../instr/prog4.dat");
        Scanner fin = new Scanner(file);
        String isbn;
        double price;
        int copies;
        int i = 0;

        while (fin.hasNext()) {
             isbn = fin.next();
                if (fin.hasNextDouble()); {
                    price = fin.nextDouble();
                }
                if (fin.hasNextInt()); {
                    copies = fin.nextInt();
                }
             Book book = new Book(isbn, price, copies);
             books[i] = book;
             i++;
        }
        fin.close();
        return books;
}

    public static void printInfo(Book[] books) {
        for(int x=0; x<books.length; x++) {
            System.out.println("ISBN: " + books[x].getISBN() + "\n Price: " +
            books[x].getPrice() + "\n Copies: " + books[x].getCopies());
        }   
    }
}

class Book {
    private String isbn;
    private double price;
    private int copies;

    public Book(String isbnNum, double priceOfBook, int copiesInStock) {
        isbn = isbnNum;
        price = priceOfBook;    
        copies = copiesInStock;
    }

    public String getISBN() {
        return isbn;
    }

    public double getPrice() {
        return price;
    }

    public int getCopies() {
        return copies;
    }

    public void setISBN(String isbn) {
        this.isbn = isbn;
    }

    public void setPrice(double price) {
        this.price = price;
    }

    public void setCopies(int copies) {
        this.copies = copies;
    }

    @Override
    public String toString() {
        return String.format("ISBN: %s, Price: %f, Copies: %d%n",
             this.getISBN(), this.getPrice(), this.getCopies());
    }
}

程序编译正常，但是当我运行程序时，我收到错误

Exception in thread "main" java.util.NoSuchElementException
    at java.util.Scanner.throwFor(Scanner.java:855)
    at java.util.Scanner.next(Scanner.java:1364)
    at Store.readInventory(Store.java:31)
    at Store.main(Store.java:13)

以下是我应该使用的文件的内容：

1234567 31.67 0
1234444 98.50 4
1235555 27.89 2
1235566 102.39 6
1240000 75.65 4
1247761 19.95 12
1248898 155.91 0
1356114 6.95 17
1698304 45.95 3
281982X 31.90 5

我一直在看我最近写的另一个程序，它处理读取文件，看到我在相同类型的地方抛出异常，看起来我是（在该程序中处理该文件的所有内容都在主要方法，这是唯一一个抛出异常的方法）。根据我从早期的Java编程类中记得的，我们应该在main方法和处理该文件的任何其他方法中抛出异常，所以在这种情况下，我在readInventory（）方法上有一个在main上，另一个在。

我错过了什么？

Answer 1

在阅读之前检查下一个值，以避免出现isbn但price和copies之类的异常。

示例代码：

 if(fin.hasNextDouble()){
      price = fin.nextDouble();
 }
 if(fin.hashNextInt()){
     copies = fin.nextInt();
 }

例如Scanner#nextInt()方法抛出：

InputMismatchException - 如果下一个标记与整数正则表达式不匹配，或者超出范围
NoSuchElementException - 如果输入已用尽
IllegalStateException - 如果此扫描仪已关闭

您正在创建大小为15的数组，其中只有10行文件导致NullPointerException。在这种情况下使用List。

即使出现错误，您也可以使用Java 7 - The try-with-resources Statement来更好地管理资源。

示例代码

public static BookDetail[] readInventory() throws Exception {
    List<BookDetail> books = new ArrayList<BookDetail>();
    java.io.File file = new java.io.File("../instr/prog4.dat");
    try (Scanner fin = new Scanner(file)) {
        String isbn;
        double price = 0;
        int copies = 0;
        while (fin.hasNext()) {
            isbn = fin.next();
            if (fin.hasNextDouble()) {
                price = fin.nextDouble();
            }
            if (fin.hasNextInt()) {
                copies = fin.nextInt();
            }
            BookDetail book = new BookDetail(isbn, price, copies);
            books.add(book);
        }
    }
    return books.toArray(new BookDetail[books.size()]);
}

Answer 2

目前，您正在检查文件是否包含更多数据，然后您尝试阅读15本书，无论如何。你的for循环从0变为books.length-1，无论文件中是什么。

由于您（理论上）不知道文件中有多少本书，因此您需要在阅读每本书之前查看fin.hasNext。

所以摆脱你的for循环并用这样的东西替换你的阅读代码：

int i = 0;
while (fin.hasNext()) {
     // read book here and assign to books[i]

     // increment i for next iteration
     i++;
}

在阅读每本书之前，将检查while条件fin.hasNext()。当它返回false时，您将停止循环播放，i将是您阅读的图书数量。

请注意，您仍然假设每本书都包含其所有字段。如果文件中只有一半的书，你的代码仍会崩溃。但这可能是一件好事，因为那不是一种可以从中恢复的方式，并且可能你可以假设你只会收到良好的输入。

Answer 3

我建议您使用异常处理。我特别推荐使用try with resources，因为您忘记关闭正在使用的扫描仪。

您需要在代码中签入您收到的每本书所需的所有信息。您不希望仅仅因为您无法接收该特定书籍上的其余数据而只获得有关该书的部分信息。

在Java中引发异常错误

3 个答案: