StackOverflow错误AS3

Question

这是我收到StackOverflow错误的代码。我不完全确定它有什么问题。代码是即插即用的，所以你可以将其插入并自行测试。有人可以帮我吗？我基本上是从一个数组中创建2个不同的对象，并尝试去除被点击的对象，然后我将该对象放入不同的数组中。

import flash.sampler.NewObjectSample;
import flash.display.Sprite;
import flash.events.MouseEvent;

var eating_breakfast:Sprite;
var walking:Sprite;
var swimming:Sprite;
var art:Sprite;
var choices:Array = new Array ();

//Sprite Creation
eating_breakfast = new Sprite ();
eating_breakfast.graphics.beginFill(0xE39D43);
eating_breakfast.graphics.drawRect(0,0,50,50);
eating_breakfast.graphics.endFill();
eating_breakfast.x = 50;
eating_breakfast.y = 50;


walking = new Sprite ();
walking.graphics.beginFill(0xC3266C);
walking.graphics.drawRect(0,0,50,50);
walking.graphics.endFill();
walking.x = 100;
walking.y = 100;


swimming = new Sprite ();
swimming.graphics.beginFill(0x48AFD1);
swimming.graphics.drawRect(0,0,50,50);
swimming.graphics.endFill();
swimming.x = 150;
swimming.y = 150;


art = new Sprite ();
art.graphics.beginFill(0xafdb44);
art.graphics.drawRect(0,0,50,50);
art.graphics.endFill();
art.x = 200;
art.y = 200;

//adding sprites into array
choices.push( eating_breakfast);
choices.push(walking);
choices.push(swimming);
choices.push(art);


var indexcount = 0;
var randomize:Number;
var storageArray: Array = new Array ();
civilizedorder();
randomizedorder();
this.addEventListener(MouseEvent.CLICK,switchpic);

//pick the target generated object
function switchpic(t:MouseEvent)
{
    //for index count
    // this works as a target so if your mouse target is the object generated by indexcount this will initiate
    if (t.target == choices[indexcount])
    {
        storageArray.push(choices[indexcount]);
        removeChild(choices [indexcount]);
        removeChild(choices [randomize]);
        choices.splice(indexcount,1);
        goNext();

    }
    // for randomize
    if (t.target == choices[randomize])
    {
        // this works as a target so if your mouse target is the object generated by randomize this will initiate
        storageArray.push(choices[randomize]);
        removeChild(choices [indexcount]);
        removeChild(choices [randomize]);
        choices.splice(randomize,1);
        indexcount++;

        goNext();
    }
}

//generates the index count object
function civilizedorder()
{
    trace("The Index count is" + indexcount);
    addChild(choices [indexcount]);
    choices[indexcount].x = 300;


}
trace("The number of choices in the choice array is " + choices.length);
//generates the randomized object
function randomizedorder()
{

    randomize = Math.floor(Math.random() * choices.length);
    trace("the random number is" + randomize);
    if (randomize == indexcount )
    {
        randomizedorder();
    }
    else
    {
        addChild(choices [randomize]);
    }

}


function goNext()
{
    trace("The storagearray has " + (storageArray.length));
    if (choices.length < 0 || choices.length > 0)
    {
        if (indexcount > choices.length-1)
        {
            indexcount = choices.length - 1;
        }
        civilizedorder();
        randomizedorder();
    }
}

Answer 1

Stack Overflow意味着你有太多的递归。在这种情况下，当randomizedorder为1且choices.length为0（即第一次调用indexcount）时，可能在goNext函数中，它会产生无限循环。 / p>

您需要重新考虑此计划的结构。尽可能避免递归。循环更好，但你也不需要它们;修复一个功能：

randomize = Math.floor(Math.random() * (choices.length - 1));
if (randomize >= indexcount ) {
    randomize ++;
}

你可能仍会得到奇怪的结果，因为它没有像你期望的那样被调用，但堆栈溢出应该消失。