我有以下代码。它导致堆栈溢出异常。

知道我做错了什么吗?

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

最佳答案

您有一个无限循环。
setTimeout期望第一个参数是回调函数-您实际上是在那儿然后调用waitForRawData函数。哪个立即再次调用自身,立即又再次调用自身,您就知道了。

做这个:

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

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

10-02 15:30