如何优雅地读取文件中的每一行？

Question

最近文件操作较多，但大多数都是一行一行地读取，每一行是一条新闻。经常用的代码是这样的：
InputStreamis=null;
try{
is=newFileInputStream(textPath);
BufferedReaderreader=newBufferedReader(newInputStreamReader(is,"UTF-8"),512);
//读取一行，存储于字符串列表中
for(Stringline=reader.readLine();line!=null;line=reader.readLine()){
line=line.trim();
//dosomethinghere
}
}catch(FileNotFoundExceptionfnfe){
fnfe.printStackTrace();
}catch(IOExceptionioe){
ioe.printStackTrace();
}finally{
try{
if(is!=null){
is.close();
is=null;
}
}catch(IOExceptione){
e.printStackTrace();
}
}
当dosomething变得很庞大时，这try语句块就变得有点臃肿。是否能存在这样的一个文件读取类，就像Iterator迭代器一样，使用hasNext()和next()遍历文件中的所有行，而将异常处理等全部隐藏起来？还是有什么其它更加优雅的方法？

Answer 1

java当然可以很优雅,宇宙第一的编程语言也不是光靠sun和oracle忽悠出来的.这里我们需要把文件处理的逻辑和业务逻辑分开,引入strategy模式是比较合适的.文件处理的部分是不变的可以重用的,业务逻辑是变化的.还有就是java7引入try-with-resource,已经不需要自己去关闭流了.代码:publicinterfaceFileProcessor{voidprocessByLine(StringFilePath,LineProcessorprocessor);}publicinterfaceLineProcessor{voiddoSomeThing(StringoneLine);}publicclassFileProcessorImplimplementsFileProcessor{@OverridepublicvoidprocessByLine(StringfilePath,LineProcessorprocessor){try(BufferedReaderbr=newBufferedReader(newFileReader(filePath))){Stringline=br.readLine();processor.doSomeThing(line.trim());}catch(FileNotFoundExceptionfnfe){fnfe.printStackTrace();}catch(IOExceptionioe){ioe.printStackTrace();}}}publicclassFileProcessImplTest{@TestpublicvoidtestProcessByLine(){finalStringBuildersb=newStringBuilder();finalStringBuildersb1=newStringBuilder();FileProcessorfp=newFileProcessorImpl();fp.processByLine("a.txt",newLineProcessor(){@OverridepublicvoiddoSomeThing(StringoneLine){sb.append(oneLine);}});fp.processByLine("a_reverse_by_line.txt",newLineProcessor(){@OverridepublicvoiddoSomeThing(StringoneLine){sb1.append(newStringBuilder(oneLine).reverse());}});assertEquals(sb.toString(),sb1.toString());}}

Answer 2

可以使用commons-io的FileUtils:LineIterator:importorg.apache.commons.io.FileUtils;importorg.apache.commons.io.LineIterator;finalLineIteratorit=FileUtils.lineIterator(newFile(""),"UTF-8");try{while(it.hasNext()){finalStringline=it.nextLine();}}finally{it.close();}如果内存足够:importorg.apache.commons.io.Charsets;importorg.apache.commons.io.FileUtils;finalListlines=FileUtils.readLines(newFile(""),Charsets.UTF_8);另外推荐LZ使用org.apache.commons.io.Charsets.UTF_8代替字符串的"UTF-8"，如：//不会抛出UnsupportedEncodingExceptionnewInputStreamReader(is,Charsets.UTF_8)str.getBytes(Charsets.UTF_8);因为Java规定了所有平台都必须实现UTF-8，这里本来就不可能抛出UnsupportedEncodingException。Java7中我们就可以用java.nio.charset.StandardCharsets了，Java6暂时拿这个顶一下。另外LZ可以用org.apache.commons.io.IOUtils.closeQuietly来关闭一个流。你的finally块可以简化为：finnaly{IOUtils.closeQuiety(is);}