Question

我正在研究C中的一个小型学习项目但是实现一个头节点会遇到麻烦，该头节点将保存数据集的第一个数据值。我应该采用未排序的链表并使用它来实现排序的链表。它应分为两个函数，如下例所示。

在为堆上的头指针节点分配内存时出现问题。在使用malloc时，似乎总是创建一个数据值为0的额外头节点（0不在数据集中，但可能是）。

    struct node* create_sorted_list(struct node *head)
    {
       struct node * curr = head;
       struct node * sorted_head = malloc(sizeof(struct node));

       while(curr != NULL){
          add_item_sorted(sorted_head, curr->data);
          curr=curr->next;
       }   
       return sorted_head;
    }



    struct node* add_item_sorted(struct node *sorted_head, int data)
    {
       struct node * curr = sorted_head;
       struct node * newN;    
       while(curr->next != NULL){

          if(data > curr->next->data){
             curr=curr->next;
          }
          else{
             newN = malloc(sizeof(struct node));
             newN->data = data;
             newN->next = curr->next;
             curr->next = newN;
             return sorted_head; 
          }
       }   
       newN = malloc(sizeof(struct node));
       curr->next = newN;
       newN->data = data;
       return sorted_head;
    }



    int main(int argc, char *argv[])
    { 
        ......
        struct node * sorted_head = create_sorted_list(head);

        //head in this case comes from an unsorted linked list with the 
        //same data values. Using head linked list to make sorted_head.
    }

我的输出是这样的：0 - ＆gt; 56 - ＆gt; 35 - ＆gt; 98 - ＆gt;结束
应该是：56 - ＆gt; 35 - ＆gt; 98 - ＆gt;端

我有理由相信额外的节点是malloc的一对多调用的结果，但是我找不到合适的解决方案。

感谢任何帮助。多谢你们。

Answer 1

我相信它正在发生，因为在你的[asda, asdsa]函数中，你正在访问列表，好像有一个虚拟头节点（你开始检查add_item_sorted，你应该在哪里检查{ {1}}）。

当您cur->next->data某个节点时，它可能在其数据字段中保留cur->data或任何其他值。因此，在您的情况下，malloc在其数据字段中保留0。现在，根据您的示例，您尝试插入值sorted_head。在0函数中，首先尝试访问56，即NULL。因此，在此sorted_head之后插入一个新节点。

Answer 2

功能create_sorted_list已经错了。

一般来说，head可以等于NULL。但是，函数中创建的sorted_head不等于NULL。

struct node* create_sorted_list(struct node *head)
    {
       struct node * curr = head;
       struct node * sorted_head = malloc(sizeof(struct node));
       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
       while(curr != NULL){
           //...
       }   
       return sorted_head;
    }

此外，其数据成员未初始化。因此，如果将为此类节点调用函数add_item_sorted，则访问数据成员next将导致程序的未定义行为。

  struct node* add_item_sorted(struct node *sorted_head, int data)
    {
       struct node * curr = sorted_head;
       struct node * newN;    
       while(curr->next != NULL){
             ^^^^^^^^^^^

至少你必须写

  while(curr != NULL){
      sorted_head = add_item_sorted(sorted_head, curr->data);
      ^^^^^^^^^^^^^^^

可以按照以下说明性程序中所示编写函数

#include <stdio.h>
#include <stdlib.h>

struct node
{
    int data;
    struct node *next;
};  

struct node* add_item( struct node *head, int data )
{
    struct node *tmp = malloc( sizeof( struct node ) );
    tmp->data = data;
    tmp->next = head;

    return tmp;
}

struct node* add_item_sorted( struct node *sorted_head, int data );

struct node * create_sorted_list( struct node *head )
{
    struct node *sorted_head = NULL;

    for ( ; head != NULL; head = head->next )
    {
        sorted_head = add_item_sorted( sorted_head, head->data );
    }   

    return sorted_head;
}

struct node* add_item_sorted( struct node *sorted_head, int data )
{
    if ( sorted_head == NULL || !( sorted_head->data < data ) )
    {
        struct node *tmp = malloc( sizeof( struct node ) );

        tmp->data = data;
        tmp->next = sorted_head;
        sorted_head = tmp;
    }
    else
    {
       struct node *current = sorted_head;

        while ( current->next != NULL && current->data < data )
        {
            current = current->next;
        }

        struct node *tmp = malloc( sizeof( struct node ) );
        tmp->data = data;
        tmp->next = current->next;
        current->next = tmp;
    }

    return sorted_head;
}

void display( struct node *head )
{
    for ( ; head != NULL; head = head->next )
    {
        printf( "%d ", head->data );
    }
    printf( "\n" );
}

#define N   10

int main( void ) 
{
    struct node *head = NULL;

    for ( int i = 1; i <= N; i++ ) head = add_item( head, i );

    display( head );

    struct node *sorted_head = create_sorted_list( head );
    display( sorted_head );

    return 0;
}

Rge程序输出

10 9 8 7 6 5 4 3 2 1 
1 2 3 4 5 6 7 8 9 10

链接列表头动态分配 - C

2 个答案: