Question

大家好我在AngularJS中使用REST在确认按钮中调用此函数。保存是可以的，问题发生在成功或错误消息中。如果成功，它应该在执行循环后只显示一条成功消息，或者在任何错误上多次显示错误消息。如果发生错误，我使用了一个标志isError来获取，但是我做的并不重要，＆＃34; if（isError == false）{＆＃34;总是在循环结束之前执行。

 $scope.confirm = function () {
        var isError = false;
        var i=0;
        $scope.displayedmodalcollection.forEach(function(item) {
            i++;
            delete item.added;
            result = DataBase.update({
                id: item._id,
                endpoint: item._type
            }, item._source).$promise.then(function(response) {

            }, function(err) {
                isError = true;
                if (err.status === 500) {
                    ngToast.create({
                        className: 'warning',
                        content: String.format('Any message.', item._id)
                    });
                }
                else if (err.status === 404) {
                    ngToast.create({
                        className: 'danger',
                        content: '404 - other message'
                    });
                } else {
                    ngToast.create({
                        className: 'danger',
                        content: 'Error'
                    });
                }
            });
        });
        if (isError == false){

            ngToast.create({
                className: 'success',
                content: 'Success'
            });
        }
     };

有没有人对此提出建议？感谢

Answer 1

您可以使用$q.all()根据所有解决的更新

显示成功吐司

而不是：

$scope.displayedmodalcollection.forEach(function(item) {

执行类似

的操作

// map array of update promises
var updatePromises = $scope.displayedmodalcollection.map(function(item){
   // return the promise
   return DataBase.update({...}).$promise.then(function(response) {
               // success code
            }, function(err) {
               // error code
            });    
});    

$q.all(updatePromises).then(function(){
      // all promises were resolved
      ngToast.create({
                className: 'success',
                content: 'Success'
      });
});

记得在控制器中注入$ q

Answer 2

我不确定您希望如何处理成功的更新和错误，但这应该会让您走上正轨：

$scope.confirm = function() {
    var successes = 0;
    var errors = 0;
    $scope.displayedmodalcollection.forEach(function(item) {
        delete item.added;
        result = DataBase.update({
            id: item._id,
            endpoint: item._type
        }, item._source).$promise.then(function(response) {
            successes++;
            checkStatus(successes, errors);
        }, function(err) {
            errors++;
            checkStatus(successes, errors);
            if (err.status === 500) {
                ngToast.create({
                    className: 'warning',
                    content: String.format('Any message.', item._id)
                });
            } else if (err.status === 404) {
                ngToast.create({
                    className: 'danger',
                    content: '404 - other message'
                });
            } else {
                ngToast.create({
                    className: 'danger',
                    content: 'Error'
                });
            }
        });
    });
};

function checkStatus(successes, errors) {
    var isCompleted = (successes + errors) == $scope.displayedmodalcollection.length;
    var notAllFailed = successes > 0;
    if (isCompleted && notAllFailed) {
        handleSuccessToast();
    }
}

function handleSuccessToast() {
    ngToast.create({
        className: 'success',
        content: 'Success'
    });
}

Answer 3

问题是isError == false分支中的成功逻辑正在所有数据库调用完成之前执行。因此，它可能在错误实际发生之前运行，因此标志为false并且遵循分支。

这是因为DB逻辑正在返回promise。 Promise不会阻止执行其余代码 - 它们只是推迟包含在＆＃39;然后＆＃39;中的逻辑。回调直到DB调用实际完成。因此，如果您的收藏中有5个项目，则可以在实际完成任何项目之前启动所有5个项目的数据库调用。关键是您需要让您的成功逻辑等待，直到您知道所有数据库承诺已经解决并且所有呼叫都已完成。有关承诺的更多信息，请参阅this Promise guide。

以下是在检查是否发生错误之前，如何编写逻辑以等待所有承诺解决的示例：

var doneCount = 0;
var isError = false;
something.foreach(item) {
    somedblogic(item).$promise.then(function (response) {
        if (++doneCount === something.length && !isError) {
            //-- success logic here - runs one time after all db calls complete
        }
    },function (err) {
        doneCount++;
        isError = true;
        //-- rest of error logic here - runs once for each error returned
    });
}

在使用promise.then AngularJS之后只显示一条消息

3 个答案: