如何在Node.js中同步连接MSSQL Server？求代码优化方案

首先看你的代码，核心问题出在proc()方法的实现上，咱们一步步拆解和修正：

现存问题拆解 #

• 你在proc()里定义了一个匿名的async函数，但完全没有执行它，所以数据库连接、查询这些异步代码根本没跑起来
• proc()本身不是async函数，你直接返回this.result的时候，异步操作连开始都没开始，结果必然是undefined
• 变量名错误：查询里用了${this.searcher}，但构造函数里存的是this.srchType和this.srchContent，这里会直接导致查询报错
• 数据库连接没做收尾处理，查询后不关闭连接会造成连接泄漏，时间长了会耗尽数据库连接数

优化后的代码实现 #

先修正报表生成类，把proc()改成标准的async函数，直接在方法内处理异步逻辑：

const mssql = require('mssql');
const events = require('events');

class Reporter {
  constructor(searcher, logger) {
    this.lg = logger;
    if (searcher.type && searcher.content) {
      this.lg.lg("reporter created", 3);
      this.srchType = searcher.type;
      this.srchContent = searcher.content;
      this.err = null; // 初始化错误状态，方便后续判断
    } else {
      this.lg.lg("!MISSING SEARCH PARAMETERS", 0);
      this.err = "!MISSING SEARCH PARAMETERS";
    }
  }

  async proc() {
    // 先检查构造时的参数错误，提前终止
    if (this.err) {
      return Promise.reject(new Error(this.err));
    }

    let pool;
    try {
      // mssql.connect默认会创建连接池，比单次连接更稳定，推荐用这种方式
      pool = await mssql.connect('mssql://username:password@localhost/database');
      // 修正变量名，这里根据你的查询逻辑调整，比如用srchContent作为id查询条件
      this.result = await pool.query`select * from mytable where id = ${this.srchContent}`;
      return this.result; // 直接返回查询结果，方便调用方获取
    } catch (err) {
      this.lg.lg(`Database error: ${err.message}`, 0);
      throw err; // 抛出错误让调用方处理，避免错误被吞掉
    } finally {
      // 不管成功失败，都要关闭连接池，避免连接泄漏
      if (pool) {
        await pool.close();
      }
    }
  }
}

然后修改调用代码，因为proc()是async函数，必须用await或者.then()来获取结果（注意await必须放在async函数里）：

async function generateReport() {
  try {
    const report1 = new Reporter(searcher, logs);
    const result = await report1.proc();
    console.log('查询结果:', result);
    // 这里写你的报表生成逻辑
  } catch (err) {
    console.error('生成报表失败:', err);
  }
}

// 执行报表生成
generateReport();

关于同步实现MSSQL连接的疑问 #

首先明确：Node.js生态里的MSSQL驱动（包括tedious和基于它的mssql包）没有官方的同步API。

因为Node.js的IO操作（包括数据库连接、查询）都是异步非阻塞设计的，强制转同步会阻塞整个Node.js事件循环——哪怕你的微服务使用率低，这种做法也不推荐，会导致服务无法同时处理其他请求（比如同时进来两个请求，第二个会被第一个的同步操作卡住）。

如果实在对异步逻辑感到困惑，你可以尝试用deasync这类工具把异步函数转成同步，但这属于hack手段，长期来看还是建议花点时间理解async/await——它其实是Promise的语法糖，写起来比回调和裸Promise更像同步代码，非常适合你的低使用率微服务场景。

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