Question

我正在尝试使用promises从DynamoDB表中获取所有记录。问题是DynamoDB不会在一次调用中返回所有项目，我必须进行多次调用。如果LastEvaluatedKey不为null意味着我需要使用该密钥进行另一次调用以获取剩余记录。在我的代码中，我正在检查并仅在LastEvaluatedKey为null之后进行解析。但是console.log("done")没有被执行。

这是我的代码：

function query(params) {
    return new Promise(function(resolve, reject) {
        docClient.query(params, function(err, data) {
            if (err) {
                reject(err)
            } else {
                resolve(data);
            }
        });    
    })
}

function getAllRecords(params, combinedData) {
   return new Promise(function(resolve, reject) {
        query(params)
       .then(function(data) {
           if(!combinedData) {
               combinedData = [];
           }
           combinedData.push(data.Items); 
           if(data.LastEvaluatedKey) {
                params.ExclusiveStartKey = data.LastEvaluatedKey;
                getAllRecords(params, combinedData)
            }
            else {
                resolve(combinedData);
            }
       })    
   })


}

getAllRecords(params)
.then(function() {
  console.log('done')
})
.catch(function(error) {
    console.log(error);
})

这可能是对我的承诺如何运作的误解。如果有人能让我知道如何使这项工作。那太好了。

Answer 1

你已经成为explicit promise construction antipattern的牺牲品，当你不需要时，你会手动构建承诺。

通常，您需要使用Promise构造函数的唯一时间是将非Promise异步代码转换为Promise异步代码。您已经在query()函数中完成了此操作，因此您无需在Promise函数中使用getAllRecords()构造函数。

你应该这样做：

function getAllRecords(params) {
    return query(params).then(function (data) {
        var items = [data.Items];

        if(data.LastEvaluatedKey) {
            params.ExclusiveStartKey = data.LastEvaluatedKey;

            return getAllRecords(params).then(function (theRest) {
                return items.concat(theRest);
            });
        }

        return items;
    });
}

蓝鸟承诺循环

1 个答案: