Node.js Promise异步场景下else分支myleadId赋值为undefined问题排查

你遇到的问题本质是异步代码执行顺序导致的：else分支中getChannelIdByName、createLeadOnEnterprise都是异步Promise操作，不会阻塞外层代码执行。你写在if/else之后的console.log和后续逻辑，会在这两个异步操作还没执行完成、还没给myleadId赋值的时候就提前运行，自然拿到的就是undefined。

另外你已经给路由处理函数加了async关键字，完全可以用await语法替代嵌套的then回调，既避免回调地狱，也能完全控制代码执行顺序，修复后代码如下：

app.post('/sendMessage2agent', async (req, res) => {
  try {
    // 用await替代then，保证代码按顺序执行
    const enterpriseId = await getEnterpriseIDbyContact(req.body.recipient, knex);
    const leadId = await getLeadIDbyContact(req.body.sender, knex);
    
    let myleadId;
    if (leadId.length > 0) {                  
      myleadId = leadId;
    } else {
      const channelId = await getChannelIdByName(req.body.channel, knex);
      const newleadId = await createLeadOnEnterprise(req.body.sender, enterpriseId, channelId, knex);
      console.log('newleadId: ' + newleadId);
      myleadId = [{'id': newleadId}];
    }                         
    
    console.log('myleadId: ' + JSON.stringify(myleadId));
    // 此处可添加后续逻辑，myleadId已经拿到有效值

    // 原代码中res.json位置错误，会提前返回结果，应等所有逻辑处理完成再返回
    res.json(req.body);
  } catch (err) {
    // 统一捕获异步操作抛出的错误，避免服务崩溃
    console.error('处理请求出错：', err);
    res.status(500).json({error: '服务内部错误'});
  }
});

如果一定要保留then的写法，需要把所有用到myleadId的逻辑都放到then的回调链内部，保证异步操作执行完成后再运行，不过async/await是目前同类场景下最简洁易维护的实现方式。

内容的提问来源于stack exchange，提问作者Aday Perera