Random Number in a loop jquery

Question

I would for every loop an other random number, but it does not want worke... I don't know why.

 $(document).ready(function(){

        setTimeout("execute()",5000);

    })

    function execute(){
        for(i=0;i<=5;i++){
            var zWindow = $(window).height();
            var yWindow = $(window).width();
            var z=Math.floor((Math.random() * zWindow) + 1);
            var y=Math.floor((Math.random() * yWindow) + 1);
            $("#egg").css({
                "top": z,
                "left": y
            });
            $("#egg").delay(1000).fadeIn(10).delay(3000).fadeOut(100);
        }

    }

Whats the point of the loop*5 in your execute() function?

Rory McCrossan
– Rory McCrossan

2015-02-26 09:12:59 +00:00
Commented Feb 26, 2015 at 9:12 — Rory McCrossan
– Rory McCrossan, Commented Feb 26, 2015 at 9:12

empiric · Accepted Answer · 2015-02-26 09:43:29Z

1

For stopping an interval, use the clearInterval()- function:

$(document).ready(function(){

    var timesRun = 0; 
    var interval = setInterval(execute, 5000); //only use functionname here

    function execute(){

        timesRun += 1;
        var zWindow = $(window).height();
        var yWindow = $(window).width();

        var z = Math.floor((Math.random() * zWindow) + 1);
        var y = Math.floor((Math.random() * yWindow) + 1);

        $("#egg").css({
            "top": z,
            "left": y
        });
        $("#egg").delay(1000).fadeIn(10).delay(3000).fadeOut(100);

        if(timesRun === 5){
            clearInterval(interval);
        }

     }
});

Demo

Reference

JS Timing

answered Feb 26, 2015 at 9:43

empiric

7,8767 gold badges40 silver badges51 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

xdazz · Accepted Answer · 2015-02-26 09:17:33Z

1

You need setInterval but not setTimeout which will only execute once:

// and use execute directly instead of "execute()"
setInterval(execute, 5000);

answered Feb 26, 2015 at 9:17

xdazz

161k38 gold badges255 silver badges278 bronze badges

1 Comment

Caststupider Over a year ago

but now it runs over and over, i want just 5 Times :)

Collectives™ on Stack Overflow

Random Number in a loop jquery

2 Answers 2

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related