蓝鸟答应每一个都不返回最后的结果

Alvin 发表于 Dev

阿尔文

我正在使用promise.each循环遍历2个数据库查询bookshelfjs，但是角色没有给我roles[resource.get('name')] = role.get('name');this- >的结果，但是它给了我来自slect语句的对象：

var promise = new Promise(
        function resolver(resolve, reject) {   
            var roles = {};
            Promise.each(user.relations.relates.models, function(relation){
                return Resource.forge({resourceId: relation.get('resourceId')}).fetch().then(function(resource){
                    return Role.forge({roleId: relation.get('roleId')}).fetch().then(function(role){
                        roles[resource.get('name')] = role.get('name');
                        return roles;
                    }).catch(function(err){
                        reject({"status":"error", "data": err});
                    });
                }).catch(function(err){
                    reject({"status":"error", "data": err});
                });
            }).then(function(roles){
              resolve(roles);
            });
        }
    );

    return promise;

jfriend00

从Bluebird文档中获得Promise.each()：

解析为未经修改的原始数组，该方法用于产生副作用。

因此，这意味着您执行以下操作：

Promise.each(array).then(function(val) {
   // iteration finished here
   // val is the original array
});

那val将是您传入的原始数组，而不是您的roles对象。

因为您的roles对象在较高范围内，所以可以删除roles在.then()处理程序中声明的参数，而直接引用较高范围的roles变量。

您还应该避免在这里使用promise构造函数的反模式，而不是创建新的promise，而只是返回您已经拥有的那个。

您可以这样做：

var roles = {};
return Promise.each(user.relations.relates.models, function (relation) {
    return Resource.forge({resourceId: relation.get('resourceId')}).fetch().then(function (resource) {
        return Role.forge({roleId: relation.get('roleId')}).fetch().then(function (role) {
            roles[resource.get('name')] = role.get('name');
        });
    }).catch(function (err) {
        // repackage how the error is presented into our own object
        throw ({"status": "error", "data": err});
    });
}).then(function () {
    // make the resolved value of the final promise be the roles object
    return roles;
});

此返回的promise的已解析值将是您的roles对象。

变更摘要：

Promise.each()从而不是从创建新的承诺返回承诺（无需在此处创建新的承诺）。
让拒绝传播回去，无需手动拒绝更高级别的承诺。
在的.then()处理程序中Promise.each()，删除已命名的已声明参数，roles因为该参数与更高范围的roles对象冲突。
返回roles该.then()处理程序，使其成为您要返回的promise的解析值。
移除内层，.catch()因为外层.catch()也可以工作。
更改.catch()以引发重新包装的错误值。
删除内部，return roles;因为它是不需要的。该roles对象在更高的范围内始终可用，因此无需使其成为这些内部承诺的已解决值（实际上，这样做可能会造成混淆）。