Javascript：递归 function 与 promise 并在达到计数时解析

Question

我一直在尝试构建一个已定义为 promise 的递归 function。

我无法在下面的代码上应用递归模式，即使loopFor初始化为20 ，我也只循环了一次我错过了什么？

要求： receivingMessages必须是一个promise。

let globalMessageArray = [];
let count = 0;
let loopFor = 20;

function receivingMessages(params, loopFor, globalMessageArray) {
    return new Promise((resolve, reject) => {
        const command = new ReceiveMessageCommand(params);
        client.send(command).then(
            (data) => {
                if (data && data.Messages && data.Messages.length) {
                    data.Messages.forEach(msg => {
                        globalMessageArray.push(msg);
                    });
                };
                return resolve(globalMessageArray);
            },
            (error) => {
                return reject(error);
            }).then(
            (globalMessageArray) => {
                count = count + 1;
                console.log("Loop Count: " + count); // always returns 1
                if (loopFor === 1) {
                    return resolve(globalMessageArray);
                } else {
                    return resolve(receivingMessages(params, loopFor - 1, globalMessageArray));
                };
            });

    });
};

Answer 1

在第一个 then 回调client.send(cmd).then(data => …你return resolve(globalMessageArray) 。这有效地缩短了你的循环，因为 promise 只能resolve一次。稍后调用resolve没有效果。

client.send(cmd).then((data) => {
    …
    return globalMessageArray;
}, …

Remove first call to resolve应该可以解决您的问题。

---

你在评论中说：

使用 async/await 意味着重写整个程序

不，你对 async/await 的理解是错误的。 任何异步 function 都会自动返回 function 的 promise，这符合您的要求。 Async/await 只是 promise 之上的语法糖。

这意味着您可以安全地重写 ONLY receivingMessages function 而无需修改调用它的其他地方。

尽管 vanilla promise 没有任何问题，但重写为 async/await 将使您的代码更加清晰。

async function receivingMessages(params, loopFor, globalMessageArray) {
    const command = new ReceiveMessageCommand(params);
    const data = await client.send(command);
    
    if (data && data.Messages && data.Messages.length) {
        data.Messages.forEach(msg => {
            globalMessageArray.push(msg);
        });
    }
             
    if (loopFor === 1) {
        return globalMessageArray;
    } else {
        return receivingMessages(params, loopFor - 1, globalMessageArray)
    };
};

Answer 2

您的代码的问题是，在 client.send promise 解析之后的 then 回调中的 resolve 调用返回调用 receivingMessages 而不是 receivingMessages promise 本身的结果。 这导致递归循环只执行一次。

要解决此问题，您可以更改 resolve 调用以直接返回调用 receivingMessages 的结果：

return receivingMessages(params, loopFor - 1, globalMessageArray);

这将导致 receivingMessages function 以递归方式被调用，直到 loopFor 达到 1。

您可能还想考虑向 function 添加基本情况以确保它终止，例如添加检查 loopFor 是否小于或等于 0 并在这种情况下返回 globalMessageArray。