Skip to main content
se añadieron 3 caracteres en el cuerpo
Origen Enlace
E. Betanzos
  • 5k
  • 1
  • 10
  • 24

Como te comentaba el problema que estás teniendo es no estás iterando todo el contenido del directorio actual de antes realizar una búsqueda recursiva en sus sub-directorios.

Aquí está el problema

if (pos < lista.length)

Luego haces esto esperando iterar el contenido del directorio actual con una llamada recursiva al método

buscarArchivo(fichero, lista[pos+1], pos)

Esto tienen muchos errores puesto que a lo sumo estarías haciendo una búsqueda en profundidad a través de una sola rama de la estructura de directorios. Pero en el momento que lo que encuentre sea un archivo el método finalizará, siendo este el buscado o no, aunque queden archivos y directorios por consultar.

Lo que haría yo es: itero sobre todo el contenido del directorio para buscar el archivo. Si el elemento analizado es un archivo y su nombre conincide con el buscado lo devuelvo. Si es un directorio entonces aplico sobre este el mismo proceso de búsqueda recursivamente.

Sería algo como esto:

private static File buscarArchivo(String nombreFichero, File directorioFile) throws IOException {
        
    File[] lista = directorioFile.listFiles();

    if (lista == null || lista.lenght == 0) {
        // Si el directorio está vacío devuelvo null porque no lo encontré
        return null;
    }
    
    // Itero el contenido del directorio para buscar el archivo en este
    for (File f : lista) {
        if (f.isDirectory()) {
            // Si f es un directorio busco recursivamente dentro de él
            File archivoEncontrado = buscarArchivo(nombreFichero, f);
            // Si encontró el archivo lo devuelvo, sino continúo con la iteración
            if (archivoEncontrado != null) {
                return archivoEncontrado;
            }
        } else if (f.getName().equals(nombreFichero)) {// Si f es un archivo compruebo si es el buscado
            // Si lo encontré lo devuelvo, sino continúo con la iteración
            return f;
        }
    }
}

Este método devolverá una instancia de File que representa el archivo encontrado, o null si no se encuentra el archivo.

Ten en cuenta que aunque apliques una iteración para examinar el contenido del directorio actual, el proceso de búsqueda es recursivo porque el método buscarArchivo() se llama a sí mismo una y otra vez.

Como te comentaba el problema que estás teniendo es no estás iterando todo el contenido del directorio actual antes realizar una búsqueda recursiva en sus sub-directorios.

Aquí está el problema

if (pos < lista.length)

Luego haces esto esperando iterar el contenido del directorio actual con una llamada recursiva al método

buscarArchivo(fichero, lista[pos+1], pos)

Esto tienen muchos errores puesto que a lo sumo estarías haciendo una búsqueda en profundidad a través de una sola rama de la estructura de directorios. Pero en el momento que lo que encuentre sea un archivo el método finalizará, siendo este el buscado o no, aunque queden archivos y directorios por consultar.

Lo que haría yo es: itero sobre todo el contenido del directorio para buscar el archivo. Si el elemento analizado es un archivo y su nombre conincide con el buscado lo devuelvo. Si es un directorio entonces aplico sobre este el mismo proceso de búsqueda recursivamente.

Sería algo como esto:

private static File buscarArchivo(String nombreFichero, File directorioFile) throws IOException {
        
    File[] lista = directorioFile.listFiles();

    if (lista == null || lista.lenght == 0) {
        // Si el directorio está vacío devuelvo null porque no lo encontré
        return null;
    }
    
    // Itero el contenido del directorio para buscar el archivo en este
    for (File f : lista) {
        if (f.isDirectory()) {
            // Si f es un directorio busco recursivamente dentro de él
            File archivoEncontrado = buscarArchivo(nombreFichero, f);
            // Si encontró el archivo lo devuelvo, sino continúo con la iteración
            if (archivoEncontrado != null) {
                return archivoEncontrado;
            }
        } else if (f.getName().equals(nombreFichero)) {// Si f es un archivo compruebo si es el buscado
            // Si lo encontré lo devuelvo, sino continúo con la iteración
            return f;
        }
    }
}

Este método devolverá una instancia de File que representa el archivo encontrado, o null si no se encuentra el archivo.

Ten en cuenta que aunque apliques una iteración para examinar el contenido del directorio actual, el proceso de búsqueda es recursivo porque el método buscarArchivo() se llama a sí mismo una y otra vez.

