那么函数没有定义

时间:2016-02-10 07:02:16

标签: javascript angularjs

我有一个函数来获取我需要存储在localstorage上的当前公司代码,并在其余的API调用中使用它。所以,直到没有检索到这个公司代码的值,我不应该进行API调用。 这就是我获取公司代码的方式

currentDivision = function() {
    var q = $q.defer();
    var division = window.localStorage.getItem("CurrentDivision");
    if(division){
        return q.resolve(division);
    }
    else{
        var url = this.fooApi + "current/Me?$select=CurrentDivision";
        $http.get(url)
            .success(function(data) {
                division = data.d.results[0].CurrentDivision;
                window.localStorage.setItem("CurrentDivision", division);
                return q.resolve(division);
            })
    }
    return q.promise;
}

这就是我在确保成功检索当前分区后尝试调用其余API调用的方法:

$scope.openModal = function() {
        $scope.modal.show().then(function() {
            currentDivision().then(function(){
            fooServices.getData().then(function() {
                $scope.closeModal();
            }, function(reason) {
                // handle the error
                $scope.showAlert(reason);
                $scope.closeModal();
            })
            });
        })
}

但是当我尝试这个时,我收到此错误消息: TypeError:无法读取未定义的属性“然后”

2 个答案:

答案 0 :(得分:1)

您应该始终返回q.promise。删除return中的if (division) { return q.resolve(division); }

var q = $q.defer();
// ...
if (division) {
    q.resolve(division);
} else { 
  //...
}
return q.promise;

即:

then

通过这种方式,您将返回已解决的承诺,立即调用<style name="TextInputLayout.Error" parent="@style/TextAppearance.Design.Error"> <item name="android:textColor">@color/textinput_error_color</item> </style> <style name="Theme.AppName.TextInputLayout" parent="@style/Widget.Design.TextInputLayout"> <item name="errorTextAppearance">@style/TextInputLayout.Error</item> </style>

答案 1 :(得分:1)

试试这个:

if (division) {
    return $q.when(division);
}

并改进您的代码:

currentDivision = function () {
    if (division) {
        return $q.when(division);
    }

    var url = this.fooApi + "current/Me?$select=CurrentDivision";
    return $http.get(url).then(function (data) {
         division = data.d.results[0].CurrentDivision;
         window.localStorage.setItem("CurrentDivision", division);
         return division;
    });
}