我正在尝试编译以下代码:
#include <stdlib.h>
static
unsigned iSqr( unsigned i )
{
unsigned res1 = 2;
unsigned res2 = i/res1;
while( abs( res1 - res2 ) > 1 )
{
res1 = (res1 + res2)/2;
res2 = i/res1;
}
return res1 < res2 ? res1 : res2;
}
使用g++ test.cc -o test
。
但是,g ++编译器失败并出现以下错误:
test.cc: In function 'unsigned int iSqr(unsigned int)':
test.cc:8:29: error: call of overloaded 'abs(unsigned int)' is ambiguous
while( abs( res1 - res2 ) > 1 )
^
In file included from /usr/include/c++/6/cstdlib:75:0,
from /usr/include/c++/6/stdlib.h:36,
from test.cc:2:
/usr/include/stdlib.h:735:12: note: candidate: int abs(int)
extern int abs (int __x) __THROW __attribute__ ((__const__)) __wur;
^~~
In file included from /usr/include/c++/6/stdlib.h:36:0,
from test.cc:2:
/usr/include/c++/6/cstdlib:185:3: note: candidate: __int128 std::abs(__int128)
abs(__GLIBCXX_TYPE_INT_N_0 __x) { return __x >= 0 ? __x : -__x; }
^~~
/usr/include/c++/6/cstdlib:180:3: note: candidate: long long int std::abs(long long int)
abs(long long __x) { return __builtin_llabs (__x); }
^~~
/usr/include/c++/6/cstdlib:172:3: note: candidate: long int std::abs(long int)
abs(long __i) { return __builtin_labs(__i); }
^~~
为什么会出现此错误以及如何解决?
g ++版本是:gcc版本6.3.0
答案 0 :(得分:3)
编辑:<cstdlib>
应该处理重载(并且应该首选),请参阅问题here
请注意,此处存在一些逻辑错误。首先,你所做的可能不是你想要的。 res1 - res2
是两种unsigned
类型之间的算术运算,因此结果也将是unsigned
。如果你低于0,你回到max int。 abs()
在这里毫无意义,该值永远不会为负,因为该类型不允许它。即使使用编译此代码的标头,编译器仍会对此发出警告。
我强烈建议您在处理算术时坚持使用有符号整数来避免这些陷阱,如果确实需要存储,请使用64位整数。
所以我的答案是,将unsigned
切换为int
另请注意:unsigned res2 = i / res1;
res2将在此处截断为0,我不确定这是否是您想要的。
答案 1 :(得分:2)
abs
中的 <stdlib.h>
(自c ++ 11开始):http://www.cplusplus.com/reference/cstdlib/abs/
int abs ( int n);
long int abs ( long int n);
long long int abs (long long int n);
如果超载,则呼叫将不明确 分辨率无法选择与此类无差别功能相比唯一更好的呼叫匹配。
您可以明确地转换参数:
static_cast<int>(res1 - res2)
答案 2 :(得分:1)
您可以将结果转换为abs的已签名数据类型,以便知道您正在调用哪一个:
while( abs( int(res1 - res2) ) > 1 )
但重新考虑你需要做什么,因为无符号上的减运算的结果仍然是无符号的,所以如果结果低于0,它将变成一个很大的数字(作为正常的溢出),所以我认为使用带符号变量是首先解决问题的最佳方法
int iSqr( unsigned i )
{
int res1 = 2;
int res2 = i/res1;
while( abs(res1 - res2) > 1 )
{
res1 = (res1 + res2)/2;
res2 = i/res1;
}
return res1 < res2 ? res1 : res2;
}