【问题标题】:QuickSorting Linked Lists in CC中的快速排序链接列表
【发布时间】:2016-02-25 21:02:31
【问题描述】:

我正在尝试更熟悉递归,并且我正在尝试以递归方式而不是迭代方式编写函数...我正在尝试为链表创建快速排序函数,我一直在重新评估代码,但我没有'不知道我错过了什么,代码总是离开列表中的最后一个节点,例如......

input list ... 1->2->3->4->NULL
output list .. 4->NULL

这是我的代码...

void lst_qsort(LIST *l){
    if(l==NULL || l->front==NULL || l->front->next==NULL)
        return;
    if(lst_length(l)<=1)
        return;

    LIST *lt = lst_create();
    LIST *pivot = lst_create();

    pivot->front = l->front;
    pivot->back = l->front;

    l->front = l->front->next;
    pivot->front->next = NULL;
    lt = lst_filter_leq(l, pivot->front->val);


    lst_qsort(lt);
    lst_qsort(l);
    lst_concat(l, lt);

}

NOTES lst_filter_leq() 是一个函数,它提取所有出现的 x

LIST * lst_filter_leq(LIST *lst, ElemType cutoff) {
    LIST *lst2 = lst_create();
    NODE *tmp = lst->front;
    int i = 0, n=1;

    while(lst->front != NULL){
        if(lst->front->val <= cutoff){
            lst_push_back(lst2, lst->front->val);
        }
        lst->front = lst->front->next;
    }
    lst->front = tmp;
    for(i=cutoff; i>=0; i--){
        n = lst_remove_all_slow(lst, i);
    }

    return lst2;
}

lst_concat() 将两个列表合并在一起

【问题讨论】:

  • 你真的需要if(lst_length(l)&lt;=1),因为你已经检查了l-&gt;front-&gt;next==NULL等,上面?
  • 我认为它的这个 OR || l-&gt;front-&gt;next==NULL 当然这意味着当队列前面之后的下一个值为 NULL 时返回。我假设在列表中最后一个值,即 null 不被视为元素
  • @PaulSullivan 遗憾的是,这不是问题,但是,列表现在是空的
  • @e0k 是的,你是对的,我想我不需要它,谢谢
  • 注意:lst_filter_leq() 是一个可能会或可能不会正常工作的函数。请发帖Minimal Complete Verifiable Example

标签: c linked-list quicksort


【解决方案1】:

查看您的代码和示例。首先,看起来似乎没有必要创建pivot 列表,因为您只使用pivot-&gt;front-&gt;val,我认为这是您列表中的第一个值(因为您之前使用了pivot-&gt;front = l-&gt;front)。

所以对于值 [1, 2, 3, 4],您在第一步中执行以下操作(我使用伪代码来显示列表成员):

// you extracted '1' from your list, because you did l->front = l->front->next;
lt = lst_filter_leq([2, 3, 4], 1);

这意味着你什么都没有结束(列表中没有一个值小于或等于 1)。接下来对l(即[2, 3, 4])进行快速排序,然后连接llt 的结果。

但是当快速排序 [2, 3, 4] 时,你什么也得不到,快速排序 [3, 4],
这没什么,[4]。
[4] 的快速排序返回 [4]。因此你的结果。

看来您忘记了在llt 列表之间插入pivot

使用glib 库实现双端队列:

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

// defines to maintain compatibility with OP code
#define LIST GQueue
#define lst_create g_queue_new
#define lst_length g_queue_get_length

// lst_push_back implementation using GQueue
void lst_push_back(LIST *list, int val) {
    g_queue_push_tail(list, GINT_TO_POINTER(val));
}

// lst_pop_front implementation using GQueue
int lst_pop_front(LIST *list) {
    return GPOINTER_TO_INT(g_queue_pop_head(list));
}

