Node.js Promise封装child_process执行命令报错问题

这个问题和Promise封装逻辑本身无关，是两边错误处理逻辑不一致导致的误判：

• Node的child_process.exec判定命令执行成功的默认规则很死板：只要子进程退出码不是0，就会返回error对象，不管stdout里有没有拿到正确的执行结果。你调用的dgate.exe是Conquest DICOM服务的组件，执行参数查询类命令后，退出时返回的状态码固定是4294967295（也就是十六进制0xFFFFFFFF，对应有符号整数-1），不符合默认的0成功码，所以每次调用都会生成error对象。
• 你写的直接调用版本，回调里根本没判断error参数，不管有没有报错都直接打印stdout，才让你以为执行没出错。只要在这个回调里加一行打印error的代码，就能看到和Promise版本一模一样的报错。
• 你的Promise封装版逻辑是只要检测到error就直接reject，把你之前一直没处理的错误抛了出来，才会让你觉得是Promise封装导致的问题。

针对dgate程序的退出码特性，调整exec的错误判断逻辑即可，不用改Promise结构：

• 对dgate返回的特殊退出码4294967295做兼容，只要此时stdout有有效内容，就当成执行成功返回结果
• 其余异常场景（比如找不到exe文件、参数错误等）保留错误抛出逻辑，避免真出问题时无感知
• 可选打印stderr的输出，方便后续排查其他问题

修复后的完整代码如下：

const { exec } = require("child_process");
const path = require("path"); // 补全原代码缺失的path模块引入
const dgate =
  path.join(__dirname, "/api/dicomapi/dgate.exe") +
  " -p5678 -hCONQUESTSRV1 -q127.0.0.1";

function os_func() {
  this.execCommand = function (cmd) {
    return new Promise((resolve, reject) => {
      exec(cmd, (error, stdout, stderr) => {
        // 打印标准错误输出方便调试
        if (stderr) console.log("命令stderr输出：", stderr);
        
        if (error) {
          // 兼容dgate的特殊正常退出码，有有效输出就返回结果
          if (error.code === 4294967295 && stdout) {
            resolve(stdout.trim());
            return;
          }
          // 其余真实错误正常抛出
          reject({ error, stdout, stderr });
          return;
        }
        resolve(stdout.trim());
      });
    });
  };
}

var os = new os_func();

os.execCommand(`${dgate} "--get_param:MAGDevice0"`)
  .then((returnvalue) => {
    console.log(returnvalue);
  })
  .catch((error) => console.log("执行报错：", error));

你可以在之前的直接调用版本里加一行error打印，就能验证错误是一直存在的，和Promise无关：

exec(`${dgate} "--get_param:MAGDevice0" `, (error, stdout, stderr) => {
  console.log("error对象实际存在：", error); // 会输出和Promise版本完全一致的错误
  console.log("pac", stdout);
});

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