169

I want to create and delete a directory using Java, but it isn't working.

File index = new File("/home/Work/Indexer1");
if (!index.exists()) {
    index.mkdir();
} else {
    index.delete();
    if (!index.exists()) {
        index.mkdir();
    }
}
10
  • 7
    What happened when you tried? Commented Nov 29, 2013 at 9:05
  • 1
    What is the question? Commented Nov 29, 2013 at 9:07
  • 1
    index file not remove.
    – Mr.G
    Commented Nov 29, 2013 at 9:07
  • 1
    Refer How to delete a directory in Java Commented Nov 29, 2014 at 19:23
  • 1
    Unfortunately, @AniketThakur, that approach will follow symbolic links and delete files and directories that may not have been intended. Commented Nov 11, 2015 at 16:07

30 Answers 30

264

Just a one-liner.

import org.apache.commons.io.FileUtils;

FileUtils.deleteDirectory(new File(destination));

Documentation here

10
  • 44
    um...no. This is a one-liner with an external dependency, which is an important thing to keep in mind. The only time using an external dependency like this is this simple is when you're doing a personal home project, or your company really doesn't care about the possibility of getting sued. Commented Apr 5, 2017 at 21:34
  • 27
    @searchengine27 but it seems the library is under Apache Commons so the risk of getting sued is negligible whitesourcesoftware.com/whitesource-blog/….
    – simtim
    Commented Jul 23, 2017 at 12:29
  • 3
    @simtim you're missing the point entirely. A company will never approve a library to use without a team of lawyers pouring over the terms of use and end user agreements, and any other legal documents associated with the library first. Somebody has to pay those lawyers...sometimes nobody wants to, which means the developer doesn't get to use it. The bigger the company you work for, the more red tape you have to go through. Commented Jul 24, 2017 at 22:02
  • 67
    @searchengine27 no, you're missing the point entirely. A company that needs an army of lawyers to allow to use apache commons is the absolute pathology, and nothing near the norm in IT world. I've never heard of anyone having such problems and if you have such problems you have most likely access to SO blocked so the answer would not be accessible to you anyway.
    – user5479362
    Commented Mar 20, 2018 at 16:20
  • 4
    @searchengine27 I have worked for large enterprise companies and can confirm that they do allow you to use open source libraries, it's just a fact of modern software development. However at my former employer they did keep long lists of all the dependencies including transitive dependencies and their licenses. if the license was not on a standard list (e.g. EPL, MPL, Apache etc) then we had to get specific approval for it. but it certainly didn't require an army of lawyers Commented Dec 8, 2022 at 18:42
131

Java isn't able to delete folders with data in it. You have to delete all files before deleting the folder.

Use something like:

String[] entries = index.list();
for (String s : entries) {
    File currentFile = new File(index.getPath(), s);
    currentFile.delete();
}

Then you should be able to delete the folder by using index.delete() Untested!

5
  • 51
    This won't delete non-empty subdirectories.
    – user3453226
    Commented Feb 15, 2015 at 17:20
  • 5
    Be VERY careful. If index is a symbolic link to another directory, you'll wind up deleting the contents of the other directory. Unfortunately, I've not yet found a good way to detect symbolic links on Windows in Java 6, though Java 7 provides Files.isSymbolicLink(). Commented Nov 11, 2015 at 16:04
  • 1
    Solution: wrap this chunk of code in if (!index.delete()) {...}. Then, if index is a symbolic link, it's deleted regardless of whether it appears it has contents. Commented Nov 11, 2015 at 16:16
  • This will throw a NullPointerException if there is an I/O exception while reading the directory. The code should check whether entries is null.
    – mernst
    Commented Oct 31, 2017 at 23:55
  • The method delete files only if index is a path of directory, example my/path/, otherwise throw NullPointerException. i.e. if index is my/path/file.txt has an error Commented May 6, 2022 at 10:46
115

This works, and while it looks inefficient to skip the directory test, it's not: the test happens right away in listFiles().

void deleteDir(File file) {
    File[] contents = file.listFiles();
    if (contents != null) {
        for (File f : contents) {
            deleteDir(f);
        }
    }
    file.delete();
}

Update, to avoid following symbolic links:

