g ++错误：重载'abs（unsigned int）'的调用是不明确的

Question

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

#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

Answer 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;
}

Answer 2

编辑：<cstdlib>应该处理重载（并且应该首选），请参阅问题here

请注意，这里存在一些逻辑错误。首先，你所做的可能不是你想要的。 res1 - res2是两个unsigned类型之间的算术，因此结果也将是unsigned。如果你低于0，你回到max int。 abs()在这里毫无意义，这个值永远不会是负数，因为类型不允许它。即使使用编译此代码的标头，编译器仍会对此发出警告。

我强烈建议你在处理算术时坚持使用有符号整数来避免这些陷阱，如果你真的需要存储，请使用64位整数。

所以我真正的答案是，将unsigned切换到int

另请注意：unsigned res2 = i / res1; res2将在此处截断为0，我不确定这是否是您想要的。

Answer 3

在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)