我必须关闭 FileInputStream 吗？

Question

我正在测试自动化领域担任实习生。 我正在使用 Eclipse 创建 Junit 代码并使用 Eclipse 运行。 我正在使用 FileInputStream 函数从 Excel 工作表中检索数据。

FileInputStream fi=new FileInputStream("c:\\search.xls");
Workbook w=Workbook.getWorkbook(fi);
Sheet s=w.getSheet(0);

是否需要关闭Inputstream功能？如果是这样，请指导我一些编码。

Answer 1

是的，如果您希望释放系统资源，您需要关闭输入流。

FileInputStream.close()< /code>就是你所需要的。

Answer 2

您要么需要 close()，要么结束您的程序。

但是，如果不关闭文件，您可能会遇到令人困惑的问题，因为

• 有时测试是单独运行的，或者一组测试是在同一进程中运行的。 （因此您可以进行一个以一种方式工作但不能以另一种方式工作的测试）
• 您无法重命名或删除打开的文件。

最佳实践是始终关闭您用完的资源，但是我将单元测试视为脚本，并不总是必须遵循最佳实践。

Answer 3

FileInputStream fi=null;
try {
    fi=new FileInputStream("c:\\search.xls");
    Workbook w=Workbook.getWorkbook(fi);
    Sheet s=w.getSheet(0);
} finally {
    if (fi!=null) {
        fi.close();
    }
}

Answer 4

关闭您使用的资源始终是个好主意，但是：

如果您在资源 B 中使用资源 A，则关闭 是明智的做法B 而不是 A（如果它有方法的话）。

在您的情况下，您在 Workbook 中使用 FileInputStream，因此您最好关闭 Workbook 并依赖 Workbook它将关闭FileInputStream。

在这种特殊情况下，实际上，Workbook 将关闭< /a> FileInputStream 位于 getWorkbook() 方法的末尾，但这仍然是一个好主意 关闭 Workbook 以便能够进行垃圾收集。

Answer 5

是的！在使用完资源后，您应该始终释放它们。 Java 有一个强大的垃圾收集机制（请注意，它与资源管理/泄漏相比是不同的。）
那么垃圾收集器无法确定您将来是否需要该资源？未能释放资源可能会导致诸如拒绝服务、性能不佳等问题。

尝试使用资源

    try (FileInputStream fi = new FileInputStream("c:\\search.xls")) {

         //do something with fi.
         //fi.getChannel() ;

    } catch(IOException e) {
        // exception handling.
    } finally {
    // some statements for finally.
   }

正如已经回答的那样，但另一种更省力的方法是立即 您不需要显式调用 fi.close() 方法。

Answer 6

Workbook 实现 Closeable 接口，该接口指示您应该调用 close() 方法或对资源使用 try 来释放 Workbook 对象获取的资源。

仅通过观察工作簿具有 close() 方法，您无法得出有关工作簿如何使用 InputStream 资源的任何结论。

真正发生的是工作簿构造函数使用流来初始化自身。构造函数不保留对流的引用。 Workbook 构造函数不会关闭InputStream。您有责任关闭InputStream，您可以在构造Workbook 对象后立即执行此操作。因此下面的代码是可以的：

private static Workbook getWorkbook() throws IOException {
    try (InputStream is = ...) {
        return WorkbookFactory.create(is); // newer API
    }
}

    try (Workbook workbook = getWorkbook()) {
        Sheet sheet = workbook.getSheet("SheetName");
        ...
    }

Answer 7

最近，当我尝试重构代码时，我必须将工作簿创建移至另一个方法，并在该方法中创建 FileInputStream。该方法创建一个 FileInputStream 并返回一个工作簿。但是FileInputStream从main方法中是不可见的；那么如何在 main 方法结束时关闭 FileInputStream 呢？答案是，您不必关闭 FileInputStream，而只需关闭工作簿，工作簿会在内部关闭 FileInputStream。简而言之，无论如何都必须关闭 FileInputStream 的说法是不正确的。

Answer 8

我这样做可以确保关闭excel文件输入流，这可能有帮助

abstract int workWithWorkBook(Workbook workBook);

protected int doWorkBook(Path excelFile) throws IOException {
    File f = excelFile.toFile();

    try (FileInputStream excelContent = new FileInputStream(excelFile.toFile())){
        POIFSFileSystem fileSystem = new POIFSFileSystem(excelContent);
        Workbook workBook = null;
        if (f.getName().endsWith("xls")) {
            workBook = new HSSFWorkbook(fileSystem);
        } else if (f.getName().endsWith("xlsx")) {
            workBook = new XSSFWorkbook(excelContent);
        }
        return workWithWorkBook(workBook);

    }catch (Exception e){
        e.printStackTrace();
        throw e;
    }
}

9b9ea92b-5b63-47f9-a865-fd40dd602cd5

Answer 9

做这样的事情。

FileInputStream fi=null;
try{
fi = new FileInputStream("c:\\search.xls");
Workbook w=Workbook.getWorkbook(fi);
Sheet s=w.getSheet(0);
}catch(IOException ioe){
}finally{
if(fi != null){
fi.close();
}
fi = null;//This will be hint to get finalize() called on fi so that underlying resources used will released like files opened.
}

Answer 10

Basic CompSci 101 告诉我们要确保关闭用 Java 或任何语言打开的资源。所以是的，您需要关闭它们。如果你不这样做，坏的juju就一定会发生。

另外，您应该学习（并愿意）使用 Javadocs。查看 FileInputStream 和 Closeable 的 Javadoc。答案就在那里。