Binary arbre de recherche C ++ (Parents)

Question

Binary arbre de recherche C ++ (Parents)

voix

0

J'ai juste besoin d'un peu plus d'aide sur mon BST. C'est ce que mon BST ressemble lors de l'insertion:

R, L, J, G

                      R   --Root at Index 0
                     / \
     L @ Index1     L   NULL
                   / \
     J @ Index3   J   NULL
                 / \
     G @ Index7 G  NULL

Voici le code qui fait se produire.

void BST::insert(const data &aData)
{   
    if ( items[Parent].empty ) 
    {
        items[Parent].theData = aData; // insert at leaf.
        items[Parent].empty = false;
        size++;

        return;
    }           
    for ( int i = 0; i <= size; i++ )
    {
        if ( aData < items[Parent].theData )
        {
            if ( items[2*i+1].empty )
            {
            items[2*i+1].theData = aData;
            items[2*i+1].empty = false;
            }
            else 
                           {
            // we must already have a left child to some root.
                                Parent++;  So make the previous data the root???
            if ( items[Parent].empty )
            {
                items[Parent].theData = items[2*i+1].theData;
                items[Parent].empty   = false;
                Parent = (i-1)/2;
            }
                           } 
        }
        else
        { ...// do the same for data greater than but with items[2*i+2] }

Ma question est que quand aurais-je besoin de faire une nouvelle racine? Quand aurais-je besoin de faire une nouvelle racine? Pour recomparison?

Cette approche est correcte? Merci à tous ceux qui, même à la fois de regarder mes messages :)

// Le constructeur de la classe BST et sa section privée.

BST::BST(int capacity) : items(new item[capacity]), size(0), Parent(0), 
leftChild(0), rightChild(0)
{
    items->empty = true;
    maxSize = capacity;
}
private:
    int size;  // size of the ever growing/expanding tree :)
    int Parent;
    int maxSize;    
    int leftChild;
    int rightChild;
    struct item
    {
        bool empty;
        data theData;
    };
    item *items;    // The tree array

c++binary-search-tree

Créé 17/11/2009 à 21:59
source utilisateur user40120

Dans d'autres langues...

2 réponses

BostonLogan · Answer 1 · 2009-11-17 22:12

voix

1

Votre logique (je dois dire assez floue) semble se tromper: Quel genre de « si » séquence est-ce?

if ( items[2*i+1].empty )
{
}
else if (!items[2*i+1].empty)
{
   if ( items[2*i+1].empty )
   {
        // any code here is unreachable
   }
}

Créé 17/11/2009 à 22:12
source utilisateur BostonLogan

Andrey · Answer 2 · 2009-11-17 22:46

Je vous suggère de réimplémentez que cela fonctionne de manière récursive. Quelque chose comme ça:

void BST::insert(const data& aData, int pos) {
    if (items[pos].empty) {
        // insert it here
    }
    else (aData < items[pos].theData) {
        // go to left child
        insert(aData, 2*pos + 1);
    }
    else {
        // go to right child
        insert(aData, 2*pos + 2);
    }
}

Ce n'est pas vraiment clair que Parent, leftChild et rightChild font dans votre classe, mais c'est une question distincte.