Como te comentaba el problema que estás teniendo es no estás iterando todo el contenido del directorio actual de antes realizar una búsqueda recursiva en sus sub-directorios.

Aquí está el problema

if (pos < lista.length)

Luego haces esto esperando iterar el contenido del directorio actual con una llamada recursiva al método

buscarArchivo(fichero, lista[pos+1], pos)

Esto tienen muchos errores puesto que a lo sumo estarías haciendo una búsqueda en profundidad a través de una sola rama de la estructura de directorios. Pero en el momento que lo que encuentre sea un archivo el método finalizará, siendo este el buscado o no, aunque queden archivos y directorios por consultar.

Lo que haría yo es: itero sobre todo el contenido del directorio para buscar el archivo. Si el elemento analizado es un archivo y su nombre conincide con el buscado lo devuelvo. Si es un directorio entonces aplico sobre este el mismo proceso de búsqueda recursivamente.

Sería algo como esto:

private static File buscarArchivo(String nombreFichero, File directorioFile) throws IOException {
        
    File[] lista = directorioFile.listFiles();

    if (lista == null || lista.lenght == 0) {
        // Si el directorio está vacío devuelvo null porque no lo encontré
        return null;
    }
    
    // Itero el contenido del directorio para buscar el archivo en este
    for (File f : lista) {
        if (f.isDirectory()) {
            // Si f es un directorio busco recursivamente dentro de él
            File archivoEncontrado = buscarArchivo(nombreFichero, f);
            // Si encontró el archivo lo devuelvo, sino continúo con la iteración
            if (archivoEncontrado != null) {
                return archivoEncontrado;
            }
        } else if (f.getName().equals(nombreFichero)) {// Si f es un archivo compruebo si es el buscado
            // Si lo encontré lo devuelvo, sino continúo con la iteración
            return f;
        }
    }
}

Este método devolverá una instancia de File que representa el archivo encontrado, o null si no se encuentra el archivo.

Ten en cuenta que aunque apliques una iteración para examinar el contenido del directorio actual, el proceso de búsqueda es recursivo porque el método buscarArchivo() se llama a sí mismo una y otra vez.

Origen Enlace
E. Betanzos
  • 5k
  • 1
  • 10
  • 24

Como te comentaba el problema que estás teniendo es no estás iterando todo el contenido del directorio actual antes realizar una búsqueda recursiva en sus sub-directorios.

Aquí está el problema

if (pos < lista.length)

Luego haces esto esperando iterar el contenido del directorio actual con una llamada recursiva al método

buscarArchivo(fichero, lista[pos+1], pos)

Esto tienen muchos errores puesto que a lo sumo estarías haciendo una búsqueda en profundidad a través de una sola rama de la estructura de directorios. Pero en el momento que lo que encuentre sea un archivo el método finalizará, siendo este el buscado o no, aunque queden archivos y directorios por consultar.

Lo que haría yo es: itero sobre todo el contenido del directorio para buscar el archivo. Si el elemento analizado es un archivo y su nombre conincide con el buscado lo devuelvo. Si es un directorio entonces aplico sobre este el mismo proceso de búsqueda recursivamente.

Sería algo como esto:

private static File buscarArchivo(String nombreFichero, File directorioFile) throws IOException {
        
    File[] lista = directorioFile.listFiles();

    if (lista == null || lista.lenght == 0) {
        // Si el directorio está vacío devuelvo null porque no lo encontré
        return null;
    }
    
    // Itero el contenido del directorio para buscar el archivo en este
    for (File f : lista) {
        if (f.isDirectory()) {
            // Si f es un directorio busco recursivamente dentro de él
            File archivoEncontrado = buscarArchivo(nombreFichero, f);
            // Si encontró el archivo lo devuelvo, sino continúo con la iteración
            if (archivoEncontrado != null) {
                return archivoEncontrado;
            }
        } else if (f.getName().equals(nombreFichero)) {// Si f es un archivo compruebo si es el buscado
            // Si lo encontré lo devuelvo, sino continúo con la iteración
            return f;
        }
    }
}

Este método devolverá una instancia de File que representa el archivo encontrado, o null si no se encuentra el archivo.

Ten en cuenta que aunque apliques una iteración para examinar el contenido del directorio actual, el proceso de búsqueda es recursivo porque el método buscarArchivo() se llama a sí mismo una y otra vez.