Node.js+Express中sendVcode1方法挂起未返回结果问题排查

你的sendVcode1方法虽然标记了async，但这里存在一个关键的异步处理错误，导致return 'success'或return 'failed to send'完全没有起到预期作用：

核心问题：回调函数的return无法传递到async函数的返回值 #

Nexmo的sendSms是基于回调函数的异步API，而你的async函数并没有正确等待这个异步操作完成。当sendSms被调用后，async函数会直接执行到函数末尾，返回一个状态为resolved但值为undefined的Promise——因为你没有在async函数里返回任何有效的Promise或值。

而回调函数里的return，仅仅是从回调函数本身返回，并不会成为外层async函数的返回值。这就导致：

1. 你用await调用sendVcode1时，拿到的结果是undefined
2. 虽然控制台会打印successfully sent vcode，但外层的console.log("code send result : ", result)打印的是undefined，后续的if(result === 'success')分支永远不会执行
3. 你感觉应用"挂起"，其实是因为代码逻辑没有按预期走到后续分支，并非真的挂起

修复方案：将回调API包装为Promise #

要让async/await正确工作，你需要把基于回调的sendSms方法包装成一个Promise，这样async函数就能等待异步操作完成，并返回正确的结果。修改后的代码如下：

sendVcode1: async function(vcode, cell, cell_country_code) { 
  switch(process.env.sms) { 
    case "nexmo" : 
      const nexmo = new Nexmo({ 
        apiKey: process.env.nexmoApiKey, 
        apiSecret: process.env.nexmoApiSecret 
      }, { debug: true }); 

      // 用Promise包装sendSms的回调逻辑
      return new Promise((resolve, reject) => {
        nexmo.message.sendSms(
          process.env.nexmo_sender_number, 
          cell_country_code + cell, 
          vcode, 
          {type: 'unicode'}, 
          (err, result) => { 
            console.log("vcode in nexmo : ", vcode); 
            if(err){ 
              console.error("Vcode failed to send : ", err.message); 
              resolve('failed to send'); // 用resolve返回失败状态
            }else{ 
              console.log("successfully sent vcode"); 
              resolve('success'); // 用resolve返回成功状态
            } 
          }
        );
      });
    default:
      // 补充默认分支，避免无返回值的情况
      return Promise.resolve('unsupported sms provider');
  }; 
}

为什么这样能解决问题？ #

• Promise会等待sendSms的回调执行完成，然后通过resolve传递结果
• async函数会返回这个Promise，当你用await调用时，就能拿到resolve传递的success或failed to send
• 后续的console.log("code send result : ", result)会正确打印预期的值，if分支也能正常执行

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