如何通过TypeScript方法正确返回SQL Server连接变量？

咱们先拆解下你当前的代码实现，再聊聊怎么优化得更合理：

原代码的不合理之处 #

• 频繁关闭连接完全浪费连接池优势：每次调用getRequest()都先执行sql.close()，这会直接关闭所有已建立的连接，等于把mssql内置的连接池机制完全废掉了。连接池本来就是为了复用连接、减少频繁创建/销毁连接的性能开销，你这操作反而会让每次请求都重新建立连接，拖慢响应速度，还可能导致并发场景下的连接冲突。
• 错误处理有遗漏：sql.close()本身是异步操作（返回Promise），但你直接调用没处理它的错误，万一关闭过程中出问题，会产生未捕获的Promise错误。
• 回调写法不够直观：用传统的Promise回调嵌套，代码可读性不如async/await，后续维护起来也麻烦。

优化后的实现方案 #

核心思路是利用mssql内置的连接池管理连接，不要手动频繁关闭/创建连接，同时用async/await简化代码逻辑：

const sql = require('mssql');

/* SQL配置，新增连接池参数 */
const config = {
  user: '***',
  password: '***',
  server: '***',
  database: '***',
  port: '***',
  multipleStatements: true,
  timezone: '+00:00',
  // 连接池配置，可根据业务需求调整
  pool: {
    max: 10,    // 连接池最大连接数
    min: 0,     // 最小空闲连接数
    idleTimeoutMillis: 30000 // 空闲连接超时时间（毫秒）
  }
};

// 可选：应用启动时提前初始化连接池，避免首次请求延迟
async function initSqlConnection() {
  try {
    await sql.connect(config);
    console.log('SQL Server连接池初始化完成');
  } catch (err) {
    console.error('连接池初始化失败:', err);
    process.exit(1); // 初始化失败直接退出，避免后续业务报错
  }
}

// 获取SQL Request的方法
async function getRequest() {
  try {
    // 复用连接池中的连接，不存在则自动创建
    const connection = await sql.connect(config);
    return connection.request();
  } catch (err) {
    throw new Error(`获取SQL Request失败: ${err.message}`);
  }
}

// 应用退出时关闭连接池（可选）
process.on('SIGINT', async () => {
  await sql.close();
  console.log('SQL连接池已关闭');
  process.exit(0);
});

module.exports = { initSqlConnection, getRequest };

优化点说明 #

• 充分利用连接池：mssql会自动管理连接的复用、空闲回收，不需要手动关闭连接，大大提升并发场景下的性能。
• 更清晰的错误处理：用try/catch捕获所有可能的错误，并且把错误信息包装后抛出，方便上层调用者统一处理。
• async/await简化逻辑：替代传统Promise回调，代码结构更直观，可读性和维护性更强。
• 增加初始化和退出钩子：提前初始化连接池减少首次请求延迟，应用退出时关闭连接池避免资源泄漏。

额外最佳实践 #

• 执行完SQL语句后不需要手动释放连接，request执行完成后会自动把连接放回连接池。
• 根据业务并发量调整连接池的max参数，不要设置过大（避免数据库连接数耗尽），也不要过小（无法支撑高并发）。
• 不要在业务代码中随意调用sql.close()，除非你确定要完全销毁连接池。

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