如何迁移到回调 nodejs

Question

我需要将此代码转换为使用回调的干净代码，因为此代码不允许我在其他地方使用正文信息。

const endpoints = [];

function getDevicesFromPartnerCloud() {
  var options = {
    method: 'GET',
    url: 'https://database-dcda.restdb.io/rest/endpoints',
    headers: {
      'cache-control': 'no-cache',
      'x-apikey': '*****************************'
    }
  };
  request(options, function (error, response, body) {
    var data = JSON.parse(body);
    data.forEach(function(data, index) {
      let endpoint = createSceneEndpoint(data._id, data.name);
      endpoints.push(endpoint);
    });
  });
  return endpoints;
}

Answer 1

我认为最简洁的方法是使用 Promise 来处理异步 request。要记住的最重要的事情之一是函数在理想情况下应该只做一件事。这样，它们更容易测试、推理和重构。我会将实际发出请求的代码拉到一个单独的函数中，并将其 return 主体，然后让您的 getDevicesFromPartnerCloud 调用该新函数，取回数据，并根据需要对其进行处理。最重要的是，这个 "frees" 数据不会卡在 request 回调中，因为你将它包装在一个承诺中，并在数据可用时解析它。

类似于：

const endpoints = [];

function requestDevices() {
  return new Promise(function(resolve, reject) {
    const options = {
      method: 'GET',
      url: 'https://database-dcda.restdb.io/rest/endpoints',
      headers: {
        'cache-control': 'no-cache',
        'x-apikey': '*****************************',
      },
    };

    request(options, function(error, response, body) {
      if (error) {
        reject(error)
      }

      resolve({ response: response, body: body });
    });
  });
}

async function getDevicesFromPartnerCloud() {
  const devicesResponse = await requestDevices();
  const data = JSON.parse(devicesResponse.body);
  data.forEach(function(data, index) {
    const endpoint = createSceneEndpoint(data._id, data.name);
    endpoints.push(endpoint);
  });

  // Do whatever else you need with devicesResponse.body

  return endpoints;
}

如果你想走更多的 es6 方向，也许像

let endpoints;

const requestDevices = () =>
  new Promise((resolve, reject) => {
    request(
      {
        method: 'GET',
        url: 'https://database-dcda.restdb.io/rest/endpoints',
        headers: {
          'cache-control': 'no-cache',
          'x-apikey': '*****************************',
        },
      },
      (error, response, body) => (error ? reject(error) : resolve(body)),
    );
  });

const getDevicesFromPartnerCloud = async () => {
  try {
    const body = await requestDevices();
    const data = JSON.parse(body);
    endpoints = data.map(({ _id, name }) =>
      createSceneEndpoint(_id, name),
    );

    // Do whatever else you need with devicesResponse.body
    // doStuff(body)

    return endpoints;
  } catch (e) {
    console.error(e);
  }
};

如何迁移到回调 nodejs

How to migrate to callback nodejs

callback

request

asynccallback

node.js