Ошибка BUS в клонировании дерева avl в c - PullRequest
Новая
       9

Ошибка BUS в клонировании дерева avl в c

0 голосов
/ 30 апреля 2018

Я пытаюсь клонировать AVLTree в C, но у меня есть 

BUS ERROR:10

в моем коде. Мне действительно нужно начать улучшать свои навыки программирования, много. Вот почему каждую неделю я пытался практиковать некоторые упражнения по программированию. Для этого я решил сделать базовое дерево AVL. Но у меня так много трудностей с этим. 

    typedef struct AVLTreeNode {
    int key; 
    int value;  
    int height; 
    struct AVLTreeNode *parent; 
    struct AVLTreeNode *left; 
    struct AVLTreeNode *right; 
} AVLTreeNode;

typedef struct AVLTree{
    int  size;      // count of items in avl tree
    AVLTreeNode *root; // root
} AVLTree;

AVLTreeNode *newAVLTreeNode(int k, int v )
{
    AVLTreeNode *new;
    new = malloc(sizeof(AVLTreeNode));
    assert(new != NULL);
    new->key = k;
    new->value = v;
    new->height = 0; // height of this new node is set to 0
    new->left = NULL; // this node has no child
    new->right = NULL;
    new->parent = NULL; // no parent
    return new;
}

AVLTree *newAVLTree()
{
    AVLTree *T;
    T = malloc(sizeof (AVLTree));
    assert (T != NULL);
    T->size = 0;
    T->root = NULL;
    return T;
}

    AVLTreeNode *CloneAVLTreeNode(AVLTreeNode *r)
{
    AVLTreeNode *n;
    if (r == NULL) return NULL;
    else
    {
        n->key = r->key;
        n->value = r->value;
        n->height = r->height;
        n->parent = r->parent;
        n->left = CloneAVLTreeNode(r->left);
        n->right = CloneAVLTreeNode(r->right);
        return n;
    }

}


AVLTree *CloneAVLTree(AVLTree *T)
{
    AVLTree *New = malloc(sizeof(AVLTree));
    //AVLTree *New;
    AVLTreeNode *p = New->root;
    AVLTreeNode *n = T->root;
    if (n == NULL) return NULL;
    else
    {
        p = CloneAVLTreeNode(n);

    }   
    return New;
}

Можете ли вы, ребята, дать мне несколько советов и указать мне в правильном направлении. Это действительно раздражает меня, потому что я знаю, что это упражнение не должно быть таким трудным.

Заранее спасибо.

1 Ответ

0 голосов
/ 30 апреля 2018

Есть как минимум две следующие проблемы:

Первая проблема: 

AVLTreeNode *CloneAVLTreeNode(AVLTreeNode *r)
{
  AVLTreeNode *n;               // here you declare n, but n just points nowhere
  if (r == NULL) return NULL;
  else
  {
    n->key = r->key;            // here n still points nowhere,
                                // therefore dereferencing it causes the Bus error.
    n->value = r->value;
    n->height = r->height;
    n->parent = r->parent;
    n->left = CloneAVLTreeNode(r->left);
    n->right = CloneAVLTreeNode(r->right);
    return n;
  }
}

Вы, вероятно, хотите это: 

AVLTreeNode *CloneAVLTreeNode(AVLTreeNode *r)
{
  if (r == NULL) return NULL;
  else
  {
    AVLTreeNode *n = malloc(sizeof AVLTreeNode);  // allocate memory for new node
    n->key = r->key;
    n->value = r->value;
    n->height = r->height;
    n->parent = r->parent;
    n->left = CloneAVLTreeNode(r->left);
    n->right = CloneAVLTreeNode(r->right);
    return n;
  }
}

Вторая проблема: 

AVLTree *CloneAVLTree(AVLTree *T)
{
  AVLTree *New = malloc(sizeof(AVLTree));
  //AVLTree *New;
  AVLTreeNode *p = New->root;    // here New->root hasn't been initialized either
                                 // so p will point nowhere either 

  AVLTreeNode *n = T->root;
  if (n == NULL) return NULL;
  else
  {
    p = CloneAVLTreeNode(n);     // but here you assign p with another value, so the previous 'p = New->root'
                                 // is pointless anyway.

  }
  return New;
}

Скорее всего, в другом месте больше ошибок.

...