JS 越跑越卡一会儿就卡死了

This topic created in 3859 days ago, the information mentioned may be changed or developed.

function  goto()
    {
        var _this=$('.urls span.url').eq(0);
        var b = new Base64();
        $('#cai_web').attr('src',"/cai_content/"+b.encode (_this.html())+'-'+_this.attr('data-gid')+'-'+'{{ $id }}');
        $('#cai_web').load(function()
        {
            _this.appendTo('.urls .item .ok');
            if (go==true){
                goto();
            }
        });
    }

做采集的时候需要用到的 JS 但是发现这个段 JS 越跑越卡一会儿就卡死了
哪个地方有问题？该如何解决呢？

28 replies • 2015-11-13 22:36:59 +08:00

penjianfeng

Nov 13, 2015

你的变量 go 呢？一直 true ？

m939594960

Nov 13, 2015

@penjianfeng
就是跑一条扔一条然后继续跑让他一直跑下去
go 这个变量当点击停止按钮之后就会变成 false

CodingPuppy

Nov 13, 2015

哪里跑一条仍一条了？都在内存里

m939594960

Nov 13, 2015

@CodingPuppy
比如一共 100 条 url 放到一个 div 里
取一个 url 然后 iframe 传 encode 的 url 到后台
然后等待 iframe 加载完成吧这个 url 传到另一个容器里
继续执行相同的动作

现在问题就是跑到第 19~20 条浏览器就崩溃了

jasontse

Nov 13, 2015 via iPad

@m939594960
上一个 goto 没结束就套下一个 goto ，哪里跑一条扔一条了。

lucky2touch

Nov 13, 2015

手机上回了两次没成功，上面说得很清楚了，没有释放......

m939594960

Nov 13, 2015

@jasontse goto 后面没有代码了应该算结束了吧要 retrun ''去结束么？
@lucky2touch 怎么释放呢？谢谢！

learnshare

Nov 13, 2015

递归地在跑吧，没有释放什么东西

kokdemo

Nov 13, 2015

……这么简单的效果用 settimeout 会不会好一点？

m939594960

Nov 13, 2015

@learnshare 这种情况不能用递归么。应该去用 while ？？
@kokdemo 得网站加载完毕后在加载另一个 iframe 啊

m939594960

Nov 13, 2015

@kokdemo 得一个 iframe 加载完毕后在加载另一个 iframe 啊

kokdemo

Nov 13, 2015

@m939594960 可以循环的检测 iframe 的状态
检测加载完成，就触发下一个 iframe

m939594960

Nov 13, 2015

@kokdemo 嗯我尝试一下应该是可以得但是总想找个完美的方法

sunjourney

Nov 13, 2015

这是学过递归还是没学过呢，出口在哪里？用循环加 flag 控制，另外你的 Indentation 是什么教派的？

otakustay

Nov 13, 2015

楼主的各位，这货似乎不是个递归，有个异步+callback 在里面……

m939594960

Nov 13, 2015

@sunjourney 缩进是在复制的时候出了一些小问题现在找到几种方法了 while 或者 settimeout 但是我还是不懂为什么我这么写不行。。求教谢谢

bdbai

Nov 13, 2015 via iPhone

每次回调都会保留作用域开调试器检查一下

LancerComet

Nov 13, 2015

go == true ? setTimeout(goto, 1) : void(0);

楼主试试：
```
var i = 0;
function a (param) {
param = param || 0;
console.log("a" + param);
if (i < 3) {
a(1);
i++;
}
console.log("a() executing finished.");
}
```
和
```
var i = 0;
function a (param) {
param = param || 0;
console.log("a" + param);
if (i < 3) {
setTimeout(a.bind(0, i), 1);
i++;
}
console.log("a() executing finished.");
}
```

LancerComet

Nov 13, 2015

@m939594960
楼主这么写的行为： goto 还没有退出就在 goto 内再次执行 goto
加了 setTimeout 后的行为：向任务队列登记一个新的计时器任务后退出 goto.

HunterPan

Nov 13, 2015

你这个用法会不会造成栈溢出啊有点递归（ js 不大懂）

HunterPan

Nov 13, 2015

@otakustay 要看的 callback 怎么处理的才能决定是否递归吧

aivier

Nov 13, 2015

$(...).load() 不是用来加载数据的么，例如

$("username").load("/getUserName.php")

dphdjy

Nov 13, 2015

iframe 有父对象~如果可以交互在子 document 建立玩在反馈~~感觉省好多事~~

m939594960

Nov 13, 2015

@kokdemo
@sunjourney
@ 、 bdbai
@LancerComet
@HunterPan
@dphdjy
找到问题了。其实并不是递归的问题。是个非常蛋疼的问题。。
我每次循环的时候都给 iframe 加上了一个监视 load 的事件
然后 goto 就无限的运行了。。现在在前面加个$('#cai_web').unbind('load'); 然后再进行监视就 ihaole

aivier

Nov 13, 2015

孤陋寡闻了...从未在循环里设定过事件绑定，用手机写的代码测试出来了刚想回复发现正好你也发现问题了...

m939594960

Nov 13, 2015

@LancerComet 嗯谢谢你以后不用递归了用这种 settimeout 应该效果更好
@aivier 我查了一下
http://www.w3school.com.cn/jquery/ajax_load.asp
http://www.w3school.com.cn/jquery/event_load.asp
应该是一个方法两个用途
@dphdjy 嗯这样也不错

otakustay

Nov 13, 2015

@HunterPan 那个是 jQuery 的 load 嘛，是个异步的，不会记在同一个 call stack 上面的，除非楼主坑我们$不是 jQuery

aivier

Nov 13, 2015

看了下官方文档，自动判断传入参数的，我感觉还是现在这种方式更好一些，如果准备一个一个来的话，这种 callback 方式就相当于多线程的了

JS 越跑越卡 一会儿就卡死了

JS 越跑越卡一会儿就卡死了