質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.37%

  • AngularJS

    587questions

    AngularJSはオープンソースのJavaScriptフレームワークです。ブラウザ上で動作するウェブアプリケーションの開発にMVCアーキテクチャを取り入れることを目的としています。

promise.thenのsuccessCallback内でのメソッドの使用方法について

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 1,280

akabee

AngularJS総合1位

前提・実現したいこと

promise.thenのsuccessCallback内でメソッド呼び出しをしているのですが、下記の通りのエラーとなります。これはなぜでしょうか?
this.a ではなく、宣言、呼び出しとも$scope.aとすると問題ありませんでしたが、これはどのような仕様によるものでしょうか。

個人的にあまり$scopeは利用したくないのですが、$scopeを利用せずにsuccessCallback内でメソッドの呼び出しをするにはどのようにすべきでしょうか。

発生している問題・エラーメッセージ

TypeError:this.a is not a function

該当のソースコード

angular.module(app_Name)

.controller('LoginController', function($http, LoginService) {

  this.login = function() {

    LoginService.login()
      .then(
        function(){
          this.a           //ここがエラーとなる

        },

        function(errorReason){

        }

      });
    };

    this.a = function(){
      //処理
    }

});

試したこと

this.a ではなく、宣言、呼び出しとも$scope.aとすると問題ありませんでした。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+1

 javascriptのthis

javascriptのthisはどこが呼び出しているかよりも、どいつが呼び出しているか
によって値が左右されるため、呼び出し方によっては期待した動作をしないことがあります。
※単純にコントローラー(どこが)に作成したメソッドが呼ばれるわけではありません

以下のサンプルですとwindowオブジェクトがthisになっています。
ですので単純にthisを使おうとすると呼び出し側の事を考えなければなら無い為
「var self = this;」等としてthisの値を別の変数に置き換えるとわかりやすいと思います。

 補足

call,apply,bind,アロー関数等を使うとその限りではありません

 動作サンプル

リンク

 コード

<html>
 <head>
  <script src="https://cdnjs.cloudflare.com/ajax/libs/angular.js/1.4.3/angular.min.js"></script>  
  <script>

    // グローバルな呼び出し
    window.a = function(){
      alert('windowオブジェクトのa()が呼び出されました')
    }

    // angularjsの定義
    var app = angular.module('myApp', []);

    // コントローラー
    app.controller('LoginController', function($scope, LoginService) {

      var self = this;

      $scope.login = function(){
        LoginService.login().then(function(){

          if(this.a){
            this.a();
          }else{
            alert('this.a()が呼び出されませんでした');
          }

          if(self.a){
            self.a();
          }else{
            alert('self.a()が呼び出されませんでした');
          }

          if($scope.a){
            $scope.a();
          }else{
            alert('$scope.a()が呼び出されませんでした');
          }
        });
      }

      // thisの呼び出し
      this.a = function(){
        alert('LoginController(this)のa()が呼び出されました');
      }

      // selfの呼び出し
      self.a = function(){
        alert('LoginController(self)のa()が呼び出されました');
      }

      // $scopeの呼び出し
      $scope.a = function(){
        alert('LoginController($scope)のa()が呼び出されました');
      }

    });

    // サービス
    app.factory('LoginService', function($q, $timeout){
      var self = {};

      // 非同期ログイン処理
      self.login = function(){
        var d = $q.defer();
        $timeout(function() {
          alert('ログイン完了!');
          d.resolve();
        }, 1000);
        return d.promise;
      }

      // サービスのthis呼び出し
      this.a = function(){
        alert('LoginService(this)のa()が呼び出されました');
      }

      // サービスのself呼び出し
      self.a = function(){
        alert('LoginService(self)のa()が呼び出されました');
      }

      return self;
    });
  </script>
  </head>
  <body>
    <div ng-app="myApp">
      <div ng-controller="LoginController">
        <button ng-click="login()">ログイン</button>
      </div>
    </div>
  </body>
</html>

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/11/28 17:51

    今回は無名関数なのでthisの中身がwindow関数のようです
    http://d.hatena.ne.jp/snaka72/20080831/1220159391

    キャンセル

  • 2016/11/29 09:44

    回答ありがとうございます。
    その後同様の内容には自分でもたどり着いていたのですが、難しいですね・・。精進します。

    キャンセル

同じタグがついた質問を見る

  • AngularJS

    587questions

    AngularJSはオープンソースのJavaScriptフレームワークです。ブラウザ上で動作するウェブアプリケーションの開発にMVCアーキテクチャを取り入れることを目的としています。