これは悪いコールバックの練習ですか？

Question

私はJavaScriptには比較的新しいので、パラメータとしてコールバックを取るヘルパーオブジェクトにメソッドを書くことを繰り返しています。

var utilities = { 
    getTweets: function (user, maxTweets, callBack) { 
      var obj = $(this); 
      $.getJSON('http://api.twitter.com/1/statuses/user_timeline.json?callback=?&screen_name=' + user + "&count=" + maxTweets, function (data) { 
       callBack(data); 
      }); 
    }; 

Iその後、そのようにそれを呼び出す：私はしたsetIntervalを使用して上記のコードを呼び出しています考える

utilities.getTweets("TESTUSER", 4, function (tweets) { 
       ..... 
      }); 

が、これは時間をかけて漏れる可能性がある/これを書くための良い方法はありますか？

あなたはこのような何か使用することができ

Answer 1

余分な閉鎖を作成する必要がないことを除いて、あなたがやっていることはほとんどうまくあります。以下のように書かれたクロージャを渡す：

function (data) { 
     callBack(data); 
} 

は、パラメータリストで直接callBackを渡すのとちょうど同じです。

あなたはjQueryの1.5以降で動作して保証できる場合は、その後、より良い方法は、単にJQXHRオブジェクトを返すgetTweets()持つことで、その後、あなたはクライアントコードで「繰延」メソッドを使用することができます

var utilities = { 
    getTweets: function (user, maxTweets) { 
     var uri = 'http://api.twitter.com/1/statuses/user_timeline.json?callback=?'; 
     var data = { 
      screen_name: user, 
      count: maxTweets 
     }; 
     return $.getJSON(uri, data); 
    }); 
}; 

を

し、クライアントコードで：

utilities.getTweets(user, maxTweets).done(/* your callback here */); 

このように、あなたは完全に実装からのコールバックを切り離すことができます。実際にutilitiesの実装に触れずに、複数のコールバックとエラーハンドラを登録できます。

NB：dataのマップを使用すると、コードをパラメータ注入から保護することもできます。

Answer 2

：

var utilities = { 
    options: { 
     user: 'value', 
     maxTweets: '4' 
    } 
    getTweets: function() { 
     // access a value 
     this.options.user; 
    } 
} 

それともそれは、この記事の読み取りを持って、それはあなたが必要な正確に何であるべきオプションなどとの適切なプラグインを作成するのが最善のようになります。

を

http://jquery-howto.blogspot.com/2009/01/how-to-set-default-settings-in-your.html

Answer 3

あなたがするたびにその関数を作成するためのメモリオーバーヘッドの恐れている場合、そのような何か：

utilities.getTweets("TESTUSER", 4, utilities.handleTweets); 

をそして、あなたはコールバックでやるようutilities.handleTweetsにあなたが。