3

Hola tengo el siguiente código:

var fs = require('fs');

function search(text) {
    var response = [];

    fs.readdir('documents',(err, files) => {
        files.forEach(file => {
            if(file.toLowerCase().indexOf(text.toLowerCase()) != -1){
                console.log("Encontrado " + file);
                response.push(file);
            }
        })
    });

    return response;
}

exports.search = search;

Lo que hace es buscar en una carpeta todos los archivos que contengan en su nombre(file) una cierta subcadena(text), la idea es que todos los archivos que coincidan deben almacenar su nombre en el array response el problema es que al retornar el array este me aparece vacío (me imprime [])

Como pueden ver tengo un console.log para ver si es que en algún momento se encuentra un match y ya confirme que esto ocurre, ahora el problema es que pareciera que la linea del push jamas se ejecutara...

2

1 respuesta 1

3

Estas haciendo una de funciones asíncronas, y, por lo tanto, cuando llegas a tu return, no se ha ejecutado absolutamente ninguna línea de tu filtro, por lo que devuelves un array vacío.

2 opciones:

1. Utiliza callbacks

function search(text, callback ) {
  var response = [];

  fs.readdir('documents',(err, files) => {
    files.forEach( file => {
      if(file.toLowerCase().indexOf(text.toLowerCase()) != -1){
        console.log("Encontrado " + file);
        response.push(file);
      }
    } )
    callback( response );
  } );
}

2. Usa funciones síncronas

function search(text) {
  var response = [ ],
      files = fs.readdirSync( 'documents' );

  files.forEach(file => {
    if(file.toLowerCase().indexOf(text.toLowerCase()) != -1){
      console.log("Encontrado " + file);
      response.push(file);
    }
  })

  return response;
}
3
  • Wow muchas gracias ! Commented el 3 abr. 2017 a las 5:12
  • Se puede usar filter e includes para hacerlo más práctico: const response = files.filter(file => file.toLowerCase().includes(text.toLowerCase())).
    – gugadev
    Commented el 3 abr. 2017 a las 16:48
  • @GustavoGarcía gracias lo tomare en cuenta Commented el 3 abr. 2017 a las 17:49

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.