Двоичное дерево поиска C ++ (родители) - PullRequest
Новая
       12

Двоичное дерево поиска C ++ (родители)

0 голосов
/ 17 ноября 2009

Мне нужно немного больше помощи для моего BST. Вот как выглядит мой BST при вставке:

R, L, J, G 

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

Вот код, который делает это возможным. 

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] }

Мой вопрос: когда мне нужно будет создать новый рут? Когда мне нужно будет сделать новый рут? Для повторного сравнения?

Правильный ли этот подход? Спасибо тем, кто хоть как то посмотрел мои посты:)

// Конструктор класса BST и его частного раздела. 

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

Ответы [ 2 ]

1 голос
/ 18 ноября 2009

Я предлагаю вам переопределить это, чтобы работать рекурсивно. Примерно так: 

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);
    }
}

Не совсем понятно, что Parent, leftChild и rightChild делают в вашем классе, но это отдельная проблема.

1 голос
/ 18 ноября 2009

Ваша логика (я должен сказать, довольно нечеткая) неверна: Что это за последовательность "если"? 

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