尽量避免诸如x.read(new FileInputStream(sourceFile)); 的写法
来源:互联网 发布:微信 for mac 2.2 dmg 编辑:程序博客网 时间:2024/05/22 12:29
一不小心,顺手写了x.read(new FileInputStream(sourceFile)); 这样的代码,却引得自己花费了半个多小时去调试问题。
原因是这样的:在开发某一个feature的时候,需要对操作的文件进行backup,于是写了诸如下面的代码(以下仅是演示代码,与实际要简易,仅供参考):
/*
* XXX是一个处理类,soureFile是一个输入的File object
*/
XXX x = new XXX();
x.read(new FileInputStream(sourceFile));
..... //complex logic process
String sourceFilePath = sourceFile.getPath();
int maxBackups = 3;
List<File> files = new ArrayList<File>();
files.add(traFile);
String base = sourceFile.getPath() + '.';
for (int generations = 1; true; generations++) {
File f = new File(base + generations);
files.add(f);
if (!f.exists()) {
break;
}
}
int generations = files.size();
for (int generation = generations - 1; generation > 0; generation--) {
if (generation > maxBackups) {
files.get(generation).delete();
} else {
if(files.get(generation).exists()) files.get(generation).delete();
files.get(generation - 1).renameTo(files.get(generation));
}
}
..... //complex logic process
x.write(new FileOutputStream(sourceFilePath));
* XXX是一个处理类,soureFile是一个输入的File object
*/
XXX x = new XXX();
x.read(new FileInputStream(sourceFile));
..... //complex logic process
String sourceFilePath = sourceFile.getPath();
int maxBackups = 3;
List<File> files = new ArrayList<File>();
files.add(traFile);
String base = sourceFile.getPath() + '.';
for (int generations = 1; true; generations++) {
File f = new File(base + generations);
files.add(f);
if (!f.exists()) {
break;
}
}
int generations = files.size();
for (int generation = generations - 1; generation > 0; generation--) {
if (generation > maxBackups) {
files.get(generation).delete();
} else {
if(files.get(generation).exists()) files.get(generation).delete();
files.get(generation - 1).renameTo(files.get(generation));
}
}
..... //complex logic process
x.write(new FileOutputStream(sourceFilePath));
当写完以后,运行TestCase并不是每次都成功,有时候就可以正确生成backup文件,而有时候则没有生成(没有生成的概率大很多)。跟踪了一下,发现当没有正确生成backup的时候,是那段file renameTo没有执行成功。就是如上面代码中“粗粉红色的标记的”:files.get(generation - 1).renameTo(files.get(generation))
然后进行Debug状态的step by step运行,却每次都成功了。
仔细了想了想,可能的原因是 files.get(generation - 1) 这个所引对象的对象,在执行renameTo操作的时候,可能还在被某个资源锁定,而没有释放。
带着个这个想法,把代码从头再过了一遍,终于发现是 x.read(new FileInputStream(sourceFile)); 这段出了问题。因为XXX这个类,在内部处理过程中,并没有对输入的inputstream进行关闭(当然,这是一个正确的设计和实现,一般我们在开发过程中,都不会在“使用者”内部关闭外部的流,这样危险性非常大,但是正常情况下,java中对流的关闭还是很科学的,一般对于
BufferedReader reader =
new
BufferedReader(
new
InputStreamReader(
new
FileInputStream(
"a.txt"
)));这样的写法,直接reader.close()就行了,关闭包装流,关闭的时候会调用其处理的节点流的关闭方法。当然本例中,作者的XXX这个类写的不合理,没有关闭流而已。一般是可以这么使用包装流的。
但是,为了代码了简洁,很随意的就进行了 x.read(new FileInputStream(sourceFile)); 这样的书写。
正是这样的书写,让后面针对soureFile的操作不能执行成功。因为所用在这个soureFile上的InputStream流并没有被“真正关闭”。
于是改成:
InputStream is = new FileInputStream(sourceFile);
x.read(is);
is.close;
就可以了。
当然,并不是说诸如 x.read(new FileInputStream(sourceFile)); 的写法不能用,而是尽量避免在“在同一个方法开闭区间内多次引用sourceFile”,在这样场合下,就肯定不能用了。
阅读全文
0 0
- 尽量避免诸如x.read(new FileInputStream(sourceFile)); 的写法
- 尽量避免诸如x.read(new FileInputStream(sourceFile)); 的写法
- 尽量避免诸如x.read(new FileInputStream(sourceFile)); 的写法
- 尽量避免诸如x.read(new FileInputStream(sourceFile)); 的写法
- FileInputStream的read方法
- FileInputStream的read方法详解
- JNI探秘-----FileInputStream的read方法详解
- bitmap的处理,尽量避免OOM
- 尽量避免臃肿的if else
- 开发中如何尽量的避免BUG
- 在C++里面诸如 *iter++这样的形式是一种规范的写法吗?
- 关于BufferedInputStream和FileInputStream中Read方法的问题
- new Fileinputstream(),拒绝访问
- 尽量避免官司缠身
- inputStreamReader(new FileInputStream(file),"UTF-8")出现乱码的问题
- JavaScript的 new function () {} 写法
- 十大网站运营要尽量避免的错误
- AS3中尽量避免的算法(边看边记)
- PHP链接mysql
- Android初级开发(十)——百度定位—确定你的位置
- PHP constant()函数
- JAVA设计模式(二十一)----访问者模式
- text()html() 以及 val() nnerHTML和innerTEXT的用法
- 尽量避免诸如x.read(new FileInputStream(sourceFile)); 的写法
- Spring MVC重定向无法实现页面跳转
- python学习之初始化实例、类属性、方法
- 有关SpringMVC-SSM整合开发跨域问题
- Altium Designer 导入DXF文件[转载]
- 通过思维导图学习知识的两种模板鉴赏
- 安装Xdebug, 版本问题
- 剑指Offer—54—字符流中第一个不重复的字符
- 串口通信-MSComm控件使用详解