无法在递归承诺函数中解决

Question

我认为我从根本上误解了 Promises 在 Node 中的工作原理。

我正在编写一个非常小的 Express 应用程序，我想返回包含某个关键字的所有 Spotify 播放列表的 JSON。我正在使用来自 npm 的spotify-web-api-node。

Spotify 的 API 限制每个请求 50 个播放列表。我有（我认为的）一个解决方案：只需增加偏移值，直到不再需要为止。

我的函数看起来像这样（playlists 在路由函数上方声明）：

app.get('/playlists', function(req, res) {
    addPlaylistsToList()
        .then((data) => res.json(data))
});

function addPlaylistsToList(offset) {
    if (!offset) { offset = 0; }
    var limit = 50;
    return new Promise((resolve, reject) => {
        spotifyApi.getUserPlaylists(config.USER, { limit: limit, offset: offset })
            .then(function(data) {
                playlists.push(data.body.items.filter((playlist) => playlist.name.includes(config.TITLE_CONDITION)));
                if (data.body.items.length == limit) {
                    addPlaylistsToList(offset + limit);
                } else {
                    let result = [...new Set(playlists)];
                    resolve(result);
                }
            })
    });
}

我不能在这里打电话给resolve吗？我的意思是，我知道我不能，因为我陷入了无限循环。是否有“最佳实践”：递归承诺？我是不是在滥用then？

Answer 1

使用递归 Promise 通常没问题，但是您在实现中缺少一些东西。对addPlaylistsToList() 的每次调用都会创建一个新的promise，但只有一次调用会创建一个实际解决的新promise：最后一个。所有其他已创建的 Promise 将无限等待。

你可以用更典型的递归风格重写函数，如下所示：

function addPlaylistsToList(offset = 0) { //default parameter value to 0
    var limit = 50;
    return spotifyApi.getUserPlaylists(config.USER, { limit: limit, offset: offset })
        .then(function(data) {
            playlists.push(data.body.items.filter((playlist) => playlist.name.includes(config.TITLE_CONDITION)));
            if (data.body.items.length == limit) {
                return addPlaylistsToList(offset + limit); //return a promise
            } else {
                return [...new Set(playlists)]; //return the final value
            }
        });
}

（未经测试）

这是可行的，因为向.then() 回调返回一个promise 会导致.then() “跟随”返回的promise - 它只会在那个 promise 解决时才解决，依此类推。您可以像这样递归地链接 Promise，最终最底层的 Promise 将使用您的结果数组解析，从而导致所有其他 Promise 也使用该数组解析。

另外，使用 async/await 可以写得更简洁：

async function addPlaylistsToList(offset = 0) { //default parameter value to 0
    var limit = 50;
    var data = await spotifyApi.getUserPlaylists(config.USER, { limit: limit, offset: offset });
    playlists.push(data.body.items.filter((playlist) => playlist.name.includes(config.TITLE_CONDITION)));
    if (data.body.items.length == limit) {
        return await addPlaylistsToList(offset + limit);
    } else {
        return [...new Set(playlists)];
    }
}