2

「app.js」という名前のjsファイルがあり、JavaScriptを簡単に最小化できるように、ほとんどのJavaScriptを保存しています。

私は、それぞれの固有の関数を次のようにラップしていることに気づきました。

$(document).ready(function () { ... });

このようにファイルにラップされた関数は約10個あります。個々の関数ではなく、app.jsファイル全体を$（document）.ready内にラップする方がよいでしょうか？

ドキュメントの乱用を避けるために読んだかもしれないように感じますが、$（this）について考えているだけかもしれません。

私がしていることにパフォーマンスの問題はありますか？

4

1 に答える 1

0

この種のアプローチではパフォーマンスの問題はないはずですが、私の意見では、それは醜くて読みにくいように見えます。

私が提案するのは、関数を1つだけ使用してready関数を実行し、init関数を外部に格納することreadyです。たとえば、次のようになります。

$(document).ready(function () { 

//check some stuff once document is ready
check_if_user_is_loggedin();

//check if user got new message
setInterval(function(){
   check_user_inbox();
}, 5000);

});


//now here store your functions like:
function check_if_user_is_loggedin(){
  if(loggedIn) return true; else return false;
}

check_user_inbox(){
  //ajax call etc...
  return messages;
}
于 2012-11-21T20:50:32.017 に答える