AngularJS'无法读取未定义的属性'then''

Question

我有这个问题，当我点击登录按钮时，chrome 控制台会记录这个：

angular.min.js:117 TypeError: Cannot read property 'then' of undefined 在 m.$scope.logIn (loginModuleController.js:11)

服务：

angular.module('loginModule')
.factory('loginService', function($http){
  return{
    login: function(username, password){
      var session;
      $http.post('../server/php/auth/auth.php', {
        username: username,
        password: password
      })
      .then(function(res){
        session = res;
      });
      return session;
    },
    isLogged: function(){
      return $http.get('../angCMS/server/php/auth.php?is_logged=true');
    },
    logOut: function(){
      return $http.get('../angCMS/server/php/auth.php?logout=true');
    }
  };
});

控制器：

angular.module('loginModule')
.controller('LoginCtrl', ['$scope', 'loginService', function($scope, loginService){

  $scope.auth = false;

  $scope.username;
  $scope.password;
  $scope.logIn = function(){
    loginService.login($scope.username, $scope.password).then(function(response){

    }, function(res){

    });
  };

  $scope.isLogged = function(){
    loginService.isLogged()
    .then(function(response){
      if(response){
        $scope.auth = true;
      }
    });
  };

  $scope.logOut = function(){
    loginService.logOut()
    .then(function(response){
      if(response){
        $scope.auth = false;
      }
    });
  };

}]);

这是html模板：

<div class="container" ng-if="auth==false">
  <div class="col-md-4 col-md-offset-4">
    <div class="row">
      <br/><h2 align="center">Login</h2>
    </div>
    <div class="well">
        <form class="form-horizontal">
        <fieldset>
          <div class="form-group">
                <input type="text" class="form-control" placeholder="Username" ng-model="username" required>
          </div>
          <div class="form-group">
                <input type="password"  class="form-control" placeholder="Password" ng-model="password" required>
          </div>
          <div class="form-group">
                <button class="btn btn-md btn-primary btn-block" type="submit" ng-click="logIn()">Sign in</button>
          </div>
        </fieldset>
      </div>
    </form>
  </div>
</div>

PHP登录方式：

public function login($user, $pass){

        $user = htmlspecialchars(trim($user));
        $pass = md5(htmlspecialchars(trim($pass)));

        $res = $this->DB->prepare("SELECT * FROM `admin` WHERE username = :user");
        if(!$res->execute(Array(":user"=>$user)))
            die(mysql_error());

        $row = $res->fetch(PDO::FETCH_ASSOC);

        if(!$row['password'] == $pass)
            die("Errore: password errata!");

        $_SESSION['logged'] = $row;
        array_push($session, $_SESSION['logged'], true);

        return $session;
    }

Answer 1

这更像是一个滥用承诺的问题。

您可能想先看看 Promise 是如何工作的：

• https://docs.angularjs.org/api/ng/service/$q
• https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Global_Objects/Promise

来自您的服务代码：

login: function(username, password){
  var session;
  $http.post('../server/php/auth/auth.php', {
    username: username,
    password: password
  })
  .then(function(res){
    session = res;
  });
  return session;
}

当调用login(username, password) 时，返回时会话仍然是undefined。

这是因为$http.post() 是一个异步函数，.then() 子句不会立即执行。

正如Snixtor's answer所指出的， 你应该“返回$http.post()的返回值”：

login: function(username, password){
  return $http.post('../server/php/auth/auth.php', {
    username: username,
    password: password
  });
}

然后参考你的控制器的logIn方法：

$scope.logIn = function(){
  loginService.login($scope.username, $scope.password).then(function(response){
    // response === 'session' 
  }, function(res){

  });
};

loginService.login().then() 中的 response 参数正是您之前实现中预期的 session 变量的值。

Answer 2

您正在返回一个名为“session”的未分配变量。因此，您在登录方法中所做的是返回未定义。 Undefined 没有调用 then 的方法。

login: function(username, password){
      var session;
      $http.post('../server/php/auth/auth.php', {
        username: username,
        password: password
      })
      .then(function(res){
        session = res;
      });
      return session;
    },

我的猜测是您实际上想要返回 $http.post 方法，该方法又返回一个承诺。这样就可以在控制器中使用会话了？

Answer 3

您的服务函数login 未能返回调用$http.post 创建的承诺。相反，你需要这个：

login: function(username, password){
  return $http.post('../server/php/auth/auth.php', {
    username: username,
    password: password
  });
}

还要注意，我已经从函数中删除了 session 变量。控制器中的 then 函数需要处理响应。

您的isLogged 和logOut 函数看起来已经不错了。只需重复该模式即可。