您的代码有问题,因为您正在对无符号数据进行有符号比较。使用以下替代方法之一:
更正统
这明显更快。
int compare_digests(const void *a, const void *b)
{
const struct digest_line *aa = (const struct digest_line *) a;
const struct digest_line *bb = (const struct digest_line *) b;
if (aa->first > bb->first)
return +1;
else if (aa->first < bb->first)
return -1;
else if (aa->second > bb->second)
return +1;
else if (aa->second < bb->second)
return -1;
else
return 0;
}
少正统
这明显变慢了;不要使用它。
int compare_digests(const void *a, const void *b)
{
struct digest_line aa = *(struct digest_line *) a;
struct digest_line bb = *(struct digest_line *) b;
if (aa.first > bb.first)
return +1;
else if (aa.first < bb.first)
return -1;
else if (aa.second > bb.second)
return +1;
else if (aa.second < bb.second)
return -1;
else
return 0;
}
时间
做了一些测量后,很明显‘不那么正统’的方法也比较慢。超过 20 次运行(每次进行 100,000,000 次迭代,每次迭代中比较不同的值对),我得到了平均时间和标准偏差(以秒为单位):
Mean Standard Deviation
Value 0.732914 0.005000
Pointer 0.655853 0.003895
Null 0.353649 0.003448
值和指针版本的差别很大(0.077s是标准差的很多倍),指针版本更快。所以使用传统的基于指针的比较器版本。 'Null' 时间使用一个比较器函数,该函数只返回 0 而不进行任何比较。
代表输出行:
Value: 0.730634 (less = 51517909, more = 48482090, equl = 1)
Pointer: 0.684107 (less = 51517909, more = 48482090, equl = 1)
Null: 0.351807 (less = 0, more = 0, equl = 100000000)
测试代码
两个比较器被重命名为compare_digests_val() 用于比较值,compare_digests_ptr() 用于比较指针。 Clock 类型和clk_* 函数是在我测试的平台上使用gettimeofday() 的高分辨率定时器包。显然,循环中的增量和统计数据的累积有相当大的开销,但这只是意味着比较器中的差异更显着。
static int compare_digests_nul(const void *a, const void *b)
{
return 0;
}
static void time_comparisons(const char *tag, int (*compare)(const void *, const void *))
{
struct digest_line a = { 0, 0 };
struct digest_line b = { 0, 0 };
int less = 0;
int more = 0;
int equl = 0;
Clock clk;
char buffer[32];
clk_init(&clk);
clk_start(&clk);
for (int i = 0; i < 100000000; i++)
{
int j = (*compare)(&a, &b);
if (j < 0)
less++;
else if (j > 0)
more++;
else
equl++;
a.first += 1234567890123ULL;
a.second += 2345678901234ULL;
b.first += 7654321098765ULL;
b.second += 8765432109876ULL;
}
clk_stop(&clk);
printf("%-8s %s (less = %9d, more = %9d, equl = %9d)\n", tag,
clk_elapsed_us(&clk, buffer, sizeof(buffer)),
less, more, equl);
}
int main(void)
{
for (int i = 0; i < 20; i++)
{
time_comparisons("Value:", compare_digests_val);
time_comparisons("Pointer:", compare_digests_ptr);
time_comparisons("Null:", compare_digests_nul);
}
return 0;
}