Logic making my life terrible

1

Hi all,

I have been here for 2 weeks now attempting a project and im still here after countless restarts and it all comes down to what i thought was initially simple.

To simplify this, consider the Node class is a simple circle sprite with mouse events enabled. A user clicks on a Node instance, moves the mouse and then clicks to create a new Node.

Once a new node is created, a Pair class instance is created. A Pair takes a reference to what node was originally clicked as well as the newly created node. It draws a line between these nodes.

Lets say i have 3 nodes (n) and therefor 2 Pairs(—).

n1     n2     n3
o-----o-----o

I also have a pairsArray and a nodesArray containing these objects.

Double click means destroy, so n2 is double clicked. My problem now comes from dealing with the pairs.

This is what i have.

function onNodeDoubleClick(e:MouseEvent)void
{
    var node:Node = e.target as Node;

    for (var i:int = 0; i < pairsArray.length; i++)
    {
        var pair:pair = pairsArray* as Pair;
        
        if (node == pair.node1 || node == pair.node2)
        {
            var trash:Pair = pairsArray.splice(pairsArray.indexOf(pair));
            trash.destroy();
        }
    }    
}

I have redone this whole class about 10 times and no matter what i do it always comes to these Pairs letting me down. Am i using arrays incorrectly or is it my logic failing me?

Thanks for reading.