Is it possible to remove the Maximum call stack size?

Question

I am making a weird block movement thing, but after it moved 10 times it says:

Uncaught RangeError: Maximum call stack size exceeded

And my purpose is to let it move the whole time, here is the code BTW:

<html>
    <head>
        <title>Look - The game</title>
    </head>
    <body>
        <div id="square" style="position:absolute; width:5px; height:5px; background:black"></div>
        <script src="http://code.jquery.com/jquery-2.1.1.min.js"></script>
        <script>
            var square = document.getElementById("square");
            var duration = 1000;
            var steps = 1;
            function movesquare(){
                var randomtop = Math.floor(Math.random() * screen.height);
                var randomleft = Math.floor(Math.random() * screen.width);
                $(square).animate({marginTop:randomtop, marginLeft:randomleft}, duration, movesquare);
                duration -= steps;
                steps = steps * 2;
            }
            movesquare();
        </script>
    </body>

Is there a reason why you are using recursion? Perhaps this is inadvertent? This is the cause of your stack problems. — Lee Taylor
– Lee Taylor, Commented Dec 7, 2014 at 15:49
Maximum call stack size exceeded indicates that you have an a kind of endless loop and if the browser would not stop your script, your browser most likely will become unresponsive in such a situation. — t.niese
– t.niese, Commented Dec 7, 2014 at 15:50
I want to get it moving the whole time, or is there an another way to do that. Here is btw the page — Ebbez
– Ebbez, Commented Dec 7, 2014 at 15:51
@LeoDeng it's only going to be recursive when duration is less than or equal to zero. So long as it's greater than zero, there'll be no recursion. — Pointy
– Pointy, Commented Dec 7, 2014 at 15:59

t.niese · Accepted Answer · 2014-12-07 15:56:02Z

3

Your problem is that:

$(square).animate({marginTop:randomtop, marginLeft:randomleft}, duration, movesquare);

Will call movesquare immediately when duration is 0 or smaller. At the time this happens you created an endless loop.

You need to make sure that duration does not become 0.

answered Dec 7, 2014 at 15:56

t.niese

41k9 gold badges78 silver badges111 bronze badges

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

1 Comment

Ebbez Over a year ago

Thanks, I didn't realised that the duration will became lower than 0 in 10 steps.

Collectives™ on Stack Overflow

Is it possible to remove the Maximum call stack size?

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related