Node.js偶现Cannot set headers after they are sent to the client错误求助

咱们先拆解下你这段Node.js代码里的几个关键问题，这就是导致偶现错误的根源：

问题1：重复发送HTTP响应 #

你在成功和错误分支里，都先调用了res.status().send()，紧接着又调用return res.json()。这两个方法都会向客户端发送完整的HTTP响应，第一次发送后响应头已经被设置并发送给客户端，第二次再尝试发送响应时，就会触发那个经典的"Cannot set headers after they are sent"错误。

问题2：错误处理逻辑无效 #

fs.readFile是基于回调的传统异步API，不是Promise，所以你在后面链式调用的.catch()根本不会被触发！当文件读取出错时，错误会被传到回调的err参数里，而不是进入你写的.catch块，这会导致错误无法被正确捕获，甚至可能引发未处理的异常，间接导致响应异常。

问题3：数据库连接永远不会关闭 #

你把dbConn.close()放在了return res.json()之后，这行代码永远不会执行——因为return会直接终止函数执行，数据库连接会一直占用，造成资源泄漏。

修正后的代码方案 #

我推荐用Promise风格的fs.promises.readFile来重构代码，错误处理更清晰，也能避免重复发送响应的问题：

方案1：Promise链式调用 #

const fs = require('fs');

exports.GetDepartmentList = function (req, res) {
  fs.promises.readFile('./api/tempFiles/department.json', 'utf8')
    .then(response => {
      // 只发送一次响应，send方法已经支持JSON格式输出
      res.status(200).send({ 
        success: true, 
        data: JSON.parse(response) 
      });
    })
    .catch(err => {
      res.status(500).send({ 
        success: false, 
        message: err.message 
      });
    })
    .finally(() => {
      // 不管成功/失败，最后统一关闭数据库连接
      dbConn.close();
    });
};

方案2：Async/Await（更易读） #

如果你习惯同步风格的异步代码，用async/await会更清晰：

const fs = require('fs');

exports.GetDepartmentList = async function (req, res) {
  try {
    const response = await fs.promises.readFile('./api/tempFiles/department.json', 'utf8');
    res.status(200).send({ 
      success: true, 
      data: JSON.parse(response) 
    });
  } catch (err) {
    res.status(500).send({ 
      success: false, 
      message: err.message 
    });
  } finally {
    dbConn.close();
  }
};

关键修改说明 #

1. 改用Promise风格的文件读取：fs.promises.readFile替代传统回调式API，让异步逻辑和错误处理更可靠。
2. 移除重复的响应发送：每个分支只调用一次res.status().send()，彻底避免重复发送响应的问题。
3. 用finally统一关闭连接：finally块会在请求处理完成（无论成功或失败）后执行，确保数据库连接被正常关闭，不会泄漏资源。

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