[callumacrae]

character

is an argument of the function, and so will be redeclared (in this case, to undefined).

[TheEmpty]

Sole_Wolf, on 22 June 2012 - 04:48 PM, said:

Here is the code that I have and I have labeled different points of the code to show what the value of characters is.

function text(text) {
	characters = ['a','b','c','d','e','f'];
        // POINT 1 outputs correctly
	textdone='';
	
	for(i=0;i<characters.length;i++) {
		textdone = $('#battletxt').text();
		character=characters[i];
                // POINT 2 outputs correctly
		setTimeout(
			function(textdone,character) {
				alert(character);
                                // POINT 3 outputs all NULL
				$('#battletxt').text(character);
			},100);
	}		
}

At point 3 for some reason, the code alerts a message that says undefined.

Does anyone know what is causing this problem or how to fix it?

Thanks.

function text(text) {
	characters = ['a','b','c','d','e','f'];
        // POINT 1 outputs correctly
	textdone='';
	
	for(i=0;i<characters.length;i++) {
		textdone = $('#battletxt').text();
		character=characters[i];
                // POINT 2 outputs correctly
		setTimeout(
			function() {
				alert(character);
                                // POINT 3 outputs all NULL
				$('#battletxt').text(character);
			},100);
	}		
}

[callumacrae]

ThatRailsGuy, on 22 June 2012 - 05:29 PM, said:

function text(text) {
	characters = ['a','b','c','d','e','f'];
        // POINT 1 outputs correctly
	textdone='';
	
	for(i=0;i<characters.length;i++) {
		textdone = $('#battletxt').text();
		character=characters[i];
                // POINT 2 outputs correctly
		setTimeout(
			function() {
				alert(character);
                                // POINT 3 outputs all NULL
				$('#battletxt').text(character);
			},100);
	}		
}

You forgot to fix the awful formatting!

[arronhunt]

Also

textDone

for consistant camelCase formatting

[lobabob]

Doesn't the above just repeat the character 'f' six times instead of going through the entire array?

why don't you just try something along the lines of this:

function text(text) {
        characters = text.split('');
        textDone='';
		i=0;
		
		disloop = setInterval(function() {
			textDone = $('#battletxt').text();
			$('#battletxt').text(textDone+characters[i]);
			
			i++;
			if(i==characters.length) {
				clearInterval(disloop);
			}
		},30);
}

ignore formatting please

[TheEmpty]

lobabob, on 22 June 2012 - 10:42 PM, said:

Doesn't the above just repeat the character 'f' six times instead of going through the entire array?

no, because it is inside of a for loop.

[lobabob]

It repeated when I ran it though......

[TheEmpty]

Did you copy and paste? I'll have to take a look at it later when I'm on a computer.

[TheEmpty]

Yeah, it's because of the timeout, strange. This worked though

function text(text) {
        characters = ['a','b','c','d','e','f'];
        // POINT 1 outputs correctly
        textdone='';
        
        for(i=0;i<characters.length;i++) {
                character=characters[i];
                // POINT 2 outputs correctly
                console.log(character)
        }               
}

[Qasim]

Yes, the timeout calls does the 100ms delay, however the For loop is done within 100ms making i equal "f", so therefore it alerts "f" every time when the timeout is complete. This happens because javascript is synchronous, meaning everything is ran at the same time.

What you need to look into to solve this problem is asynchronous looping, here is a good stackoverflow question on it:
http://stackoverflow...e-in-javascript