При извлечении файла из API и его передаче по электронной почте возникает ошибка, когда файл не найден. - PullRequest
Новая
       16

При извлечении файла из API и его передаче по электронной почте возникает ошибка, когда файл не найден.

0 голосов
/ 13 сентября 2018

Я извлекаю файл Excel из API, и мне нужно прочитать и изменить его с помощью библиотеки xlsx.Я использую узел 8.10 и async / await для этого: 

const report = await getVisitorReport(visitorQueryString, 'test12.xls', uri);

let workbook = XLSX.readFile('test12.xls');

А вот функция getVisitorReport.Обратите внимание, что я разрешаю обещание о завершении для канала: 

async function getVisitorReport(queryString, reportPath, uri) {
return new Promise((resolve, reject) => {
request({
  url: uri,
  qs: queryString,
  headers: {
    'Content-Type': 'application/json',
    'Accept': 'application/json',
    'Authorization': 'Basic ' + new Buffer(process.env.username + ':' + process.env.password, 'utf8').toString('base64')
  }
}, (error, response, body) => {
  if (error) {
    reject(error);
  } else {
    if (response.statusCode === 200) {
      resolve(body);
    } else if (response.statusCode === 409) {
      setTimeout(() => {
        resolve(getVisitorReport(queryString));
      }, response.headers['Retry-After'] * 1000);
    } else {
      reject(response);
    }
  }
}).pipe(fs.createWriteStream(reportPath)).on('finish', resolve(reportPath));
});
}

Файл извлекается и создается правильно.Кажется, что вторая строка XLSX.readFile ('test12.xls') происходит до того, как файл будет сохранен локально.Что я здесь не так делаю?Как мне убедиться, что файл сохранен, прежде чем я попытаюсь его прочитать?Почему .pipe.on («Готово», «Решить») не выполняет этого?Спасибо за помощь!

1 Ответ

0 голосов
/ 13 сентября 2018

Код немедленно вызывает resolve здесь: .on('finish', resolve(reportPath)).

Вы можете предоставить функцию разрешения непосредственно обработчику on: 

.on('finish', resolve)

Или, если вам нужно передать некоторые данные, используйте функцию стрелки: 

.on('finish', () => resolve(args))

редактирование:

если вы сделаете это так: .on('finish', resolve(reportPath)) примерно равно 

resolve(reportPath); // immediately calls resolve which makes a promise resolved 
                     // so await doesn't stop the further code execution
....on('finish', undefined)

Чтобы лучше понять, посмотрите на этот пример: 

const [bad, good] = document.querySelectorAll("button");

bad.addEventListener("click", console.log("I log immediately when the page loads and not at click event"))

good.addEventListener("click", () => console.log("I log properly when button is clicked"))
<button>broken button</button>
<button>good button</button>
...