使用 C qsort() 排序结构

Question

在实现 C 中内置的 qsort() 以按存储的 int 值 (hitCount) 对结构数组进行排序时遇到了麻烦。

我的结构：

typedef struct words {
    const char *word;
    int hitCount;
} words;

我正在尝试使用 Microsoft (http://support.microsoft.com/kb/73853) 给出的示例。

所以我在顶部：

typedef int (*compfn)(const void*, const void*);

及比较方法：

int compare (words *a, words *b) {
    if (a->hitCount > b->hitCount) {
        return -1;
    } else if (a->hitCount < b->hitCount) {
        return 1;
    } else {
        return 0;
    }
}

然后在另一个方法中，我调用 qsort 并用我的数组名称和其他详细信息替换 Microsoft 示例：

qsort((void *) &output, outputLength, sizeof(words), (compfn)compare);

这会导致分段错误。

我不完全了解如何使用 qsort，所以我假设我从 Microsoft 的示例中改编它的地方我做错了。

我希望我已经包含了这个错误，并且可以对我应该做什么以使其正常工作得到一些启发。

非常感谢！

Answer 1

您必须将数组而不是数组的地址传递给 qsort。

qsort( output, ... );

您的比较函数必须返回一个 int 并接受两个 const void* 参数。 将您的函数 int compare (words *a, words *b) 转换为不同的（但正确的）类型，然后由 qsort() 调用将导致未定义的行为。

比较函数必须是：

int compare (const void *a, const void *b)...

然后你将 a 和 b 转换为正确的类型：

((words*)a)->hitCount < ((words*)b)->hitCount

Answer 2

我怀疑 outputLength 计算不正确。一个完整的工作示例：

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

typedef struct words {
    const char *word;
    int hitCount;
} words;

int compare(const void * left, const void * right) {
    const words * a = (const words *) left;
    const words * b = (const words *) right;
    if (a->hitCount > b->hitCount) {
        return -1;
    } else if (a->hitCount < b->hitCount) {
        return 1;
    } else {
        return 0;
    }
}

int main() {
    struct words output[] = {
      { "hello", 314 },
      { "world", 42 },
      { "answer", 42 }
    };
    int outputLength = sizeof(output) / sizeof(output[0]);
    int i;

    output[0].word = "hello";
    output[0].hitCount = 314;
    output[1].word = "world";
    output[1].hitCount = 42;
    qsort(output, outputLength, sizeof(words), compare);

    for (i = 0; i < outputLength; ++i) {
        printf("%d %s\n", output[i].hitCount, output[i].word);
    }

    return 0;
}

Answer 3

标准库函数qsort的原型是

void qsort(void *base, size_t nmemb, size_t size, 
           int (*compar)(const void *, const void *));

注意compare 函数的签名。 您不能对指向不同签名函数的指针进行类型转换并使其正常工作。因此，对比较函数进行类型转换是行不通的。它必须具有与qsort 原型中声明的签名相同的签名。将您的 compare 函数更改为 -

int compare(const void *a, const void *b) {
    int c = ((words *) a)->hitCount;
    int d = ((words *) b)->hitCount;

    if(c > d) return -1;
    if(c < d) return 1;
    return 0;
}

qsort 的第一个参数base 是包含要排序的元素的缓冲区的基地址。此外，任何指针类型都与void * 变量兼容，因此您不需要转换基地址。因此，您应该将qsort 函数称为 -

qsort(output, outputLength, sizeof output[0], compare);

Answer 4

得到它的工作：

    int compare (const void *a, const void *b) {
        if (((words *)a)->hitCount > ((words *)b)->hitCount) {
            return -1;
        } else if (((words *)a)->hitCount < ((words *)b)->hitCount) {
            return 1;
        } else {
            return 0;
        }
    }

并调用排序：

    qsort(output, outputLength, sizeof(words), compare);

感谢大家的帮助，但大部分归功于“自我”。