Question

function getNamesById(nameIds) {
    var defer = $q.defer();

    var result = [];
    nameIds.forEach(function (nameId) {
        account.getNameById(nameId).then(function (name) {
            result[nameId] = name;
        });
    });
    defer.resolve(result);
    return defer.promise;
}

我有上面的代码巫婆显然没有按预期工作。我必须迭代一个id的数组并构造另一个具有id作为键的数组，以及从异步函数中获取的值

Answer 1

您应该使用$q.all

返回另一个承诺

function getNamesById(nameIds) {

    var result = [];
    nameIds.forEach(function (nameId) {
        result.push(account.getNameById(nameId));
    });
    return $q.all(result);
}

然后将其链接到其他地方

getNamesById(someNameIds)
     .then(...);

更清楚的方法是使用.map而不是声明一个数组并使用forEach来推送它：

function getNamesById(nameIds) {
    return $q.all(nameIds.map(function (nameId) {
        return account.getNameById(nameId)
    }));
}

如果使用ES6（并且您的方法肯定会忽略除第一个参数之外的所有参数），可以进一步简化：

let getNamesById = nameIds => Promise.all(nameIds.map(account.getNameById));

因此，根据评论，我猜测你正试图以nameId作为关键字以及异步调用的结果作为值来获取关联数组。您需要再执行一个步骤 - 您需要返回您then中可以使用的值

function getNamesById(nameIds) {
    return $q.all(nameIds.map(function (nameId) {
        return account.getNameById(nameId)
                    .then(function(name){
                        return {id:nameId, name:name};
                     })
    }));
}

使用此方法时，您可以轻松地reduce结果到关联数组

getNamesById(["a","b","c"])
    .then(function(results){
        var values = results.reduce(function(p,n){
            p[n.id] = n.name;
            return p;
        },{});
        // values is your associative array with keys a,b & c
     });

在ES6中写出来的结果要简单得多 - 为了完整性：

function getNamesById(nameIds) {
    return $q.all(nameIds.map(
               nameId => account.getNameById(nameId)
                    .then(name => ({id:nameId, name:name}))
    );
}

和

getNamesById(["a","b","c"])
    .then(results => {
        var values = results.reduce((p,n) => {
            p[n.id] = n.name;
            return p;
        },{});
        // values is your associative array with keys a,b & c
     });

在forEach中返回承诺

1 个答案: