如何正确使用嵌套的异步代码?

时间:2018-10-08 09:37:37

标签: javascript node.js asynchronous micro

我已经使用micro创建了一个小api。

运行localhost:3000/get-status应该返回数据对象。 到目前为止,console.log()正在打印预期的对象。

但是在浏览器上我得到Endpoint not found,在服务器上我得到错误Si7021 reset failed: Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client

我的getStatus()函数在做什么?我想我把承诺和异步的东西混在一起了。也许我不必要地嵌套了函数...

const { send } = require('micro')
const { router, get } = require('microrouter')
const Si7021 = require('si7021-sensor')

const getStatus = async (req, res) => {
  const si7021 = new Si7021({ i2cBusNo: 1 })
  const readSensorData = async () => {
    const data = await si7021.readSensorData()
    console.log(data)
    send(res, 201, { data })
  }

  si7021.reset()
    .then((result) => readSensorData())
    .catch((err) => console.error(`Si7021 reset failed: ${err} `))
}

const notFound = (req, res) => {
  console.log('NOT FOUND.')
  send(res, 404, 'Endpoint not found')
}

module.exports = router(
  get('/get-status', getStatus),
  get('/*', notFound)
)

1 个答案:

答案 0 :(得分:2)

看起来您的处理程序返回了一个立即解决的承诺。您可以尝试这样重写最后一行吗?

return si7021.reset()
    .then((result) => readSensorData())
    .catch((err) => console.error(`Si7021 reset failed: ${err}`));

可以更清楚地写成:

const getStatus = async (req, res) => {
  try {
    const si7021 = new Si7021({ i2cBusNo: 1 });
    await si7021.reset();
    const data = await si7021.readSensorData();
    console.log(data);
    return send(res, 201, { data });
  } catch (e) {
    console.error(`Si7021 reset failed: ${err}`)
  }
}

但是您可能也想在catch处理程序中发送一些东西。

还请注意,诺言是从

返回的
si7021.reset()
    .then((result) => readSensorData());

不仅不会在.reset失败时拒绝。它还拒绝readSensorData失败。因此您的错误消息不完整。总而言之,我宁愿按照以下方式推荐一些东西:

const getStatus = async (req, res) => {
  try {
    const si7021 = new Si7021({ i2cBusNo: 1 });
    await si7021.reset();
    const data = await si7021.readSensorData();
    console.log(data);
    send(res, 201, { data });
  } catch (e) {
    console.error(`Si7021 failed: ${err.message}`);
    send(res, 500, err.message);
  }
}