标签:路径 分析 ring director exist 文件的 测试 最简 style
一直在做C++相关开发的工作。突然某一天一时兴起,想学习下Java开发。然后再网上找到一本Java简明教程,入门是够用了。看到文件IO这一章,想起之前用C++做的删除文件或目录的练习,于是打算用Java来实现这个功能。本以为很简单的功能,还是遇到一些问题。仔细分析下代码,定位到出问题的地方。在此处记录下,提醒自己不会重蹈覆辙。
1. 第一版的删除函数是这样写的
1 static boolean delFile(String filename) { 2 File file = new File(filename); 3 if (!file.exists()) { 4 return false; 5 } 6 7 if (file.isFile()) { 8 return file.delete(); 9 } else { 10 String[] filenames = file.list(); 11 for (String f : filenames) { 12 delFile(f); 13 } 14 return file.delete(); 15 } 16 }
代码能编译过,但是运行结果并不符合预期结果。如果在一个大型系统,出现这样的问题,往往是一件很头疼的小事。我的做法是,遇到自己不熟悉的,先自己建一个小工程实现这一功能,添加测试用例。如果测试通过,才会移植到大系统上去。
经过仔细分析,上面的代码出问题的地方是第10行,file.list()返回的结果是当前路径下的所有文件的文件名,并不包含路径。所以递归调用时,因为检测文件不存在,而返回。然后就有下面这一版。
2. 正确的删除文件或目录方法
1 static boolean delFile(File file) { 2 if (!file.exists()) { 3 return false; 4 } 5 6 if (file.isFile()) { 7 return file.delete(); 8 } else { 9 File[] files = file.listFiles(); 10 for (File f : files) { 11 delFile(f); 12 } 13 return file.delete(); 14 } 15 }
这版的代码直接以文件对象File为参数,file.listFiles()返回的也是自文件的File对象集合。File对象保存了该文件的具体位置,因此不会出现上一版出现的问题,这一版能正常的工作。
代码虽然能正常工作,但不是最简洁的版本。下面有一个更为简洁的版本。
3. 简单并能正常工作的版本
1 static boolean delFile(File file) { 2 if (!file.exists()) { 3 return false; 4 } 5 6 if (file.isDirectory()) { 7 File[] files = file.listFiles(); 8 for (File f : files) { 9 delFile(f); 10 } 11 } 12 return file.delete(); 13 }
同样可以正常工作,但是代码没有重复的。
标签:路径 分析 ring director exist 文件的 测试 最简 style
原文地址:https://www.cnblogs.com/zhugaopeng/p/9416857.html