g ++错误:重载'abs(unsigned int)'的调用是不明确的

时间:2018-05-11 23:27:32

标签: c++ compiler-errors g++ std

我正在尝试编译以下代码:

#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

3 个答案:

答案 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;
}