我正试图用以下方法创建文件：
private boolean createFileMain(String path){
File file = new File(path);
if(file.isDirectory()){
return this.createDirectory(file);
} else if(file.isFile()) {
return this.createFile(file);
} else {
return false;
}
}
private boolean createFile(File file){
if(!file.exists()){
if(file.getParentFile().exists()){
try{
if(file.createNewFile()){
return true;
}
}catch(IOException e){
return false;
}
} else {
if(this.createDirectory(file)){
this.createFile(file);
} else {
return false;
}
}
}
return true;
}
private boolean createDirectory(File file){
if(!file.exists()){
if(file.mkdirs()){
return true;
}
return false;
}
return true;
}
文件的路径：
/用户/用户名/目录/帐户/
/Users/username/Directory/Srcs/file1.txt
/用户/用户名/目录/file2.txt
当我尝试运行这个时，下面的方法抛出一个StackOverFlowError。
public void writeInFile(String path, List<String> content) {
if ((new File(path)).exists()) {
try {
writer = new PrintWriter(path, "ASCII");
for (String contentItem : content) {
writer.println(contentItem);
}
writer.close();
} catch (FileNotFoundException e1) {
//DO STUFF
} catch (UnsupportedEncodingException e) {
//DO STUFF
}
} else {
this.createFileMain(path);
this.writeInFile(path, content);
}
为什么没有创建任何文件？
你读过isDirectory()等上的JavaDocs吗。？对于isDirectory()，它说：
当且仅当由该抽象路径名表示的文件存在并且是目录时，返回true；false否则
因此，如果目录不存在，则会得到false，并且不创建目录。然后您继续尝试写入、创建和写入等，直到您得到StackOverFlowError。
要修复stackoverflow，您应该检查创建文件的返回值，例如
boolean created = this.createFileMain(path);
if( created ) {
this.writeInFile(path, content);
}
要修复文件/目录的创建，您需要检查文件是否已经存在，然后创建它（也可以通过file.getParentFile().mkdirs()创建父目录）。
问题是，你应该知道你是想创建一个文件还是一个目录，因为你不能仅仅通过名称来判断路径是一个目录还是文件名（除非你发明了一些标记，比如总是用分隔符结束目录路径，或者要求文件总是有扩展名）。如果你想写一些你需要创建文件的内容，目录会再次破坏你的代码。
您的createFileMain只创建已经存在的文件。
你不需要创建一个文件来写入它，你所需要的只是你想将它写入的目录。
public void writeInFile(String path, List<String> content) {
File file = new File(path);
File parent = file.getParentFile();
if (parent != null && !parent.exists())
parent.mkdirs();
try (PrintWriter writer = new PrintWriter(path, "ASCII")) {
for (String contentItem : content) {
writer.println(contentItem);
}
writer.close();
} catch (IOException e1) {
//DO STUFF
}
}
private boolean createFile(File file) {
if(!file.exists()) {
if(file.getParentFile().exists()) {
// ...
} else { // In this case, neither file nor its parent exist
if(this.createDirectory(file)) {
this.createFile(file); // HERE, you're calling the same method again
} else {
// ...;
}
}
}
return true;
}
我相信您想用this.createFile(file.getParentFile());替换标记为HERE的行。这里发生的是，你的函数用相同的参数递归地调用自己，所以什么都不会发生，你的程序会陷入循环，直到耗尽堆栈内存。