Как передать параметр в Promise?

Question

[grearte]

Нужно передать переменную partFilename из router.get в getFilename.
Возможно, что то не так делаю, так как полистал документацию и гугол, промиса с параметрами не нашел.
Если этот вариант неактуален, то опишите альтернативу.

var getFilename = new Promise(<b>partFilename</b>, (resolve, reject) => {
  var files = fs.readdirSync(uploadPath);
  files.forEach(file => {
      var comparedPart = file.split('.')[0];
      if (comparedPart == partFilename) {
        resolve(file);
      }
    });
  reject(new Error('404'));
  });

router.get('/:id', function(req, res, next) {
  var partFilename = req.params.id;
  getFilename(<b>fileName</b>)
    .then(
      result => {
        res.download(uploadPath + result, result, function(err) {
          if (err) {
            console.log(err);
          }
        });
      },
      error => {
        res.send(error.message);
      }
    );
});

Comments

[Aves]

Серьезно?

const doStuff = param => new Promise(resolve => resolve(param));
doStuff('data').then(console.log);

Answer 1

[de1m]

Я не тестировал, но должно быть как-то так:

var getFilename = function (partFilename) {
    return new Promise((resolve, reject) => {
        var files = fs.readdirSync(uploadPath);
        files.forEach(file => {
            var comparedPart = file.split('.')[0];
            if (comparedPart == partFilename) {
                resolve(file);
            }
        });
        reject(new Error('404'));
    });
}

Но вообще, я бы асинхронную функцию тут использовал

var getFilename = function (partFilename) {
    return new Promise((resolve, reject) => {
        var files = fs.readdir(uploadPath, (err, files) => {
            if (err) {
                reject(new Error('404'));
            } else {
                files.forEach(file => {
                    var comparedPart = file.split('.')[0];
                    if (comparedPart == partFilename) {
                        resolve(file);
                    }
                });
            }
        })
    });
}

Comments

[grearte]

поправь, чтоб return перед new Promise был.
Если кому то будет интересна эта ветка, вот ссыль где подробнее написано про Promise Тык