// adds elements from list2 to list1
// list2 is destroyed afterwards
void lst_concat(LIST *list1, LIST *list2) {
    int length = lst_length(list2);
    int i, v;
    for (i = 0; i < length; ++i) {
        v = lst_pop_front(list2);
        lst_push_back(list1, v);
    }
    g_queue_free(list2);
}

// filters 'list' elements
// elements less or equal 'value' are returned in newly created list
// elements greater than 'value' are left in 'list'
LIST *lst_filter_leq(LIST *list, int value) {
    LIST *lte = lst_create();
    LIST *gt  = lst_create();
    int length = lst_length(list);
    int i, v;
    for (i = 0; i < length; ++i) {
        v = lst_pop_front(list);
        if (v <= value) {
            lst_push_back(lte, v);
        } else {
            lst_push_back(gt, v);
        }
    }
    lst_concat(list, gt);
    return lte;
}

void lst_qsort(LIST *l) {
    if (l == NULL) {
        return;
    }
    if (lst_length(l) <= 1) {
        return;
    }

    LIST *lt = lst_create();
    LIST *pivot = lst_create();

    int val = lst_pop_front(l);
    lst_push_back(pivot, val);

    // this function divides the list int two parts
    // elements less or equal 'val' are returned in lt list ("right" list)
    // elements greater than 'val 'are left in l list ("left" list)
    lt = lst_filter_leq(l, val);

    lst_qsort(lt); // sort "right" part of list

    lst_qsort(l);  // sort "left" part of list

    lst_concat(l, pivot); // add the pivot element
    lst_concat(l, lt);    // add right part of list
}

void printList(LIST *list) {
    GList *vList = g_queue_peek_head_link(list);
    while (vList != NULL) {
        printf("%d", GPOINTER_TO_INT(vList->data));
        vList = g_list_next(vList);
        if (vList != NULL) {
            printf("->");
        } else {
            printf("\n");
        }
    }
}

int main(void) {
    LIST *l = lst_create();
    lst_push_back(l, 4);
    lst_push_back(l, 3);
    lst_push_back(l, 2);
    lst_push_back(l, 1);

    printf("unsorted: ");
    printList(l);
    lst_qsort(l);
    printf("sorted: ");
    printList(l);

    g_queue_clear(l);

    lst_push_back(l, 1);
    lst_push_back(l, 2);
    lst_push_back(l, 3);
    lst_push_back(l, 4);

    printf("unsorted: ");
    printList(l);
    lst_qsort(l);
    printf("sorted: ");
    printList(l);

    g_queue_clear(l);

    srand(time(NULL));
    int i;
    for (i = 0; i < 16; ++i) {
        lst_push_back(l, rand() % 32);
    }
    printf("unsorted: ");
    printList(l);
    lst_qsort(l);
    printf("sorted: ");
    printList(l);

    g_queue_free(l);

    return 0;
}

示例输出:

unsorted: 4->3->2->1
sorted: 4->3->2->1
unsorted: 1->2->3->4
sorted: 4->3->2->1
unsorted: 27->16->20->4->14->30->26->28->10->13->19->1->30->8->3->14
sorted: 30->30->28->27->26->20->19->16->14->14->13->10->8->4->3->1

【讨论】:

  • 如果您从未重新添加枢轴,则列表每次都会缩短一。当只剩下一个节点时,它会停止尝试,这就是您的示例结果。
  • 那么你是在建议我concat(lt,pivot) 很抱歉无法理解你的答案。
  • @MComp:你应该连接 (lt, pivot) 然后连接 (lt, l)。那么lt 就是你的排序列表。
  • @MComp 通常,列表分为三部分:左、枢轴、右。然后你递归地左右排序,最后从三个部分重新组装列表。如果您只将输入分成两个列表,您会冒这样的风险:一个包含 2 个项目的输入列表将导致一个包含 2 个项目的左列表和一个包含 0 个项目的右列表,从而导致无限递归。
  • @MComp:如果你做了我写的,那么你应该让你的l指向lt中返回的值。做相反的事情:concat (l, pivot),然后 concat (l, lt),但你会以相反的排序列表结束。