2

Bueno, estoy intentado borrar carpetas de un directorio recursivamente:

  • Parent directory
    • Carpeta 1
      • Sub carpeta 1
      • Sub carpeta 2
    • Carpeta 2

Se supone que todas esas carpetas están vacías, pues borrarlas todas menos el parent directory. Como podría hacerlo?

4 respuestas 4

3

Propongo que el método en lugar de reportar en texto lo que hace devuelva un List<File> de todas las carpetas que fueron eliminadas o de todas las carpetaas que van a ser eliminadas para así tener un mejor control (pruebas unitarias/de integración) sobre el mismo.

El código podría ser como sigue:

public List<File> buscaCarpetasVacias(File carpetaRaiz) {
    //siempre retornamos al menos una lista vacía
    List<File> resultado = new ArrayList<>();
    if (carpetaRaiz.isDirectory()) {
        File[] carpetas = carpetaRaiz.listFiles(File::isDirectory);
        for (File carpeta : carpetas) {
            if (carpeta.listFiles().length == 0) {
                resultado.add(carpeta);
            } else {
                resultado.addAll( buscaCarpetasVacias(carpeta) );
            }
        }
    }
    return resultado;
}

public void eliminaCarpetasVacias(File carpetaRaiz) {
    List<File> listaCarpetasVacias = buscaCarpetasVacias(carpetaRaiz);
    //de la implementación anterior, sabemos que
    //el método por lo menos retornará una lista vacía
    //no necesitamos hacer una validaciób en caso de nulos
    for (File carpeta : listaCarpetasVacias) {
        //es mejor usar un logger porque así es independiente
        //de la plataforma donde se ejecute este código
        logger.debug(String.format("Borrando la carpeta %s", carpeta.getAbsolutePath()));
        carpeta.delete();
    }
}
1

Me autorespondo. Con esto, podeis borrar recursivamente todos los directorios y subdirectorios sí estan vacios.

public StringBuilder utilsCheckedFilesCorrectly(File[] files, String option) throws IOException {
    String selectedOption = option;
    StringBuilder sb = new StringBuilder();
    for (File file : files) {
        if (file.isDirectory()) {
            utilsCheckedFilesCorrectly(file.listFiles(), selectedOption);
            if (file.list().length == 0) {
                file.delete();
            } else {
                sb.append("La carpeta: ").append(file.getName()).append(" no está del todo vacía");
            }
        }
    }
    return sb;
}
3
  • 1
    Hay algunos problemas de código con el método, por ejemplo se pierde el StringBuilder que retorna la llamada recursiva de utilsCheckedFilesCorrectly y que el método hace 2 cosas en lugar de solo 1: borra carpetas vacías y devuelve un StringBuilder indicando un reporte (que en realidad es de la carpeta raíz, no de las subcarpetas) de las carpetas eliminadas.
    – user227
    Commented el 31 ago. 2016 a las 15:13
  • Tampoco se entiende qué hace el argumento option. Todo método debería documentar claramente que argumentos recibe, qué hace y qué retorna. Otra cosa objetable es que el nombre del método utilsCheckedFilesCorrectly no es nada descriptivo
    – leonbloy
    Commented el 31 ago. 2016 a las 15:15
  • Entiende que el método esta adaptado a mi programa, puedes hacerlo tipo void
    – Cazs
    Commented el 31 ago. 2016 a las 22:05
1

También puedes probar con FileSystemUtils Spring El ejemplo inferior copia y elimina recursivamente.

Ej:

 for(File f : files){
    patherOrigin = FilenameUtils.separatorsToUnix(filesOrigin.getPath()+"/"+f.getName());
    patherDestination = FilenameUtils.separatorsToUnix(filesDestination.getPath()+"/"+f.getName());
    ori = new File(patherOrigin);                                           
    desti = new File(patherDestination);
    org.springframework.util.FileSystemUtils.copyRecursively(ori, desti);                                           
    boolean delete = org.springframework.util.FileSystemUtils.deleteRecursively(ori);
    //XXX hacemos algubna cosa si el tema ha fallado. ??
     if(delete) LOGGER.info("File deleted "+patherOrigin); 
  }

filenameUtils de apache

2
  • 1
    Puedes crear una instancia de File utilizando otro File que es la carpeta contenedora, así: File ori = new File(filesOrigin, f); y ahorrarte los problemas de separadores de carpetas
    – user227
    Commented el 1 sept. 2016 a las 15:37
  • ¡me lo apunto! :)
    – ZaoTaoBao
    Commented el 1 sept. 2016 a las 15:45
0

Hay que recordar que no se pueden borrar folders que contienen archivos, por lo que puedes tratar borrando todos en un loop o detectando si están vacíos:

File file  = new File("Directorio principal");

    File[] fList = file.listFiles();
    for (File file : fList) {

if(file.isDirectory()){
        if(file.list().length>0){
            System.out.println("El directorio " + file.getPath() + " no esta vacio!");
        }else{
            System.out.println("El directorio " + file.getPath() + " esta vacio!, se procede a borrar.");
            file.delete();
        }

    }

 }
1
  • No se cubre la eliminación de las sub carpetas tal como pide el OP.
    – user227
    Commented el 31 ago. 2016 a las 17:46

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.