2

私は次のコードを持っています。スタック オーバーフロー例外が発生します。

私は何を間違えたのですか？

var myApi = {    

      rawData: null,

      initData: function() {
               // ajax call to get data and populate myApi.rawData, max 10 seconds
      },

      waitForRawData: function(callback) {
              if(myApi.rawData === null || myApi.rawData.length ===0) {
                 window.setTimeout(myApi.waitForRawData(callback),1000); // complain this line stack overflow
              }else{
                     callback();
              }              
      },

      updateHtmlWithNewData: function() {
              // base on myApi.rawData update html element
      },
      workflow: function() {  // this function call is invoke from page
              myApi.initData();
              myApi.waitForRawData(myApi.updateHtmlWithNewData);
      }
}
4

1 に答える 1

5

無限ループがあります。

setTimeout最初のパラメーターがコールバック関数であることが期待されます。実際には、その場でwaitForRawData関数を呼び出しています。これはすぐに再び自分自身を呼び出します、それはすぐに再び自分自身を呼び出します、それは...あなたは考えを理解します。

これを行う：

window.setTimeout(function() { myApi.waitForRawData(callback) },1000);

それを関数として渡すと、タイムアウトは、あなたがそれを指示するときはいつでもそれを呼び出すことができます-あなたの場合、1秒後。ラッピング関数なしでそれを行うことは、今同じコードを呼び出しています。

于 2013-01-04T05:54:35.123 に答える