在for循环中中断promise块 [英] Break out of promise block in a for loop

问题描述

我有以下代码 -

  this.storeNameValidate = function（stores）{
 var deferred = $ q.defer（）; 
 
 console.log（商店）; 
 for（商店中的storeIndex）{
 this.nameValidate（stores [storeIndex] .storeName，3,10）
 .then（function（）{
 console.log（有效商店名称）; 
}，function（）{
 console.log（商店名称无效）; 
 deferred.reject（商店名称无效）; 
 }）
} 
 return deferred.promise; 
} 
  

如果调用nameValidate错误块，我需要断开for循环。我怎样才能做到这一点 ？

我可以使用标记

 这样做。 storeNameValidate = function（stores）{
 var deferred = $ q.defer（）; 
 var flag = false; 
 console.log（商店）; 
 for（商店中的storeIndex）{
 this.nameValidate（stores [storeIndex] .storeName，3,10）
 .then（function（）{
 console.log（有效的商店名称）; 
 var flag = true; 
}，function（）{
 var flag = false; 
 console.log（商店名称无效）; 
 deferred.reject（无效商店名称）; 
}）
 if（！flag）break; 
} 
 return deferred.promise; 
} 
  

他们有更好的方式吗？

解决方案

你应该使用 $ q.all ，包括一系列承诺和返回合并承诺。如果其中一个内部承诺失败，这个承诺将拒绝。

  this.storeNameValidate = function（stores）{
 
 var promises = Object.keys（stores）.map（function（storeIndex）{
 return this.nameValidate（stores [storeIndex] .storeName，3,10）.then（function（）{
 console.log（有效商店名称）; 
}，function（）{
返回$ q.reject（无效商店名称）; 
}）; 
}，这个）; 
 
返回$ q.all（承诺）; 
 
}; 
  

另外，不要滥用 $ q.defer 在你的情况下你不需要它。这种多余的使用称为延迟反模式。 / p>

此外，如果您要拒绝的错误消息始终是无效的商店名称（不是特定于商店），并且您实际上不需要在验证后执行其他操作商店，你也可以省略错误和成功回调。然后代码将变得更清晰：

  this.storeNameValidate = function（stores）{
 
 var promises = Object.keys（stores）.map（function（storeIndex）{
 return this.nameValidate（stores [storeIndex] .storeName，3,10）; 
}，this）; 
 
返回$ q.all（承诺）; 
 
}; 
  

I have the following code -

this.storeNameValidate = function(stores) {
    var deferred = $q.defer();

    console.log(stores);
    for (storeIndex in stores) {
        this.nameValidate(stores[storeIndex].storeName, 3, 10)
            .then(function() {
                console.log("valid store name");
            }, function() {
                console.log("invalid store name");
                deferred.reject("invalid store name");
            })
    }
    return deferred.promise;
}

I need to break out off the for loop if the nameValidate error block is called. How can i do this ?

I can do it something like this using flagging

this.storeNameValidate = function(stores) {
    var deferred = $q.defer();
    var flag = false;
    console.log(stores);
    for (storeIndex in stores) {
        this.nameValidate(stores[storeIndex].storeName, 3, 10)
            .then(function() {
                console.log("valid store name");
                var flag = true;
            }, function() {
                var flag = false;
                console.log("invalid store name");
                deferred.reject("invalid store name");
            })
            if (!flag)break;
    }
    return deferred.promise;
}

Is their some better way ?

解决方案

You should use $q.all with array of promises and return combined promise. This single promise will reject if one of the inner promises fails.

this.storeNameValidate = function (stores) {

    var promises = Object.keys(stores).map(function(storeIndex) {
        return this.nameValidate(stores[storeIndex].storeName, 3, 10).then(function () {
            console.log("valid store name");
        }, function () {
            return $q.reject("invalid store name");
        });
    }, this);

    return $q.all(promises);

};

Also, don't abuse $q.defer you don't need it in your case. Such a redundant usage of it is known as deferred anti-pattern.

Also, if error message you are going to reject with is always "invalid store name" (not store specific) and you don't really need to preform additional actions on validated stores, you can omit error and success callbacks all together too. Then the code will become even cleaner:

this.storeNameValidate = function (stores) {

    var promises = Object.keys(stores).map(function(storeIndex) {
        return this.nameValidate(stores[storeIndex].storeName, 3, 10);
    }, this);

    return $q.all(promises);

};

这篇关于在for循环中中断promise块的文章就介绍到这了，希望我们推荐的答案对大家有所帮助，也希望大家多多支持IT屋！