void deleteDir(File file) {
    File[] contents = file.listFiles();
    if (contents != null) {
        for (File f : contents) {
            if (! Files.isSymbolicLink(f.toPath())) {
                deleteDir(f);
            }
        }
    }
    file.delete();
}
4
  • 2
    As it turns out, there is a bug in this. If another process deletes the files during the loop, it can cause an exception that should be caught and ignored. Commented Sep 8, 2017 at 21:53
  • 4
    @9ilsdx9rvj0lo Rather than being snarky, perhaps you could provide an edit to handle symlinks. The OP said nothing about symbolic links in his post. Just creating and deleting a directory. Please also list out the "many things missing". Help us out.
    – Perry Tew
    Commented Mar 21, 2018 at 17:31
  • @PerryTew I'm not being snarky. I'm just pointing out, that I do fully disagree with your comment about the answer being better because no external libraries are being used. It is not. There is a good reason people are using apache commons : you don't have to program any single thing yourself. Symlinks are just an example of things you'll miss writing everything from scratch.
    – user5479362
    Commented Mar 22, 2018 at 7:31
  • 3
    It's not a matter of better/worse, but pros and cons. Not relying on external libraries is sometimes a significant benefit. Of course, there's a significant benefit to using tried-and-true software. It's up to the developer to balance the issues. If there are bugs other than the two already mentioned, we'd certainly like to know about them. Commented Apr 2, 2018 at 23:40
68

I prefer this solution on Java 8:

  Files.walk(pathToBeDeleted)
    .sorted(Comparator.reverseOrder())
    .map(Path::toFile)
    .forEach(File::delete);

From this site: http://www.baeldung.com/java-delete-directory

Update: To close the stream:

  try (Stream<Path> pathStream = Files.walk(repoDirFile.toPath())) {
    pathStream.sorted(Comparator.reverseOrder())
      .map(Path::toFile)
      .forEach(File::delete);
  }
11
  • 9
    Note that this may have scalability issues since it builds the full list, creates a sorted copy, and then iterates over the sorted copy. Back in the bad old days when memory wasn't inexhaustible, this would be a very bad idea. It's concise but at a cost in space (O(N) vs O(1)) and efficiency (O(N log N) vs O(N)). This wouldn't matter in most use cases. Commented Apr 2, 2018 at 22:59
  • 2
    this is elegant, works and does not rely on external libraries. loved it Commented Feb 12, 2019 at 18:48
  • 5
    Doesn't this have the problem of file handle leaks? This example does not close the stream returned by Files.walk(), which is explicitly indicated in the API docs. I know that if you don't close the stream returned by Files.list() for example, you can run out of handles and the program will crash. See e.g. stackoverflow.com/q/36990053/421049 and stackoverflow.com/q/26997240/421049 . Commented Apr 18, 2019 at 21:38
  • 1
    Why do you need to sort if walk() is depth-first?
    – shmosel
    Commented Jun 23, 2021 at 18:29
  • 3
    @shmosel Because forEach will start deleting from the beginning and will not be able to delete the directory with the contents. Therefore, the tree must be turned in the opposite direction and deleted from the end. Commented Aug 12, 2021 at 20:40
27

