我对NodeJS上的Promises还是陌生的,正在尝试在AWS Lambda中编写一些后端代码来查询包含设备的DynamoDB表。
代码如下:
var geohash = require('ngeohash');
var AWS = require('aws-sdk');
var documentClient = new AWS.DynamoDB.DocumentClient();
exports.handler = (event, context, callback) => {
if(event.params.querystring.minLat && event.params.querystring.minLon
&& event.params.querystring.maxLat && event.params.querystring.maxLon ){
var bboxes = geohash.bboxes(event.params.querystring.minLat, event.params.querystring.minLon, event.params.querystring.maxLat, event.params.querystring.maxLon, 6);
var params = {
"TableName": "Devices",
"IndexName": "GeoIndex",
"KeyConditionExpression": "GeohashPrefix = :bbox",
"ExpressionAttributeValues": {
":bbox": ""
},
};
var devices = [];
var promises = [];
for (var num in bboxes) {
params.ExpressionAttributeValues[":bbox"] = bboxes[num];
promises = promises.concat(documentClient.query(params, function(err, data) {
if (err) {
console.error("Unable to query. Error:", JSON.stringify(err, null, 2));
} else {
console.log("Query succeeded.");
devices = devices.concat(data.Items);
console.log(devices);
}
}).promise())
}
Promise.all(promises).then(callback(null, devices));
}
}
这里发生的是Promise.all()实际上并没有等待所有的承诺,而是立即返回。
我在做什么错了?
答案 0 :(得分:4)
您需要将函数传递给then()
,而不是函数的结果。
此:
Promise.all(promises).then(callback(null, devices));
应该更像这样:
Promise.all(promises).then(() => callback(null, devices));
在第一种情况下,在定义了诺言后立即调用callback
。