基本上,我想打开一个文件,读取一些字节,然后关闭该文件。这就是我想到的:
try
{
InputStream inputStream = new BufferedInputStream(new FileInputStream(file));
try
{
// ...
inputStream.read(buffer);
// ...
}
catch (IOException e)
{
// TODO Auto-generated catch block
e.printStackTrace();
}
finally
{
try
{
inputStream.close();
}
catch (IOException e)
{
// TODO Auto-generated catch block
e.printStackTrace();
}
}
}
catch (FileNotFoundException e)
{
// TODO Auto-generated catch block
e.printStackTrace();
}
也许我被RAII宠坏了,但在Java中一定有更好的方法可以做到这一点,对吧?
如果IOException
和FileNotFoundException
有相同的异常处理代码,那么只需一个catch
子句就可以以更紧凑的方式重写示例:
try {
InputStream input = new BufferedInputStream(new FileInputStream(file));
try {
// ...
input.read(buffer);
// ...
}
finally {
input.close();
}
}
catch (IOException e) {
e.printStackTrace();
}
如果您可以传播异常,那么您甚至可以去掉外部try-catch
,这可能比手动打印堆栈跟踪更有意义。如果你没有在程序中发现一些异常,你会自动打印堆栈跟踪。
此外,手动关闭流的需求将在Java 7中通过自动资源管理来解决。
通过自动资源管理和异常传播,代码简化为以下内容:
try (InputStream input = new BufferedInputStream(new FileInputStream(file))) {
// ...
input.read(buffer);
// ...
}
通常这些方法都封装在库中。除非您想在这个级别上编写,否则最好创建自己的辅助方法或使用现有的方法,如FileUtils。
String fileAsString = Fileutils.readFileToString(filename);
// OR
for(String line: FileUtils.readLines(filename)) {
// do something with each line.
}
我不知道这是否是正确的方法,但您可以将所有代码放在同一个try块中,然后将不同的catch块放在一起。
try {
...
}
catch (SomeException e) {
...
}
catch (OtherException e) {
...
}
如果您想在纯Java中实现这一点,那么您发布的代码看起来还可以
您可以使用第三方库,如Commons IO,在其中您需要编写的代码要少得多。例如
查看Commons IO,网址:
http://commons.apache.org/io/description.html
有时您可以将代码简化为以下内容:
public void foo(String name) throws IOException {
InputStream in = null;
try {
in = new FileInputStream(name);
in.read();
// whatever
} finally {
if(in != null) {
in.close();
}
}
}
当然,这意味着foo
的调用者必须处理IOException
,但大多数情况下都应该是这样。最后,您并没有真正降低那么多复杂性,但由于嵌套的异常处理程序较少,代码变得可读性更强。
在不使用实用程序的情况下,我对此的看法是:
InputStream inputStream = null;
try {
inputStream = new BufferedInputStream(new FileInputStream(file));
// ...
inputStream.read(buffer);
// ...
} catch (IOException e) {
e.printStackTrace();
throw e; // Rethrow if you cannot handle the exception
} finally {
if (inputStream != null) {
inputStream.close();
}
}
不是一句话,但还不错。比如说,使用Apache Commons IO,它将是:
//...
buffer = FileUtils.readFileToByteArray(file);
//...
需要记住的是,标准Java缺乏许多这样的小实用程序和每个人都需要的易于使用的接口,所以你必须依赖一些支持库,如Apache Commons、Google Guava。。。在您的项目中,(或实现您自己的实用程序类)。
使用该包中的org.apache.commons.io.FileUtils.readFileToByteArray(File)
或类似内容。它仍然抛出IOException,但它为您处理清理。
尝试以下操作:
try
{
InputStream inputStream = new BufferedInputStream(new FileInputStream(file));
byte[] buffer = new byte[1024];
try
{
// ...
int bytesRead = 0;
while ((bytesRead = inputStream.read(buffer)) != -1) {
//Process the chunk of bytes read
}
}
catch (IOException e)
{
// TODO Auto-generated catch block
e.printStackTrace();
}
finally
{
inputStream.close();
}
}
catch (FileNotFoundException e)
{
// TODO Auto-generated catch block
e.printStackTrace();
}
谷歌番石榴试图通过引入Closeables来解决这个问题。
否则,您必须等到JDK7中的AutoCloseable出现,因为它解决了抛出IOException时的一些情况。