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

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

新規登録して質問してみよう
ただいま回答率
85.48%
AngularJS

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

Q&A

解決済

2回答

3275閲覧

Angular.jsのコントローラーの処理の分割に関して

ivezuki

総合スコア33

AngularJS

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

0グッド

2クリップ

投稿2014/07/24 16:35

下のコードはAngular.jsで通信してデータを取得している処理でしっかり動いているのですが、非常にファットコントローラーになってしまっています。

`
otorecoApp.controller('NewSongListCtrl', ['$scope', '$http', '$sce', function($scope, $http, $sce) {
$scope.newSongs = [];
$http({method: 'GET', url: 'top/getNewSongs'}).
success(function(data, status, headers, config) {
for(var i = 0; i < data.length; i++) {
$scope.newSongs.push(
{
songName: data[i].Song.name,
artistName: data[i].Artist.name,
songNameForSearch: optimize(data[i].Song.name),
artistNameForSearch: optimize(data[i].Artist.name),
artistId: data[i].Artist.id,
naturalLow: returnPitchName(data[i].Song.natural_low),
naturalHigh: returnPitchName(data[i].Song.natural_high),
falsettoLow: returnPitchName(data[i].Song.falsetto_low),
falsettoHigh: returnPitchName(data[i].Song.falsetto_high)
}
);
}
}).
error(function(data, status, headers, config) {
console.log(status);
});

$scope.search = function(index) { $scope.searchResults = []; $scope.googleUrl = []; artistName = $scope.newSongs[index].artistNameForSearch; songName = $scope.newSongs[index].songNameForSearch; $http({method: 'GET', url: 'http://gdata.youtube.com/feeds/api/videos?vq=' + artistName + ' ' + songName + '&max-results=10&alt=json'}). success(function(data, status, headers, config) { $scope.googleUrl.push({url: 'www.google.co.jp/#q=' + artistName + ' ' + songName + '歌詞&qscrl=1'}); for(var i = 0; i < 4; i++) { var rs = data.feed.entry[i].link[0].href.match(/http://www.youtube.com/watch?v=(.*?)&feature=youtube_gdata/); $scope.searchResults.push( { artistName: artistName, songName: songName, movieTitle: data.feed.entry[i].title.$t, movieUrl: $sce.trustAsResourceUrl('http://www.youtube.com/embed/' + rs[1] + '?rel=1&fs=1') } ); } }).error(function(data, status, headers, config) { console.log(status); }); }

}]);
`

そこで通信してる処理だけでも別の関数に切り分けるなりしたいのですが、どうやってやればいいのかわかりません。
良い分け方を知っていましたらご教授お願い致します。

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

factoryでサービスを作るのが一番楽だと思います。
newSongsあたりをモデルとして切り出すのが良いのではないかなと。
あとは$httpの通信部分もサービスとしてまとめるのも良いと思います。

投稿2014/07/25 23:42

naga3

総合スコア1293

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

ベストアンサー

serviceを作って呼び出すのが良いと思います。

http://qiita.com/opengl-8080/items/2fe0a20c314b1c824cc5

投稿2014/07/25 07:49

h_sakurai

総合スコア228

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問