In JDK 7 you could use Files.walkFileTree() and Files.deleteIfExists() to delete a tree of files. (Sample: http://fahdshariff.blogspot.ru/2011/08/java-7-deleting-directory-by-walking.html)

In JDK 6 one possible way is to use FileUtils.deleteQuietly from Apache Commons which will remove a file, a directory, or a directory with files and sub-directories.

2
26

Using Apache Commons-IO, it is following one-liner:

import org.apache.commons.io.FileUtils;

FileUtils.forceDelete(new File(destination));

This is (slightly) more performant than FileUtils.deleteDirectory.

3
  • 1
    group: 'commons-io', name: 'commons-io', version: '2.+' - useful Commented Oct 20, 2016 at 17:49
  • Sweet answer, thanks
    – stavros.3p
    Commented Jun 18, 2022 at 16:42
  • 1
    Can you explain the "slightliness"? Commented Jun 15, 2023 at 7:56
17

As mentioned, Java is unable to delete a folder containing files, so first delete the files and then the folder.

Here's a simple example to do this:

import org.apache.commons.io.FileUtils;



// First, remove files from into the folder 
FileUtils.cleanDirectory(folder/path);

// Then, remove the folder
FileUtils.deleteDirectory(folder/path);

Or:

FileUtils.forceDelete(new File(destination));
13

One more choice is to use Spring's org.springframework.util.FileSystemUtils relevant method which will recursively delete all content of the directory.

File directoryToDelete = new File(<your_directory_path_to_delete>);
FileSystemUtils.deleteRecursively(directoryToDelete);

That will do the job!

1
  • 1
    it works, short and smart way!!
    – Noshaf
    Commented Sep 23, 2022 at 14:24
11

This is the best solution for Java 7+:

public static void deleteDirectory(String directoryFilePath) throws IOException
{
    Path directory = Paths.get(directoryFilePath);

    if (Files.exists(directory))
    {
        Files.walkFileTree(directory, new SimpleFileVisitor<Path>()
        {
            @Override
            public FileVisitResult visitFile(Path path, BasicFileAttributes basicFileAttributes) throws IOException
            {
                Files.delete(path);
                return FileVisitResult.CONTINUE;
            }

            @Override
            public FileVisitResult postVisitDirectory(Path directory, IOException ioException) throws IOException
            {
                Files.delete(directory);
                return FileVisitResult.CONTINUE;
            }
        });
    }
}
1
  • I prefer this because exception can be thrown. The short stream version in other comment requires reverse of whole stream and ignore the returned boolean of File.delete.
    – Franz Wong
    Commented May 2, 2024 at 6:24
9

My basic recursive version, working with older versions of JDK:

public static void deleteFile(File element) {
    if (element.isDirectory()) {
        for (File sub : element.listFiles()) {
            deleteFile(sub);
        }
    }
    element.delete();
}
1
  • 2
    This will throw a NullPointerException if there is an I/O exception while reading the directory. The code should check whether listFiles() returns null, rather than calling isDirectory().
    – mernst
    Commented Oct 31, 2017 at 23:57
7

Guava 21+ to the rescue. Use only if there are no symlinks pointing out of the directory to delete.

com.google.common.io.MoreFiles.deleteRecursively(
      file.toPath(),
      RecursiveDeleteOption.ALLOW_INSECURE
) ;

(This question is well-indexed by Google, so other people usig Guava might be happy to find this answer, even if it is redundant with other answers elsewhere.)

5

I like this solution the most. It does not use 3rd party library, instead it uses NIO2 of Java 7.

/**
 * Deletes Folder with all of its content
 *
 * @param folder path to folder which should be deleted
 */
public static void deleteFolderAndItsContent(final Path folder) throws IOException {
    Files.walkFileTree(folder, new SimpleFileVisitor<Path>() {
        @Override
        public FileVisitResult visitFile(Path file, BasicFileAttributes attrs) throws IOException {
            Files.delete(file);
            return FileVisitResult.CONTINUE;
        }

        @Override
        public FileVisitResult postVisitDirectory(Path dir, IOException exc) throws IOException {
            if (exc != null) {
                throw exc;
            }
            Files.delete(dir);
            return FileVisitResult.CONTINUE;
        }
    });
}
5

You can try this

public static void deleteDir(File dirFile) {
    if (dirFile.isDirectory()) {
        File[] dirs = dirFile.listFiles();
        for (File dir: dirs) {
            deleteDir(dir);
        }
    }
    dirFile.delete();
}
4

2020 here :)

With Apache commons io FileUtils, contrary to the "pure" Java variants a folder does not need to be empty to be deleted. To give you a better overview I list the variants here, the following 3 may throw exceptions for various reasons:

  • cleanDirectory: Cleans a directory without deleting it
  • forceDelete: Deletes a file. If file is a directory, delete it and all sub-directories
  • forceDeleteOnExit: Schedules a file to be deleted when JVM exits. If file is directory delete it and all sub-directories. Not recommended for running servers as JVM may not exit any time soon ...

The following variant never throws exceptions (even if the file is null !)

  • deleteQuietly: Deletes a file, never throwing an exception. If it is a directory, delete it and all sub-directories.

One more thing to know is dealing with symbolic links, it will delete the symbolic link and not the target folder... be careful.

