web-dev-qa-db-ja.com

最後にCリンクリスト挿入ノード

Cのリンクリストの挿入方法に問題があります。リストの先頭にのみ追加するようです。他の挿入は失敗します。そして、このCodeBlocksデバッガーは理解するのがとても難しいので、まだ理解できません。メモリ内のアドレスだけで、価値は決してありません。とにかくこれは私の機能です。失敗する理由はありますか？

/* function to add a new node at the end of the list */
int addNodeBottom(int val, node *head){

    //create new node
    node *newNode = (node*)malloc(sizeof(node));

    if(newNode == NULL){
        fprintf(stderr, "Unable to allocate memory for new node\n");
        exit(-1);
    }

    newNode->value = val;

    //check for first insertion
    if(head->next == NULL){
        head->next = newNode;
        printf("added at beginning\n");
    }

    else
    {
        //else loop through the list and find the last
        //node, insert next to it
        node *current = head;
        while(current->next != NULL)
        {
            if(current->next == NULL)
            {
                current->next = newNode;
                printf("added later\n");
            }
            current = current->next;
        }
    }
    return 0;
}

次に、メインでは、929のみが追加されます。

   //testing addNodeBottom function
    addNodeBottom(929, head);
    addNodeBottom(98, head);
    addNodeBottom(122, head);
    addNodeBottom(11, head);
    addNodeBottom(1034, head);
cinsertlinked-listbinary-tree
9
Tony

このコードは機能します。 samplebiasからの答えはほぼ正しいですが、3番目の変更が必要です。

_int addNodeBottom(int val, node *head){

    //create new node
    node *newNode = (node*)malloc(sizeof(node));

    if(newNode == NULL){
        fprintf(stderr, "Unable to allocate memory for new node\n");
        exit(-1);
    }

    newNode->value = val;
    newNode->next = NULL;  // Change 1

    //check for first insertion
    if(head->next == NULL){
        head->next = newNode;
        printf("added at beginning\n");
    }

    else
    {
        //else loop through the list and find the last
        //node, insert next to it
        node *current = head;
        while (true) { // Change 2
            if(current->next == NULL)
            {
                current->next = newNode;
                printf("added later\n");
                break; // Change 3
            }
            current = current->next;
        };
    }
    return 0;
}
_

変更1：_newNode->next_をNULLに設定する必要があるため、リストの最後に無効なポインターを挿入しません。

変更2/3：ループは、最後の要素を見つけたときに_break;_で飛び出す無限ループに変更されます。 while(current->next != NULL)が以前if(current->next == NULL)と矛盾していたことに注意してください。

編集：whileループに関しては、このようにはるかに優れています：

_  node *current = head;
  while (current->next != NULL) {
    current = current->next;
  }
  current->next = newNode;
  printf("added later\n");
_
12
schnaader

malloc a nodeを実行したら、必ずnode->next = NULL。

int addNodeBottom(int val, node *head)
{    
    node *current = head;
    node *newNode = (node *) malloc(sizeof(node));
    if (newNode == NULL) {
        printf("malloc failed\n");
        exit(-1);
    }    

    newNode->value = val;
    newNode->next = NULL;

    while (current->next) {
        current = current->next;
    }    
    current->next = newNode;
    return 0;
}

このバージョンでは、headはまだダミーとして使用されており、値の保存には使用されていません。これにより、headノードだけで空のリストを表すことができます。

2
samplebias

検討のためにコードを記述する前に、キーについて言及したいと思います。

//キー

temp = malloc関数によって割り当てられた新しいノードのアドレス（member od Cのalloc.hライブラリ）

prev =既存のリンクリストの最後のノードのアドレス。

next =次のノードのアドレスを含む

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

void addnode_end(int a) {
    struct node *temp, *prev;
    temp = (struct node*) malloc(sizeof(node));
    if (temp == NULL) {
        cout << "Not enough memory";
    } else {
        node->data = a;
        node->next = NULL;
        prev = head;

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

        prev->next = temp;
    }
}
0
user2800488

これは古い投稿ですが、参考のためだけです。空のリストの特別なケースチェックなしで追加する方法を次に示しますが、コードはより複雑になります。

void Append(List * l, Node * n)
{
    Node ** next = &list->Head;
    while (*next != NULL) next = &(*next)->Next;
    *next = n;
    n->Next = NULL;
}
0
Ashley Duncan

これはうまくいきます：

struct node *addNode(node *head, int value) {
    node *newNode = (node *) malloc(sizeof(node));
    newNode->value = value;
    newNode->next = NULL;

    if (head == NULL) {
        // Add at the beginning
        head = newNode;
    } else {
        node *current = head;

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

        // Add at the end
        current->next = newNode;
    }

    return head;
}

使用例：

struct node *head = NULL;

for (int currentIndex = 1; currentIndex < 10; currentIndex++) {
    head = addNode(head, currentIndex);
}
0
BullyWiiPlaza

新しいノードは常に、指定されたリンクリストの最後のノードの後に​​追加されます。たとえば、指定されたリンクリストが5-> 10-> 15-> 20-> 25で、最後にアイテム30を追加すると、リンクリストは5-> 10-> 15-> 20-> 25-になります> 30。通常、リンクリストはその先頭で表されるため、最後までリストを走査し、最後のノードの次を新しいノードに変更する必要があります。

/* Given a reference (pointer to pointer) to the head
   of a list and an int, appends a new node at the end  */


    void append(struct node** head_ref, int new_data)
    {
    /* 1. allocate node */
         struct node* new_node = (struct node*) malloc(sizeof(struct node));

        struct node *last = *head_ref;  /* used in step 5*/

    /* 2. put in the data  */
        new_node->data  = new_data;

    /* 3. This new node is going to be the last node, so make next 
          of it as NULL*/
        new_node->next = NULL;

    /* 4. If the <a href="#">Linked List</a> is empty, then make the new node as head */
        if (*head_ref == NULL)
        {
       *head_ref = new_node;
       return;
        }  

    /* 5. Else traverse till the last node */
        while (last->next != NULL)
        last = last->next;

    /* 6. Change the next of last node */
        last->next = new_node;
        return;    
}
0
Nikhil Sharma