HTTP请求回调返回response.body赋值为undefined问题排查

出现undefined是典型的异步执行时序错误，和回调写法本身没有关系：

• 你用的request是异步非阻塞HTTP库，发起请求后不会等待接口响应，会直接往下执行后续同步代码
• 你的createDynamicLink函数本身没有定义返回值，同步执行阶段默认返回undefined，所以外层赋值拿到的就是这个默认值
• 回调里写的return returnValue，只是把值返回给了request库的内部逻辑，根本不会传到外层的dynamicLink变量；等你执行外层console.log(dynamicLink)的时候，HTTP请求还没完成，回调函数甚至还没被触发。

方案1：直接在回调内处理依赖返回值的逻辑 #

这是回调模式下的标准写法，不要尝试在异步请求的同步外层接值，所有需要用到短链的逻辑都放到回调内部执行：

function createDynamicLink(callback){
  // 省略你的前置业务逻辑
  request({
    url: dynamicLinksApi,
    method: "POST",
    json: true,
    body: body,
  }, function (error, response, body) {
    // 必须先处理请求错误，不要直接读响应
    if (error) return callback(error);
    callback(null, response.body.shortLink);
  });
}

// 外层不要定义变量接返回值，直接在回调里写后续逻辑
createDynamicLink(function (err, shortLink){
  if (err) {
    console.error('生成短链失败', err);
    return;
  }
  console.log('生成的短链为：', shortLink);
  // 所有依赖shortLink的业务逻辑全部写在这里
}); 

方案2：封装Promise配合async/await使用 #

如果不想写嵌套回调，可以把异步逻辑封装为Promise，用async/await写成线性的同步风格，可读性更好：

// 把原回调版函数封装为Promise
function createDynamicLink(){
  // 省略你的前置业务逻辑
  return new Promise((resolve, reject) => {
    request({
      url: dynamicLinksApi,
      method: "POST",
      json: true,
      body: body,
    }, function (error, response, body) {
      if (error) return reject(error);
      resolve(response.body.shortLink);
    });
  })
}

// 调用时需要在async函数内使用await
async function runBusiness() {
  try {
    const dynamicLink = await createDynamicLink();
    console.log(dynamicLink); // 此处可以正常拿到短链值
    // 后续依赖dynamicLink的逻辑直接往下写即可
  } catch (err) {
    console.error('生成短链失败', err);
  }
}
runBusiness();

关键提醒：JS里所有异步逻辑的结果，都不可能通过同步赋值的方式在外层作用域拿到。依赖异步返回值的逻辑，要么放在回调函数内，要么放在Promise的then回调中，要么放在await语句之后，不要试图在异步调用的同步执行阶段接收返回值。

内容的提问来源于stack exchange，提问作者Recep Oğuzhan Şenoğlu