IE8内存泄漏？

Question

http://jsfiddle.net/mplungjan/SyHFR/

抱歉代码混乱 - 我修改了脚本不是由我选择的，最初来自 Dynamic Drive - 它是由我正在帮助的人使用的，而不是从头开始重写整个事情，我同意功能蔓延。预期的更改是在 vars 之后添加重复和延迟

现在我只想了解哪里可能存在问题 - 我将代码从每秒使用日期对象更改为仅在初始化时使用它。

代码

cdtime.prototype.updateTime=function(){
  var thisobj=this;
  this.currentTime+=1000; // one second
  setTimeout(function(){thisobj.updateTime()}, 1000) //update time every second
}

给出了一条

Message: 'thisobj' is null or not an object

在 XP 上的 IE8 中运行大约 9 个小时后，

我现在周末在另一个盒子上运行它，但想知道是否有人可以启发我想知道 IE 可能有什么问题。

HMM - 现在粘贴该函数，我看到 settimeout 位于原型内部 - 突然看起来不正确。

也请随时向我指出一个更好的计数器，可以完成这个计数器的工作例如，在延迟后开始，在给定时间后重复，并在由 CSS 设计的页面上有多个计数器。

更新

尝试 setInterval 使整个倒计时非常不稳定 http://jsfiddle.net/mplungjan/z2AQF/

Answer 1

如果确实是内存泄漏，请尝试从传递给 setTimeout 的函数中清除 thisobj 变量：

cdtime.prototype.updateTime = function () {
    var thisobj=this;
    this.currentTime+=1000; // one second
    setTimeout(function () {
        thisobj.updateTime();
        thisobj = null;
    }, 1000); //update time every second
};

如果你仔细观察，这个函数基本上是一个区间，所以下面的代码更加优化，因为它不会叠加在旧函数上：

cdtime.prototype.updateTime = function () {
    var thisobj = this;
    setInterval(function () {
        thisobj.currentTime += 1000;
    }, 1000);
};