Question

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

任何想法我做错了什么？

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

Answer 1

你有一个无限循环。

setTimeout期望第一个参数是回调函数 - 您实际上是在那里调用waitForRawData函数。它立即再次调用自己，它会立即再次调用自己，这......你明白了。

这样做：

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

当你把它作为一个函数传递时，超时可以在你告诉它时调用它 - 在你的情况下，一秒钟之后。在没有包装功能的情况下执行它会立即调用相同的代码。

setTimeout导致堆栈溢出

1 个答案: