SetTimeout fails

Question

SetTimeout fails

My setTimeout function doesn't want to work. I am not getting any warnings, so I guess I did something incredibly stupid.

var timeout = 1000;
for(k=0;k<pages.length;k++)
{
    randomnumber=Math.floor(Math.random()*pages[k].length);
    setTimeout(function() {
        $.ajax({
            type: 'GET',
            url: pages[k][randomnumber],
            success: function(data) {
                alert(data);
                if(data.indexOf('VIDEO_LENGTH') > 0)
                {
                    timeouttext = data.substr(data.indexOf('VIDEO_LENGTH')+12);
                    timeouttext = timeouttext.substr(timeouttext.indexOf('.'));
                    timeout = parseInt(timeouttext);
                    alert(timeout);
                }
                else
                    timeout = 1000;
                $('#loader').hide('fast','fade');
                $('#information').html(data);
                $('#information').show('fast','fade');
            }
        })
    },timeout);
    alert("PAGE " + k + " RandomNumber " + randomnumber + " : " + pages[k][randomnumber]);
    if(k==3) {
        k = 0;
    }
}

Can anyone help me.

+3

javascript

Albert 19 Mar 12 at 12:26

source to share

4 answers

When you debug your code with the FireBug or Chrome developer tools, do you see your Ajax requests?

It seems to me that you never get caught in a "successful" event for every request. Since you don't have an "error" event in your Ajax request, you will never see any feedback.

+1

Louis Ricci 19 Mar 12 at 12:31

source to share

Your setTimeout is correct. To additionally check for errors:

check if your request is being sent.
add error: function(){}

or complete:function(){}

block that logs the status of the request to see if your server side is breaking your code.

edit: (I suggest you don't use shorthand if

, it's very error prone.

+1

Christoph 19 Mar 12 at 13:06

source to share

Try the ( Updated ) fiddle here .

var funfacts= ['URL1','URL2','URL3'];
var multimedia= ['URL4','URL5'];
var employees = ['URL6','URL7','URL8','URL9','URL10'];
var information = ['URL11','URL12','URL13'];

var pages = [funfacts, multimedia, employees, information];
var timeout=1000;
var t=0;
for(i=0;i<pages.length;i++)
{
    (function(value){
        for(j=0;j<pages[value].length;j++)
        {
            t++;
            (function(iv, ij){
                var tmo = timeout*t;
                setTimeout(function(){
                    callAjax(iv, ij);
                }, tmo);

            })(value, j); 
        }

    })(i);             
}

function callAjax(i, j)
{
    $.ajax({
        type: 'GET',
        url: pages[i][j],
        success: function(data) {    
            if(data.indexOf('VIDEO_LENGTH') > 0) // indexOf should be checked if(var.indexOf('sometext')!=-1)
            {
                // your code
            }
            else
            {
                // your code
            }
        }
    });
}

If everything else is okay, then it should work.

you can also read this and this in SO format.

0

The alpha 19 Mar 12 at 14:29

source to share

Luca rainone · Accepted Answer · 2012-03-19T12:54:01+0000

EDIT: ok follows the error:

var k when you call the page url is not what you expected. So try:

{
    randomnumber=Math.floor(Math.random()*pages[k].length);
    var selectedIndex = k; // store current k
    setTimeout(function() {

    $.ajax({
        type: 'GET',
        url: pages[selectedIndex][randomnumber], // use stored k
 ...

SetTimeout fails

More articles: