反向链接列表[重复]

Question

这个问题已经在这里有了答案：

• 如何仅使用两个指针来反转单链列表？ 33个答案

我是一个初学者，正尝试反向链接列表，因此在这里我编写了一个反向链接列表的函数。

void reverse_list(struct node **head_ref)
{
    struct node *temp = *head_ref;
    if(temp == NULL)
        return;
    else if(temp->next == NULL)
        return;
    else
    {
        struct node *temp_r = temp;
        int count = count_list(temp); // no.of nodes in the list
        while(temp != NULL)
        {
            struct node *temp_t = temp_r;
            int i;
            for(i=0;i<count;i++)
            {
                if(i!=0)
                    temp_t = temp_t->next;   // loop to find the node which is to be swapped with temp
            }
            if(temp_t == temp)    // base condition to stop swapping
                return;
            else
            {
                swap_node(&temp_r,temp->data,temp_t->data);
            }
            temp = temp->next; // traverse through the list
            count--; 
        }
    }
}

我使用的逻辑 ：我想通过以这种方式交换节点（1，n），（2，n-1），（3，n-3）来反向链接列表。

但是当我执行此代码时，它只打印列表的第一个元素。

运行调试器后，我了解到我制作了原始列表的两个副本，并且实际上是在交换两个不同列表的节点，这是代码中定义的函数swap_node()无法实现的，这里是swap_node()函数

void swap_node(struct node **head_ref,int key1,int key2) // function to swap two nodes.
{
    if(key1==key2)
        return;
    // search for key1
    struct node *prevx = NULL, *currx = *head_ref;
    while(currx && currx->data != key1)
    {
        prevx = currx;
        currx = currx->next;
    }
    //search for key2
    struct node *prevy = NULL, *curry = *head_ref;
    while(curry && curry->data!=key2)
    {
        prevy = curry;
        curry = curry->next;
    }
    // if key1 or key2 are not present in the list
    if(currx == NULL || curry == NULL)
        return;
    // check if key1 is not head of the list
    if(prevx != NULL)
        prevx->next = curry;
    else
        *head_ref = curry; // then make key2 the head

    // check if key2 is not head of the list
    if(prevy != NULL)
        prevy->next = currx;
    else
        *head_ref = currx; // then make key2 the head

    // swapping the next pointers of the nodes
    struct node *temp = curry->next;
    curry->next = currx->next;
    currx->next = temp;

}

我想使用上述逻辑来反向链接列表，但我无法做到这一点，所以请有人帮我实现这一点，我该如何改进此代码以及如何进行操作。

预先感谢。

Answer 1

您的问题在while循环内：

您不需要节点数，因为while知道遍历列表。

同样，您已经知道，应该有3个指针（上一个，当前和下一个）。

因此，您的代码最终看起来像这样：

void reverse_list(struct node **head_ref) {
    if(*head_ref == NULL || (*head_ref)->next == NULL)
        return;

    struct node *temp = *head_ref;
    struct node *next = NULL;
    struct node *pre = NULL;

    while(temp != NULL)
    {
        next = temp->next;
        temp->next = pre;
        pre = temp;
        temp = next;    
    }
    *head_ref = pre;
}

请注意您如何更新列表的（原始）标题以指向相反的标题。

Answer 2

实际上，您使用的算法对于反转链表太复杂了。

while(currx && currx->data != key1)
{//btw, these code may cause bugs if there are some same value in the linked list
    prevx = currx;
    currx = currx->next;
}

上面的代码表明您并没有真正理解链表的本质 ，在这些代码中，您只是像处理数组一样处理链表。 这将使您的代码运行非常缓慢，应避免使用。

反向链接列表的一种更好更快的方法是这样的：

struct Node
{
    int dataa;
    struct Node* next;
};
typedef struct Node* node;//some definition
node head = NULL;//global variables is not recommended in writing code ,but here it just make the code easier to read .

//Here you should do something to create the linked list

//eg: the linked list is 1 2 3 4 5 now ,the head pointer points at 1
void reverse(void)
{
    if (head != NULL)
    {
        node temp1 = head->next;
        node temp2 = head->next;
        head->next = NULL;
        while (temp2 != NULL)
        {
            temp2 = temp2->next;
            temp1->next = head;
            head = temp1;
            temp1 = temp2;
        }
    }
}
//now it will be 5 4 3 2 1 ,the head pointer points at 5

只需将列表从头到尾依次颠倒即可。