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

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

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

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

解決済

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

ivezuki
ivezuki

総合スコア33

AngularJS

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

2回答

0リアクション

2クリップ

3120閲覧

投稿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); }); }

}]);
`

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

以下のような質問にはリアクションをつけましょう

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

リアクションが多い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

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

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

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

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

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

適切な質問に修正を依頼しましょう。

まだ回答がついていません

会員登録して回答してみよう

アカウントをお持ちの方は

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

ただいまの回答率
86.12%

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

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

質問する

関連した質問

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

AngularJS

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