0

我试图构建一个 BST 并在其中插入节点。但是，在创建新节点时，我不断收到 exc_bad 访问错误。可能是什么原因？这是我的代码：

struct Node *node_create(struct BSTree *bst,void *nodeKey, struct Value *nodeVal, struct     Node *rightChild, struct Node *leftChild)
{
struct Node *node = malloc(sizeof *node);
nodeKey= malloc (sizeof (bst->key_size));
nodeVal = malloc(sizeof(bst->value_size));
size_t sizeKey = sizeof(nodeKey);
memcpy(node->key, nodeKey, sizeKey);  // exc_bad access
size_t sizeVal = sizeof (nodeVal);
memcpy(node->val, nodeVal, sizeVal); // exc_bad access
node->right = rightChild;  
node->left = leftChild;

return node;

}

struct Node {
void *key;
struct Value *val;
struct Node *left;
struct Node *right;
};



struct BSTree {
size_t key_size, key_alignment;
size_t value_size, value_alignment;
int (*compare_func)(void *, void *);
struct Node *root;
// ... Maybe some other stuff.
};

struct Value {
char name[10];
int id;
};
4

2 回答 2

2

在不知道Node, 看起来像什么的情况下，我会说，即使您已经分配了node，您还没有分配所有成员（它们似乎是指针）。

将您的代码更改为：

// Allocate node
struct Node *node = malloc(sizeof *node);
// Now members
node->key = malloc (sizeof (bst->key_size)); 
// :

如果您要传入键和值，则将memcpy这些值中的一个执行到上述位置。但很难说没有更多的代码......

于 2013-02-07T14:28:22.233 回答
0

不看节点结构，我猜你想要做的是：

如果节点被定义为

struct Node {
    void *key;
    struct Value *val;
    struct Node *right;
    struct Node *left;
};

然后

struct Node *node_create(struct BSTree *bst,void *nodeKey, struct Value *nodeVal, struct     Node *rightChild, struct Node *leftChild)
{
struct Node *node = malloc(sizeof *node);

  node->key = malloc(bst->key_size);          /* No sizeof here */
  node->val = malloc(bst->value_size);
  memcpy(node->key, nodeKey, bst->key_size);  
  memcpy(node->val, nodeVal, bst->value_size);
  node->right = rightChild;  
  node->left = leftChild;

  return node;
}

由于您不检查mallocs 的返回值（这是一个可以证明的设计选择），您甚至可以用这种方式更简单地编写它。

struct Node *node_create(struct BSTree *bst,void *nodeKey, struct Value *nodeVal, struct     Node *rightChild, struct Node *leftChild)
{
struct Node *node = malloc(sizeof *node);

  node->key = memcpy(malloc(bst->key_size)  , nodeKey, bst->key_size); 
  node->val = memcpy(malloc(bst->value_size), nodeVal, bst->value_size);
  node->right = rightChild;  
  node->left = leftChild;
  return node;
}

有些人对这种风格感到畏缩，但我不想在冗余上过多地稀释我的代码。

于 2013-02-07T14:41:20.437 回答