Also keep in mind that deleting a large file or folder can be a blocking operation for a good while ... so if you do not mind having it run async do it (in a background thread via executor for example).

2

If you have subfolders, you will find troubles with the Cemron answers. so you should create a method that works like this:

private void deleteTempFile(File tempFile) {
        try
        {
            if(tempFile.isDirectory()){
               File[] entries = tempFile.listFiles();
               for(File currentFile: entries){
                   deleteTempFile(currentFile);
               }
               tempFile.delete();
            }else{
               tempFile.delete();
            }
        getLogger().info("DELETED Temporal File: " + tempFile.getPath());
        }
        catch(Throwable t)
        {
            getLogger().error("Could not DELETE file: " + tempFile.getPath(), t);
        }
    }
2

You can use FileUtils.deleteDirectory. JAVA can't delete the non-empty foldres with File.delete().

2

In this

index.delete();
    
if (!index.exists()) {
    index.mkdir();
}

you are calling

if (!index.exists()) {
    index.mkdir();
}

after

index.delete();

This means that you are creating the file again after deleting File.delete() returns a boolean value.So if you want to check then do System.out.println(index.delete()); if you get true then this means that file is deleted

File index = new File("/home/Work/Indexer1");
if (!index.exists()) {
    index.mkdir();
} else {
    System.out.println(index.delete());//If you get true then file is deleted
    if (!index.exists()) {
        index.mkdir();// here you are creating again after deleting the file
    }
}

from the comments given below,the updated answer is like this

File f = new File("full_path"); // full path like c:/home/ri
if(f.exists()) {
    f.delete();
} else {
    try {
        //f.createNewFile();//this will create a file
        f.mkdir();//this create a folder
    } catch (Exception e) {
        e.printStackTrace();
    }
}
0
1

directry cannot simply delete if it has the files so you may need to delete the files inside first and then directory

public class DeleteFileFolder {

public DeleteFileFolder(String path) {

    File file = new File(path);
    if(file.exists())
    {
        do{
            delete(file);
        }while(file.exists());
    }else
    {
        System.out.println("File or Folder not found : "+path);
    }

}
private void delete(File file)
{
    if(file.isDirectory())
    {
        String fileList[] = file.list();
        if(fileList.length == 0)
        {
            System.out.println("Deleting Directory : "+file.getPath());
            file.delete();
        }else
        {
            int size = fileList.length;
            for(int i = 0 ; i < size ; i++)
            {
                String fileName = fileList[i];
                System.out.println("File path : "+file.getPath()+" and name :"+fileName);
                String fullPath = file.getPath()+"/"+fileName;
                File fileOrFolder = new File(fullPath);
                System.out.println("Full Path :"+fileOrFolder.getPath());
                delete(fileOrFolder);
            }
        }
    }else
    {
        System.out.println("Deleting file : "+file.getPath());
        file.delete();
    }
}
0
1

we can use the spring-core dependency;

boolean result = FileSystemUtils.deleteRecursively(file);
1

Most of answers (even recent) referencing JDK classes rely on File.delete() but that is a flawed API as the operation may fail silently.
The java.io.File.delete() method documentation states :

Note that the java.nio.file.Files class defines the delete method to throw an IOException when a file cannot be deleted. This is useful for error reporting and to diagnose why a file cannot be deleted.

As replacement, you should favor Files.delete(Path p) that throws an IOException with a error message.

The actual code could be written such as :

Path index = Paths.get("/home/Work/Indexer1");

if (!Files.exists(index)) {
    index = Files.createDirectories(index);
} else {

    Files.walk(index)
         .sorted(Comparator.reverseOrder())  // as the file tree is traversed depth-first and that deleted dirs have to be empty  
         .forEach(t -> {
             try {
                 Files.delete(t);
             } catch (IOException e) {
                 // LOG the exception and potentially stop the processing

             }
         });
    if (!Files.exists(index)) {
        index = Files.createDirectories(index);
    }
}
1

Here is a simple way to do it :

public void deleteDirectory(String directoryPath)  {
      new Thread(new Runnable() {
          public void run() {
             for(String e: new File(directoryPath).list()) {
                 if(new File(e).isDirectory()) 
                     deleteDirectory(e);
                 else 
                     new File(e).delete();
             }
          }
      }).start();
  }
1
You can simply remove the directory which contains a single file or multiple files using the apache commons library.

gradle import :

implementation group: 'commons-io', name: 'commons-io', version: '2.5'

File file=new File("/Users/devil/Documents/DummyProject/hello.txt");
    
