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