我正在制作一个将灰度图像更改为黑白图像的程序，该程序必须从文件夹和子文件夹读取所有.png文件，并将它们编辑后保存到新目录中。一切都按预期工作，但我进行了一些更改，该程序现在仅从具有图像的第一个文件夹或子文件夹中读取文件。
public class FindFiles {
public void find(String dir, String finDir){
try {
File folder = new File(dir);
File[] listOfFiles = folder.listFiles();
ChangeColor change = new ChangeColor();
for(int i = 0; i < listOfFiles.length; i++) {
if (listOfFiles[i].isFile()) {
change.ChangeC(dir, finDir, listOfFiles[i].getName());
} else if (listOfFiles[i].isDirectory()) {
File f = new File(finDir+"/"+listOfFiles[i].getName());
if (!f.isDirectory()) {
new File(finDir+"/"+listOfFiles[i].getName()).mkdirs();
}
find(dir+"/"+listOfFiles[i].getName(),finDir+"/"+listOfFiles[i].getName());
}
}
} catch (IOException ex) {
Logger.getLogger(FindFiles.class.getName()).log(Level.SEVERE, null, ex);
}
}
}
感谢您的协助，以找出背后的原因。
考虑先清理代码，然后查找错误：
public class FindFiles {
public void find(File src, File target) {
assert(src.isDirectory() && target.isDirectory());
try {
File[] fileList = src.listFiles();
ChangeColor change = new ChangeColor();
for (File file : fileList) {
if (file.isFile()) {
change.ChangeC(file, target);
} else if (file.isDirectory()) {
// create the File object for the directory name
File newDir = new File(target, file.getName());
newDir.mkdir();
find(file, newDir);
}
}
} catch (IOException ex) {
Logger.getLogger(FindFiles.class.getName()).log(Level.SEVERE, null, ex);
}
}
}
当a将执行操作时，您会产生很多不必要的Files并混合String和Files File。
您需要更改的签名，ChangeColor.changeC()以接受File需要从源目录转换的File对象以及应以目标形式写入该对象的对象。该方法应确保第二个参数是我在此所做的目录-永远不要相信调用者！
我没有测试此代码，还有一些其他事情我会做不同的事情（例如异常处理），但是我不希望结构有太大区别，以便您可以更轻松地看到更改。
我在这里所做的是一种防御性编码形式。花时间与奥卡姆剃刀获得代码下来到最低限度了前面。选择在读取方法时有意义的变量名，而不是在顶部看起来不错的变量名（不要害怕不断更改它们，直到它易于阅读为止）。存在的错误将更容易发现，从长远来看，您将节省大量时间。
本文收集自互联网，转载请注明来源。
如有侵权，请联系[email protected] 删除。
我来说两句