Concantenar en Javascript

[marimeli]

Hola comunidad, tengo un problema para concatenar un array de archivos. No estoy logrando concatenarlo y no logro encontrar el error. ¿Podrían ayudarme?
Lo que estoy buscando es que esa función me retorne un array de archivos (todavía sin filtrar), aplicando recursividad y promesas, con métodos asíncronos. Pero no estoy segura de estar haciéndolo bien.

const { isAbsolute, resolve, join, extname, dirname, basename } = require('path')
const { lstat, readFile, readdir } = require('fs-extra');

//Función que valida si la ruta ingresada es un archivo o directorio. Debería retornar un array de archivos
const getFiles = (path) => {
    return lstat(path) // stat about a file
        .then(stat => {
            let arrayLinks3 = []
            if (stat.isFile()) { //verifica si es archivo
                return [path]
            } else {
                return readdir(path) //Sino es carpeta, entonces la lee
                    .then(files => {
                        const arrayOfPromises = files.map(file => {
                            return getFiles(resolve(path, file))
                        })
                        Promise.all(arrayOfPromises)
                            .then(respuestas => {
                                respuestas.forEach(res => {
                                    arrayLinks3 = arrayLinks3.concat(res)
                                    console.log('concatenar', arrayLinks3)
                                })
                                /* console.log('f', arrayLinks3);
                                return arrayLinks3; */
                            })
                    })
            }
        })
        .catch(err => {
            console.log(`El archivo o file no existe ${err.path}`);
            return err;
        });
};

getFiles(process.cwd() + '/test/directory')
    .then(a => console.log('j', a))

También está subido en mi repositorio:

marimeli/lim20181-Track-FE-markdown-list/blob/master/index.js

[fabian]

Hola @marimeli,

Dos problemas por lo que veo rápidamente :

1. Clausura : Está bien que quieras acumular todo en un arreglo fuera de las respuestas de las promesas, pero… tienes que tener claro el momento en que lo retornarás, cómo lo retornarás y los momentos en que pasará eso. Recuerda que los “then” de una promesa se ejecutan una vez que, por ejemplo, se ha leido el archivo, que es algo MUY lento para la computadora y esa función no tendrá su resultado secuencialmente como pensamos.
2. Promesas y su retorno : Al retornar una promesa o querer que el resultado aparezca en el siguiente then (ver la parte de concatenar promesas en una de las respuestas de acá en la comunidad) hay que tener cuidado de que en TODOS los casos estemos retornando respuestas parecidas (esto es, del mismo tipo). Lo que se ve acá es que, si es archivo retornas la ruta dentro de un arreglo de 1 elemento, pero si es un directorio, ojo con esto, estás retornando una promesa que no retorna nada, fíjate que el Promise.all que tienes no lo retornas, pero si dentro de él estás retornando el arreglo con las rutas, pero no hay nadie que le pueda hacer then a esa promesa.

Entonces, como receta del médico, trata de retornar esa promesa “Promise.all” y ve que sucede, luego ve bien los pasos a seguir para que la concatenación de promesas te de el resultado que esperas.

Cualquier cosa no dudes en seguir el hilo para que te resulte lo que estás haciendo y entiendas a fondo las promesas.

[marimeli]

Muchas gracias, @fabian. Logré solucionarlo y ya me quedó más claro el tema de promises.