Node.js中如何通过递归正确返回最终结果？

核心问题拆解 #

• 第一个版本代码中，当data.total >= totalUser时，result未被赋值，且递归调用的返回值没有传递到当前Promise的result中，直接resolve(result)自然返回undefined——哪怕await了递归调用，没把递归结果存下来，当前函数的result还是空值。
• 第二个版本代码中，只有result为true时才执行resolve，但当data.total >= totalUser时result始终未定义，导致Promise一直处于pending状态，服务端无法及时返回响应，最终触发超时，返回503错误。

修正方案 #

1. 传递递归结果 #

递归调用时，需要把下层递归的返回值赋值给当前的result，确保最终能传递到最外层调用：

groupProduct: (data, totalUser, number) => {
    return new Promise(async (resolve, reject) => {
        let result;
        if (data.total < totalUser) {
            result = true;
        } else {
            let nextNumber = number;
            if (number < totalUser) {
                nextNumber += 1;
            } else if (number === totalUser) {
                nextNumber -= totalUser;
            }
            // 把递归的返回值赋值给当前result
            if (nextNumber >= 0 && nextNumber < totalUser) {
                result = await prodHelper.groupProduct(data, totalUser, nextNumber);
            }
            // 添加终止条件，避免无限递归
            if (number > totalUser * 2) {
                result = false;
            }
        }
        // 无论result是什么都resolve，避免Promise pending
        resolve(result);
    })
}

2. 避免无限递归 #

原代码的递归逻辑没有明确终止条件，当data.total一直大于等于totalUser时，会无限递归下去，最终导致内存溢出或请求超时。必须添加终止条件，比如限制递归次数，或者在某个条件下返回false。

3. 优化Promise写法 #

没必要嵌套new Promise(async...)，如果函数本身是async的，可以直接返回值，Node.js会自动包装成Promise：

groupProduct: async (data, totalUser, number) => {
    if (data.total < totalUser) {
        return true;
    }
    let nextNumber = number;
    if (number < totalUser) {
        nextNumber += 1;
    } else if (number === totalUser) {
        nextNumber -= totalUser;
    }
    // 终止条件：递归次数超过阈值则返回false
    if (number > totalUser * 2) {
        return false;
    }
    if (nextNumber >= 0 && nextNumber < totalUser) {
        return await prodHelper.groupProduct(data, totalUser, nextNumber);
    }
    // 默认返回，避免无返回值
    return false;
}

4. 修复product函数 #

原product函数没有返回结果或resolve，导致调用它的地方也会一直pending，需要补充：

product: async (data, totalUser) => {
    let number = 1;
    let res = await prodHelper.groupProduct(data, totalUser, number);
    console.log('RESULT', res);
    return res;
}

关键注意点 #

• 递归必须有明确的终止条件，否则会无限调用，耗尽资源导致服务崩溃或超时。
• 递归调用的返回值必须向上传递，否则外层调用拿不到最终结果。
• Promise必须确保被resolve或reject，不能一直处于pending状态，否则会导致请求超时。

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