27

解决了

关于这个主题,网络上有很多相互矛盾的信息。感谢@John,我设法弄清楚闭包(如下所示)不是内存泄漏的原因,而且 - 即使在 IE8 中 - 它们并不像人们声称的那样普遍。事实上,我的代码中只发生了 1 次泄漏,事实证明这并不难修复。

从现在开始,我对这个问题的回答将是:
AFAIK,IE8 唯一一次泄漏,是在全局对象上附加事件/处理程序时。( window.onload, window.onbeforeunload,...)。要解决这个问题,请参阅下面的答案。


巨大的更新:

我现在完全迷失了......经过一段时间的新旧文章和tuts挖掘后,我至少留下了一个巨大的矛盾。虽然其中一位JavaScript 大师(Douglas Crockford)说

由于 IE 无法完成它的工作并回收循环,我们就必须这样做。如果我们明确地打破循环,那么 IE 将能够回收内存。根据微软的说法,闭包是内存泄漏的原因。这当然是非常错误的,但它会导致微软在如何应对微软的错误方面给程序员提供了非常糟糕的建议。事实证明,打破 DOM 端的循环很容易。在 JScript 端破坏它们几乎是不可能的。

正如@freakish 指出的,我下面的代码片段与jQuery 的内部工作原理相似,我对我的解决方案不会导致内存泄漏感到非常安全。同时我发现了这个 MSDN 页面,我对这个部分Circular References with Closures特别感兴趣。下图几乎是我的代码如何工作的示意图,不是吗:

带闭包的循环引用

唯一的区别是我有不将我的事件侦听器附加到元素本身的常识。
尽管如此,Douggie还是非常明确的:闭包不是 IE 中内存泄漏的来源。这种矛盾让我不知道谁是对的。

我还发现泄漏问题在 IE9 中也没有完全解决(找不到链接 ATM)。

最后一件事:我还了解到 IE 在 JScript 引擎之外管理 DOM,当我<select>根据 ajax 请求更改元素的子元素时,这让我感到很麻烦:

function changeSeason(e)
{
    var xhr,sendVal,targetID;
    e = e || window.event;//(IE...
    targetID = this.id.replace(/commonSourceFragment/,'commonTargetFragment');//fooHomeSelect -> barHomeSelect
    sendVal = this.options[this.selectedIndex].innerHTML.trim().substring(0,1);
    xhr = prepareAjax(false,(function(t)
    {
        return function()
        {
            reusableCallback.apply(this,[t]);
        }
    })(document.getElementById(targetID)),'/index/ajax');
    xhr({data:{newSelect:sendVal}});
}

function reusableCallback(elem)
{
    if (this.readyState === 4 && this.status === 200)
    {
        var data = JSON.parse(this.responseText);
        elem.innerHTML = '<option>' + data.theArray.join('</option><option>') + '</option>';
    }
}

如果 IE 确实管理 DOM,就好像 JScript 引擎不存在一样,那么使用此代码不释放选项元素的可能性有多大?
我特意添加了这个片段作为示例,因为在这种情况下,我将作为闭包范围一部分的变量作为参数传递给全局函数。我找不到关于这种做法的任何文档,但根据 Miscrosoft 提供的文档,它应该会破坏任何可能发生的循环引用,不是吗?



