搜索

首页  >  问答  >  正文

javascript递归函数问题求大神看看

大神晚上好,请帮我看看我的函数为什么不能执行呢?
情况说明:由于p中的table是通过ajax加载过来的,函数的目的是判断有没有这table,如果有则让其背景变红,没有的话,就1秒后再执行以下这个函数,但是现在当table已经加载显示后,find()函数并没有让table变红(报错:Uncaught RangeError: Maximum call stack size exceeded)
先谢谢大神们了!

漂亮男人漂亮男人2712 天前690

全部回复(2)我来回复

  • 我想大声告诉你

    我想大声告诉你2017-06-28 09:26:22

    因为你用p.getElementsByTagName('table')[0]这个取到的是一个DOM对象,由于DOM对象没有.length属性,所以target.length其实是未定义的。而undefined > 0的值一直是false,所以你会无限次地调用else分支,所以也就会添加无数次的find(p)绑定。所以浏览器提示find调用次数超出最大限制。

    正确的做法是让targetp.getElementsByTagName("table"),这才是一个数组,才有.length的值。

    Update

    代码:
    方案1:(判断取到所有table的数组长度,并取第一个操作)

    function find(p) {
        var target = p.getElementsByTagName("table");
        if (target.length > 0) {
            target[0].style.background = 'red';
        } else {
            setTimeout(function() {
                find(p);
            }, 1000)
        }
    };

    方案2:(直接判断table,并直接操作取到的table)

    function find(p) {
        var target = p.getElementsByTagName("table")[0];
        if (target) {
            target.style.background = 'red';
        } else {
            setTimeout(function() {
                find(p);
            }, 1000)
        }
    };

    回复
    0
  • 世界只因有你

    世界只因有你2017-06-28 09:26:22

    target.length target 是 table, table.length 是什么?

    参考一下

    function find(p) {
      var interval = setInterval(function () {
        var target = p.getElementsByTagName("table")[0]
        if (target) {
          clearInterval(interval)
          target.style.background = 'red'
        }
      }, 1000)
    }

    回复
    0
  • 取消回复