Express.js路由执行子进程时stdout为undefined如何解决

cp.exec是Node.js的异步非阻塞API，传入的回调会在子进程执行完成后才进入事件队列触发，不会阻塞后续代码运行。当前代码调用cp.exec后立刻执行了res.json，此时回调还没执行，msg始终是初始声明的undefined，自然返回的字段值不符合预期。

• 打日志确认执行顺序：分别在cp.exec调用前、exec回调函数内、res.json调用前打印标记日志，就能看到执行顺序是「调用exec → 执行res.json → 触发exec回调」，赋值操作发生在响应返回之后，根本不会影响返回结果。
• 核对API的调用范式：Node.js原生回调风格的API（最后一个参数是回调函数的这类）默认都是异步执行，不存在「等回调跑完再往下走」的逻辑，不能在回调函数外部的同步代码里，直接拿回调内部赋值的变量。
• 检查变量赋值时机：msg声明后未赋值的初始值就是undefined，只有子进程执行完、回调被调用时才会被赋值为stdout，响应发送时机早于赋值时机，拿到的值必然是初始的undefined。

方案1：把响应逻辑放到exec回调内 #

不需要改动现有异步写法，直接等子进程执行完拿到结果再返回，记得补全错误处理：

const express = require("express")
const asyncHandler = require("express-async-handler")
const cp = require('child_process')
const router =  express.Router()

router.get("/status", asyncHandler(async (req, res) => {
    cp.exec('ls', (err, stdout, stderr) => {
        if (err) {
            res.status(500).json({ message: stderr || err.message })
            return
        }
        res.json({ message: stdout.trim() })
    })
}))

module.exports = router

方案2：适配async/await写法，包装为Promise #

既然路由已经用了async函数，可以用Node.js内置的promisify把回调风格的exec转成Promise版本，用await等待执行完成，代码逻辑更线性：

const express = require("express")
const asyncHandler = require("express-async-handler")
const cp = require('child_process')
const { promisify } = require('util')
const exec = promisify(cp.exec)
const router = express.Router()

router.get("/status", asyncHandler(async (req, res) => {
    try {
        const { stdout } = await exec('ls')
        res.json({ message: stdout.trim() })
    } catch (err) {
        res.status(500).json({ message: err.stderr || err.message })
    }
}))

module.exports = router

• 不要用模板字符串包裹未确认是否有值的变量：${msg}会把undefined类型转成字符串"undefined"，会掩盖「变量根本没赋值」的问题，增加排查成本。
• 所有子进程调用必须加错误处理：命令不存在、权限不足、执行报错都会触发err参数，不处理的话会导致请求挂起超时，甚至因为未捕获异常拖垮服务。
• 如果执行的命令输出体积超过默认缓冲区大小（默认1MB），不要用exec，换成spawn以流的方式接收输出，避免触发缓冲区溢出错误。

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