在foreach循环完成执行后,Response.json应该执行
var todoarr = (req.body.data) ? req.body.data : undefined
todoarr.forEach(function(element) {
if(element.done == true) {
TodoService.removeTodo(element, function(success) {
});
}
});
res.json("success");
答案 0 :(得分:3)
您可以尝试使用async.js http://caolan.github.io/async/。
each
方法http://caolan.github.io/async/docs.html#each
或者您可以尝试使用Promise.all
。
例如:
let promiseArr = [];
todoarr.forEach(function(element) {
if(element.done == true) {
promiseArr.push(somePromiseMethod(element));
}
});
//now execute promise all
Promise.all(promiseArr)
.then((result) => res.send("success"))
.catch((err) => res.send(err));
此处有更多信息。 https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Global_Objects/Promise/all
一些承诺示例:
function somePromiseMethod(element) {
return new Promise((resolve,reject) => {
TodoService.removeTodo(element, function(success) {
resolve();
});
});
}
希望这有帮助。
答案 1 :(得分:2)
您可以在forEach
的回调函数中发送响应。
修改您的函数,使其仅在上一次迭代时调用res.json()
。
示例:
var todoarr = (req.body.data) ? req.body.data : undefined
todoarr.forEach(function(element,index) {
if(element.done == true) {
TodoService.removeTodo(element, function(success) {
});
}
if(index==todoarr.length-1){
res.json("success");
}
});
但是,它可能不符合编码标准,但可以肯定地解决了这个问题。
答案 2 :(得分:1)
您无法在单个请求中发送多个响应,您唯一可以做的就是使用结果数组的单个响应:
es with async:
const async = require('async')
// FIX ME: this isn't correctly handled!
const todoarr = (req.body.data) ? req.body.data : undefined
let results = []
async.each(todoarr, function(element, callback) {
console.log('Processing todo ' + element)
if(element.done == true) {
TodoService.removeTodo(element, function(err, success) {
if(err){
callback(err)
} else {
results.push(success)
callback(null, success)
}
})
}
}, function(err) {
if(err) {
console.log('A element failed to process', err)
res.status(500).json(err)
} else {
console.log('All elements have been processed successfully')
// array with the results of each removeTodo job
res.status(200).json(results)
}
})