常见而又隐秘的比较函数 BUG


// Returns 0 if a == b, a negative number if a < b or a positive otherwise.

int cmp(const void *a, const void*b) {
	return *(int *)a  - *(int *)b;  
}


这是一个非常常见地使用在 qsort 等排序里的比较函数,然后这个函数本身却是存在 bug 的 —— 溢出的危险。

FAILURE TEST CASE:

</pre><p></p><pre name="code" class="cpp">int a = INT_MIN, b = 1;  // INT_MIN defined in limits.h
int result = cmp(&a, &b);  // result = INT_MAX, 这显然与 a < b 的事实不符合。

那么正确的写法是怎样的呢:

// Returns 0 if a <= b, or non-zero value otherwise.
int cmp(const void *a, const void*b) {
	return *(int *)a > *(int *)b;
}

据说《程序设计实践》一书有提到,另此书评价甚高,有空可以拜读一下。


评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值