使用Express.js连接SQL Server时返回的错误变量始终为undefined的问题排查

• 回调版本存在异步时序错位：sql.connect属于异步IO操作，传入的回调函数需要等数据库网络交互完成后才会触发执行。但代码中同步逻辑部分的console.log(msg)、外层函数的return msg不会等待回调跑完，执行到这两行时msg还处于未赋值的初始状态，自然返回undefined。回调内部写的return msg只是回调函数自身的返回值，和外层函数的返回链路完全没有关联。
• async/await版本存在冗余逻辑：虽然已经通过await正确等待了连接动作，但多此一举声明了未初始化的msg变量，在连接成功的分支直接返回从未赋值的msg；错误信息仅在catch块中给msg赋值后再返回，这个中间变量完全没有存在的价值。

不需要中转的中间变量，直接在对应分支返回结果即可，既解决赋值时序的问题，也能避免全局变量带来的并发串值隐患（Express是单进程异步模型，多用户同时发起连接请求时，全局变量会被所有请求共享，很容易出现A用户的错误信息被B用户的成功状态覆盖的诡异问题）。

修正后的可运行代码：

// 引入项目中已配置的mssql连接库
const sql = require('mssql');

async function mysqlConnection(userParam) {
  const config = {
    user: userParam.connectionUsername,
    password: userParam.connectionPassword,
    server: userParam.hostname,
    database: userParam.databaseAccount,
  };

  try {
    // 等待连接流程完成，成功则直接返回成功标识
    await sql.connect(config);
    return "数据库连接成功";
  } catch (err) {
    // 直接输出、返回捕获到的错误信息，无需中间变量中转
    console.log('连接错误：', err.message);
    return err.message;
  }
}

• 不要用跨请求的全局变量存储单次连接的临时结果，一旦出现并发请求，变量值会被反复覆盖，排查问题时很难定位原因。
• 如果坚持使用回调写法而非async/await，需要把回调逻辑封装为Promise，再通过await等待执行结果，否则在同步执行流程里永远拿不到异步回调内的赋值内容。

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