timer problem

[genesistr]

Hi,

I have timer class and some classes use timer.
but when i come back from timer with callback function, i can see this. properties.

function Timer(dName, dCallback){
    this.timer = null;
	this.name = dName;
	this.isRunning = false;
	this.interval = 50;
	this.callback = dCallback;
}

Timer.prototype.Start = function()
{
	this.timer = self.setTimeout(this.callback,this.interval);
	this.isRunning = true;
}

var timer;
function File(path) 
{
	....
	if (timer == null) 
		timer = new Timer("load",File.prototype.WaitForUnFinishedFiles);
	....
}

File.prototype.WaitForUnFinishedFiles = function()
{
	var isAllFinished = true;
    for(eFile in cache)
    {
        if(!cache[eFile].isReady)
		{
			isAllFinished = false;
			break;
		}
    }
	debugtext("load :::" + this.fileName);
	if(!isAllFinished)
		timer.Start();
}

and i call it in main.js

	
        file = new File("http://bytes.com/images/simon_00.png");
	file.WaitForUnFinishedF();

i used debugtext just for testing something but it says undefined. and its important for me to see this. properties after coming back from timer.

thanks

[Dormilich]

file.WaitForUnFinishedF();

this method is indeed not defined. try file.WaitForUnF inishedFiles();.

timer = new Timer("load",File.prototype.WaitForUnFinishedFiles);

timer = new Timer("load",File.WaitForUnFinishedFiles);

prototype is not necessary when calling the method…

[genesistr]

sorry i pasted with wrong function name. it's already file.WaitForUnF inishedFiles(); .

[Dormilich]

other than that the shown code looks fine.

[genesistr]

it looks fine for me also but i dont understand why i lost this' objects. i got this error just when i use timer callback method.

[gits]

i think you somewhere start your timers ... so the code that is executed is:

Timer.prototype.Start = function() {
    this.timer = self.setTimeout(this.callback, this.interval);
    this.isRunning = true;
}

here you loose the scope of this since the timeout 'unbinds' the function from your current scope. you might (probably) fix that the following way (basicly a closure):

Timer.prototype.Start = function() {
    var me = this;

    this.timer = self.setTimeout(function() {
        me.callback.apply(me, arguments);
    }, this.interval);

    this.isRunning = true;
}

kind regards

[gits]

hi ... since you pmed me that you have further problems with it i had a closer look at it and now i think you basicly want the this of your file-obj closured ... so you would need to use (the previous answer considered to have the this scope of your timer-obj):

function File(path) {
    this.fileName = path;
    var me = this;
    
    if (timer == null) {
        timer = new Timer("load", function() {
            me.WaitForUnFinishedFiles.apply(me, arguments);
        });
    }
}

kind regards

[gits]

aah ... and perhaps you need to initialize your global timer variable to null?