AnguarJS： 'this'ポインタまたはreturn文を使用

Question

サービスを宣言する正しい方法はどれですか？

.service('myService', function(myFactory) { 
    myFactory.LoadData("...",function(newData){ 
     this.data= newData; 
    }); 
} 

または

.service('myService', function() { 
    var data= {}; 
    myFactory.LoadData("...",function(newData){ 
     data= newData; 
     return data ; 
    }); 

} 

私は、http私はそれを使用しますが、一度だけ、その後、私はいつでもデータにアクセスするたびに呼び出しを行う必要はありません。

私は最初の例を試しましたが、コントローラで使用しようとすると「未定義」という値が得られます。

編集：ここ

OKが私のコードです：

サービス：

.service('ClassService', function(ClassFactory) {     

       ClassFactory.query(function(rawClasses){ 
        this.classes= []; 
        rawClasses.forEach(function(classe) { 
         var classeRow={}; 
         classeRow.grade=classe.grade; 

         classe.branch.forEach(function(branch) { 
          classeRow._id=branch._id; 
          classeRow.branch= branch.name; 
          classeRow.fees= branch.fees; 

          branch.sub_class.forEach(function(subClass) { 
           classeRow.label=subClass.label; 
           classeRow.name= _getGradeName_(classeRow.grade)+(classeRow.branch || '')+ (subClass.label || ''); 
           console.info('ClasseService hihihi!'); 
           this.classes.push(_byValue_(classeRow)); 
            console.log(this.classes);  

          }, this); 
         }, this);     
        }, this); 
       });  
     }) 

とコントローラ：

.controller('StudentController', function(StudentFactory, ClassService, $scope, $stateParams) { 
    //... 
    $scope.classes= ClassService.classes; 
    //but here console.log($scope.classes) gives 'undefined' 
    //... 
}); 

Answer 1

あなたのサービスでクエリが実行されています非同期的に、あなたは約束を使用する必要がありますo値が利用可能になるのを待ちます。

ClassFactory.query()がコールバックを使用する代わりに約束を返すようにすると、すべてがはるかに簡単になります。完了を処理するには、ClassServiceにthePromise.then()を使用しますが、それはコントローラに公開する約束を返します。コントローラでは、別の.then()を使用してデータがいつ利用可能かを知ることができます。

だから、あなたが約束を返すようにClassFactory.query()を更新しました仮定：

.service('ClassService', function(ClassFactory) {     

    var promise =ClassFactory.query(); 
    this.dataLoaded = promise.then(... the code to convert rawClasses to this.classes goes here...); 
}) 

と：

.controller('StudentController', function(StudentFactory, ClassService, $scope, $stateParams) { 
    //... 
    ClassService.dataLoaded.then(function() { 
     $scope.classes= ClassService.classes; 
    }); 
});