0

任何人都可以帮助我解决这个分段错误。这段代码很简单,但错误很难弄清楚。

struct Link {
  int key;
  unsigned data: 2;
  struct Link *next;
  struct Link *previous;
};

struct Link* addInOrder(struct Link *, struct Link);

int main() {
  struct Link *head;
  struct Link data1;
  struct Link data2;
  struct Link data3;
  data1.key = 25;
  data1.data = 1;
  data1.next = NULL;
  data2.key = 50;
  data2.data = 0;
  data2.next = NULL;
  data3.key = 100;  
  data3.data = 2; 
  data3.next = NULL;
  head = NULL;
  head = addInOrder(head, data2);
}

struct Link* addInOrder(struct Link *srt, struct Link l) {
  if(!srt) {
    return &l;
  }

  struct Link *temp = srt;
  while(temp->next && l.key > temp->key)
    temp = temp->next;

  printf("here\n");

  if(l.key > temp->key) {
    printf(" 1\n");
    temp->next = &l;
    l.previous = temp;
  }
  else {
    printf(" 2\n");
    l.previous = temp->previous;
    l.next = temp;
    printf( "2.2\n");
    if(temp->previous) {
      //printf("%i\n",temp->previous->key);
      temp->previous->next = &l;
    }
    printf(" 2.3\n");
    temp->previous = &l;
  }
  return srt;
}

我在 addInOrder() 的第一行不断收到错误消息。编译器所说的只是分段错误。

编辑:另外,如果我添加printf("..."); 在 if 语句之后并运行它...不打印

4

2 回答 2

3

您正在传递addInOrder()按值 ( struct Link l) 的第二个参数。这会在您调用函数时创建参数的副本,并且 inaddInOrder()存在l于堆栈中。然后,您将返回局部变量的地址并将其分配给head,但是当函数退出时,该变量超出范围并被释放。因此,您为 分配了无效地址head,这会导致段错误。

于 2012-12-06T01:53:02.303 回答
2

零件(到处&l都用)

if (!srt)
    return &l;

正在返回堆栈变量的地址。您的addInOrder函数可能应该有签名

struct Link* addInOrder(struct Link* srt, struct Link* l);
于 2012-12-06T01:53:09.713 回答