如何选择正确的功能过载?

时间:2016-04-22 12:10:02

标签: c++ c++11 overloading stl-algorithm template-function

在下列情况下选择正确的函数重载的正确方法是什么?

#include <iostream>
#include <algorithm>

/** the correct overload **/
bool predicate( const char& c )
{
    return c == '0';
}

/** the wrong overload **/
template< typename CharType >    
bool predicate( const CharType& c, int some_other_parameters )
{
    return c == '0';
}

std::string
process_string( const std::string& str )
{
    std::string result;
    std::copy_if( str.begin( ),
                  str.end( ),
                  std::back_inserter( result ),
                  predicate );

    return result;
}

int main()
{
    std::cout << process_string("AK0NNDK0ASDAS0") << std::endl;
    return 0;
}

1 个答案:

答案 0 :(得分:3)

您可以使用lambda来解决谓词歧义;

std::string
process_string( const std::string& str )
{
    std::string result;
    std::copy_if( str.begin( ),
                  str.end( ),
                  std::back_inserter( result ),
                  [](char const& c) { return predicate(c); } );
    //            ^^ use the lambda to call the correct overload

    return result;
}

同样重要的是要记住,非模板将优先于模板功能。

或者,您可以转换函数指针(但我认为这更麻烦);

std::copy_if( str.begin( ),
              str.end( ),
              std::back_inserter( result ),
              static_cast<bool(*)(const char&)>(&predicate) );

Demo

指针强制转换包括键入函数指针类型,然后获取指向所需函数的局部变量;

using predicate_t = bool(*)(const char&);
predicate_t my_predicate = &predicate;
std::copy_if( str.begin( ),
              str.end( ),
              std::back_inserter( result ),
              my_predicate );

Demo

关于哪个选项更好,它取决于样本之外的代码的复杂性,它们的位置(即代码与第三方代码),模糊错误的数量,谓词本身。

鉴于OP代码中的简单条件,lambda可以包含测试本身。在这种情况下,lambda非常简单。

如果计数很高,则更高范围的using版本(使用指针转换的局部变量)可能是合适的。

如果是“一次性”问题,static_cast也会很好。虽然演员看起来“不合时宜”。

最终,它可能最受个人偏好的影响(或指导,如果你有任何涵盖这种情况)。

lambda还可以与auto&&和一些可变参数列表as seen in this answer in the linked question的更现代的使用相结合。值得注意的是,这些现代技术相得益彰。在这种情况下,大多数现代编译器也乐观地使用lambda,因此使用它没有任何成本(这适用于所有选项,只是解决了歧义)。

理想情况下,开始时应该没有真正的含糊,但它发生了,我们需要在我们找到的上下文中以最恰当的方式处理它。