insertAtTail 未在链表中添加节点答案

【问题标题】：insertAtTail is not adding node in linkedlistinsertAtTail 未在链表中添加节点
【发布时间】：2019-12-10 08:26:45
【问题描述】：

我是数据结构的新手。我正在创建一个链表程序。我已经创建了程序。但是不明白为什么 insertAtTail 没有将节点添加到链表中。

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

struct node *head=NULL;

int main( void ) {
    insertAtHead(3);
    insertAtHead(4);
    insertAtHead(5);
    insertAtHead(8);
    insertAtTail(2);
    display();

    return 0;
}

void insertAtHead( int data ){
    struct node *newNode;

    newNode = (struct node *)malloc( sizeof(struct node) );
    newNode->data = data;
    newNode->next = NULL;

    if( head == NULL ){
        head = newNode;
    } else {
        newNode->next = head;
        head = newNode;
    }
}

void insertAtTail(int data){
    struct node *newNode, *temp;

    newNode = (struct node *)malloc( sizeof(struct node) );
    newNode->data = data;
    newNode->next = NULL;

    if( head == NULL){
        head = newNode;
    } else {
        temp = head;
        while ( temp != NULL){
            temp = temp->next;
        }

        temp = newNode;
    }

}

void display(){
    struct node *temp;

    temp = head;
    while ( temp != NULL)
    {
        printf("%d -> ", temp->data);
        temp = temp->next;
    }
    printf("NULL\n");   
}

期望输出：8 -> 5 -> 4 -> 3 -> 2 -> NULL

实际输出：8 -> 5 -> 4 -> 3 -> NULL

【问题讨论】：

难道你不应该将最后一个节点的 temp->next 更改为指向你的节点...

标签： c linked-list insert singly-linked-list

【解决方案1】：

变量temp是函数的局部变量，它占用的内存与链表的节点不同。

在这个循环之后

    temp = head;
    while (temp != NULL) {
        temp = temp->next;
    }

变量temp 设置为NULL。那么在这个语句中

    temp = newNode;

您正在更改变量本身占用的内存。列表的节点没有改变。

按以下方式更改循环

    temp = head;
    while (temp->next != NULL) {
        temp = temp->next;
    }

    temp->next = newNode;

还要注意两点，main中使用的所有函数都必须在使用前声明。

例如

void insertAtHead( int data );

//...

int main( void )
{
    //...
}

此外，函数处理全局变量head 也不是一个好主意。您应该重写函数，以便将指向列表的头部作为参数传递给函数。

例如在这种情况下，函数insertAtHead可以定义如下方式

int insertAtHead( struct node **head, int data )
{
    struct node *newNode = malloc( sizeof(struct node) );

    int success = newNode != NULL;

    if ( success )
    {
       newNode->data = data;
       newNode->next = *head;
       *head = newNode;
    }

    return success;
}

作为一个程序的结果，您可以通过在主头中本地声明来拥有多个列表。

这是一个演示程序

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

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

int insertAtHead( struct node **head, int data )
{
    struct node *newNode = malloc( sizeof(struct node) );

    int success = newNode != NULL;

    if ( success )
    {
       newNode->data = data;
       newNode->next = *head;
       *head = newNode;
    }

    return success;
}

int insertAtTail( struct node **head, int data )
{
    struct node *newNode = malloc( sizeof(struct node) );
    int success = newNode != NULL;

    if ( success )
    {
        newNode->data = data;
        newNode->next = NULL;

        while ( *head != NULL ) head = &( *head )->next;

        *head = newNode;
    }

    return success;
}

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

    puts( "NULL" );   
}

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

    insertAtHead( &head, 3 );
    insertAtHead( &head, 4 );
    insertAtHead( &head, 5 );
    insertAtHead( &head, 8 );
    insertAtTail( &head, 2 );

    display( head );

    return 0;
}

它的输出是

8 -> 5 -> 4 -> 3 -> 2 -> NULL

【讨论】：

【解决方案2】：

改变这个：

while (temp != NULL)  
  temp = temp->next;
}

temp = newNode;

到这里：

while (temp->next != NULL) {
  temp = temp->next;
}
temp->next = newNode;

因为你要做的是去old last node，做完do之后，指定old next to the oldlast node，即new 最后一个节点是newNode。

Live demo（请注意我是如何将您的方法的定义移到 main 方法之前的，以便它可以编译 - 或者，您可以在 main 之前声明原型，并且仍然保留定义为你现在有它们了）。

PS：Do I cast the result of malloc?号

【讨论】：

【解决方案3】：

temp = head;
while ( temp != NULL) {
    temp = temp->next;
}
temp = newNode;

在这里重新分配temp 将无济于事。相反，检查 temp->next 是否为 null 并使用它：

temp = head;
while (temp->next != NULL) {
    temp = temp->next;
}
temp->next = newNode;

这样，您仍然在更改head，而不是temp。

【讨论】：