警告:冗长的问题......(对不起

我已经编写了几个相当大的 JavaScript 来在我的 Web 应用程序中进行 Ajax 调用。为了避免大量的回调和事件,我充分利用了事件委托和闭包。现在我编写了一个函数,让我想知道可能的内存泄漏。虽然我知道 IE > 8 比它的前辈更好地处理闭包,但公司政策仍然支持 IE 8。

下面我提供了一个我正在讨论的例子,在这里你可以找到一个类似的例子,虽然它不使用 ajax,而是一个 setTimeout,结果几乎相同。(您当然可以跳过下面的代码,直接到问题本身)

我想到的代码是这样的:

function prepareAjax(callback,method,url)
{
    method = method || 'POST';
    callback = callback || success;//a default CB, just logs/alerts the response
    url = url || getUrl();//makes default url /currentController/ajax
    var xhr = createXHRObject();//try{}catch etc...
    xhr.open(method,url,true);
    xhr.setRequestMethod('X-Requested-with','XMLHttpRequest');
    xhr.setRequestHeader('Content-type','application/x-www-form-urlencoded');
    xhr.setRequestHeader('Accept','*/*');
    xhr.onreadystatechange = function()
    {
        callback.apply(xhr);
    }
    return function(data)
    {
        //do some checks on data before sending: data.hasOwnProperty('user') etc...
        xhr.send(data);
    }
}

除了onreadystatechange回调之外,所有的东西都非常简单。我在直接绑定处理程序时注意到 IE 的一些问题:xhr.onreadystatechange = callback;,因此是匿名函数。不知道为什么,但我发现这是使其工作的最简单方法。

正如我所说,我使用了很多事件委托,因此您可以想象访问触发 ajax 调用的实际元素/事件可能会很有用。所以我有一些看起来像这样的事件处理程序:

function handleClick(e)
{
    var target,parent,data,i;
    e = e || window.event;
    target = e.target || e.srcElement;
    if (target.tagName.toLowerCase() !== 'input' && target.className !== 'delegateMe')
    {
        return true;
    }
    parent = target;
    while(parent.tagName.toLowerCase() !== 'tr')
    {
        parent = parent.parentNode;
    }
    data = {};
    for(i=0;i<parent.cells;i++)
    {
        data[parent.cells[i].className] = parent.cells[i].innerHTML;
    }
    //data looks something like {name:'Bar',firstName:'Foo',title:'Mr.'}
    i = prepareAjax((function(t)
    {
        return function()
        {
            if (this.readyState === 4 && this.status === 200)
            {
                //check responseText and, if ok:
                t.setAttribute('disabled','disabled');
            }
        }
    })(target));
    i(data);
}

如您所见,onreadystatechange回调是函数的返回值,它target在调用回调时提供对元素的引用。多亏了事件委托,当我决定从 DOM 中删除它时(我有时会这样做),我不再需要担心可能绑定到该元素的事件。
然而,在我看来,回调函数的调用对象可能对 IE 的 JScript 引擎及其垃圾收集器来说太过分了:

Event ==> handler ==> prepareAjax 是一个非常正常的调用序列,但是回调参数:

[匿名。func(参数 t = 目标)返回 anon。F(可以访问 t ,而 t 又引用回目标)]
   ===> 传递给匿名回调函数,使用 xhr 对象的 .apply 方法调用,进而将私有变量传递给 prepareAjax 函数

我已经在 FF 和 chrome 中测试了这个“结构”。它在那里工作得很好,但是这种在关闭时关闭时关闭的调用堆栈,每次传递对 DOM 元素的引用都会在 IE 中成为问题(尤其是 IE9 之前的版本)?


不,我不会使用 jQuery 或其他库。我喜欢纯 JS,并且想尽可能多地了解这种被严重低估的语言。代码片段不是实际的复制粘贴示例,而是为 IMO 提供了我如何在整个脚本中使用委托、闭包和回调的良好表示。因此,如果某些语法不完全正确,请随时纠正它,但这当然不是这个问题的内容。

4

2 回答 2

23

我曾经与微软的 JavaScript 前项目经理一起工作,在一个非浏览器的 EcmaScript (err.. JScr ... JavaScript) 项目上。我们就关闭进行了一些冗长的讨论。最后,关键是它们更难 GC,并非不可能。我必须阅读 DC 关于 MS 如何“错误”导致内存泄漏的讨论——因为在 IE 的旧实现中,闭包肯定是有问题的,因为它们很难用 MS 实现进行垃圾收集。我觉得奇怪的是,雅虎的人会试图告诉 MS 架构师,他们的代码的一个已知问题在其他地方。尽管我很欣赏他的工作,但我看不出他有什么依据。

请记住,您上面引用的文章指的是 IE6,因为在撰写本文时 IE7 仍在大力开发中。

顺便说一句——谢天谢地,IE6 已经死了(不要让我挖掘葬礼图片)。虽然,不要忘记它的遗产......我还没有看到有人提出可信的论点,说它在发布的第一天就不是世界上最好的浏览器——问题是他们赢得了浏览器大战. 因此,这相当于他们历史上最大的错误之一——他们随后解雇了这支球队,这支球队停滞了近 5 年。多年来,IE 团队只有 4 或 5 个人进行错误修复,这造成了巨大的人才流失并大大落后于曲线。当他们重新雇用团队并意识到他们所处的位置时,他们已经落后了数年,因为处理一个没人真正理解的单一代码库增加了麻烦。这是我作为公司内部人员的看法,

还要记住,IE 从来没有针对闭包进行过优化,因为没有 ProtoypeJS(哎呀,没有 Rails),而 jQuery 在 Resig 的头脑中几乎没有一丝曙光。

在撰写本文时,他们还针对具有 256 兆 RAM 的机器,这些机器也没有报废。

在让我读完你的整本书之后,我认为给你上这堂历史课是公平的。

最后,我的意思是你引用的材料已经过时了。是的,避免 IE6 中的闭包,因为它们会导致内存泄漏——但 IE6 中没有什么?

最后,这是一个 MS 已经解决并将继续解决的问题。您将进行某种程度的关闭,即使在当时也是如此。

我知道他们在 IE8 周围的这个领域做了大量的工作(因为我无法提及的项目使用了非当时的标准 JavaScript 引擎),并且这项工作一直持续到 IE9/10。StatCounter (http://gs.statcounter.com/) 表明 IE7 的市场份额已从一年前的 6% 下降到 1.5%,并且在开发“新”站点时,IE7 变得越来越不重要。您还可以为引入 JavaScript 支持的 NetScape 2.0 进行开发,但这只会稍微不那么愚蠢。

真的......不要为了不再存在的引擎而过度优化。

于 2012-06-26T13:52:04.140 回答
4

是的,在使用该工具一段时间后IEJSLeaksDetector,我发现我在最初的问题中谈到的内容不会导致 MEM LEAKS。但是,确实出现了 1 个泄漏。幸运的是,我设法找到了解决方案:

我有一个主脚本,在底部,有一个老派:

window.onload = function()
{
    //do some stuff, get URI, and call:
    this['_init' + uri[0].ucFirst()](uri);//calls func like _initController
    //ucFirst is an augmentation of the String.prototype
}

这会导致 IE8 中的泄漏,我无法使用window.onbeforeunload处理程序修复。看来您必须避免将处理程序绑定到全局对象。解决方案在于闭包和事件监听器,这有点小题大做,但这是我最终做的:

(function(go)
{//use closure to avoid leaking issue in IE
    function loader()
    {
        var uri = location.href.split(location.host)[1].split('/');
        //do stuff
        if (this['_init' + uri[0].ucFirst()] instanceof Function)
        {
            this['_init' + uri[0].ucFirst()](uri);
        }
        if (!(this.removeEventListener))
        {
            this.detachEvent('onload',loader);//(fix leak?
            return null;
        }
        this.removeEventListener('load',loader,false);
    }
    if (!(go.addEventListener))
    {
        go.attachEvent('onload',loader);//(IE...
    }
    else
    {
        go.addEventListener('load',loader,false);
    }
})(window);

这样,(on)load 事件就在window.load处理程序返回之前解除绑定,根据 IEJSLeaksDetector 工具,的应用程序中没有泄漏。我对此很满意。我希望这个片段对你们中的某个人有用 - 如果有人有改进这种方法的建议,请不要犹豫!

干杯,感谢所有在上面阅读和尝试我运球的人!


PS:如果有人关心,这里是 ucFirst String 方法:

if (!(String.prototype.ucFirst))
{
    String.prototype.ucFirst = function()
    {
        "use strict";
        return this.charAt(0).toUpperCase() + this.slice(1);
    };
}
于 2012-07-03T10:19:39.977 回答