 File parentDirLocation=new File(file.getParent);
    
//Confirming the file parent is a directory or not.
             
if(parentDirLocation.isDirectory){
    
    
 //after this line the mentioned directory will deleted.
 FileUtils.deleteDirectory(parentDirLocation);
   
 }
0
private void deleteFileOrFolder(File file){
    try {
        for (File f : file.listFiles()) {
            f.delete();
            deleteFileOrFolder(f);
        }
    } catch (Exception e) {
        e.printStackTrace(System.err);
    }
}
0
        import org.apache.commons.io.FileUtils;

        List<String> directory =  new ArrayList(); 
        directory.add("test-output"); 
        directory.add("Reports/executions"); 
        directory.add("Reports/index.html"); 
        directory.add("Reports/report.properties"); 
        for(int count = 0 ; count < directory.size() ; count ++)
        {
        String destination = directory.get(count);
        deleteDirectory(destination);
        }





      public void deleteDirectory(String path) {

        File file  = new File(path);
        if(file.isDirectory()){
             System.out.println("Deleting Directory :" + path);
            try {
                FileUtils.deleteDirectory(new File(path)); //deletes the whole folder
            } catch (IOException e) {
                // TODO Auto-generated catch block
                e.printStackTrace();
            }
        }
        else {
        System.out.println("Deleting File :" + path);
            //it is a simple file. Proceed for deletion
            file.delete();
        }

    }

Works like a Charm . For both folder and files . Salam :)

0

You can make recursive call if sub directories exists

import java.io.File;

class DeleteDir {
public static void main(String args[]) {
deleteDirectory(new File(args[0]));
}

static public boolean deleteDirectory(File path) {
if( path.exists() ) {
  File[] files = path.listFiles();
  for(int i=0; i<files.length; i++) {
     if(files[i].isDirectory()) {
       deleteDirectory(files[i]);
     }
     else {
       files[i].delete();
     }
  }
}
return( path.delete() );
}
}
0

You may also use this to delete a folder that contains subfolders and files.

  1. Fist, create a recursive function.

     private void recursiveDelete(File file){
    
             if(file.list().length > 0){
                 String[] list = file.list();
                 for(String is: list){
                     File currentFile = new File(file.getPath(),is);
                     if(currentFile.isDirectory()){
                             recursiveDelete(currentFile);
                     }else{
                         currentFile.delete();
                     }
                 }
             }else {
                 file.delete();
             }
         }
    
  2. then, from your initial function use a while loop to call the recursive.

     private boolean deleteFolderContainingSubFoldersAndFiles(){
    
             boolean deleted = false;
             File folderToDelete = new File("C:/mainFolderDirectoryHere");
    
             while(folderToDelete != null && folderToDelete.isDirectory()){
                 recursiveDelete(folderToDelete);
             }
    
             return deleted;
         }
    
0
List<File> temp = Arrays.asList(new File("./DIRECTORY").listFiles());
for (int i = 0; i < temp.size(); i++)
{
    temp.get(i).delete();
}
-1

Some of these answers seem unnecessarily long:

if (directory.exists()) {
    for (File file : directory.listFiles()) {
        file.delete();
    }
    directory.delete();
}

Works for sub directories too.

0
-2

Remove it from else part

File index = new File("/home/Work/Indexer1");
if (!index.exists())
{
     index.mkdir();
     System.out.println("Dir Not present. Creating new one!");
}
index.delete();
System.out.println("File deleted successfully");
0
-3

You can use this function

public void delete()    
{   
    File f = new File("E://implementation1/");
    File[] files = f.listFiles();
    for (File file : files) {
        file.delete();
    }
}
2
  • It works fine with a directory with all closed files. But when tried on directory with open files it doesnt work. Can you help me find a way to delete folder inspite of open files Commented Mar 22, 2016 at 5:30
  • 2
    This won't delete non-empty subdirectories.
    – Pang
    Commented May 14, 2016 at 4:39

Not the answer you're looking for? Browse other questions tagged or ask your own question.