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);

当您将它作为函数传递时,超时可以在您告诉它时调用它 - 在您的情况下,一秒钟后。在没有包装函数的情况下执行它现在正在调用相同的代码。

于 2013-01-04T05:54:35.123 回答