9

我正在尝试使用 ANSI C++ for_each 语句来迭代并打印标准向量的元素。如果我让 for_each 调用非重载函数,它可以工作,但如果我让它调用重载函数,则会产生编译器错误。

这是一个最小的测试程序,用于显示编译器错误发生的位置:

#include <algorithm>
#include <iostream>
#include <vector>

struct S {
    char c;
    int i;
};
std::vector<S> v;

void print_struct(int idx);
void print_struct(const struct S& s);

// f: a non-overloaded version of the preceding function.
void f(const struct S& s);

int main()
{
    v.push_back((struct S){'a', 1});
    v.push_back((struct S){'b', 2});
    v.push_back((struct S){'c', 3});

    for (unsigned int i = 0; i < v.size(); ++i)
        print_struct(i);

    /* ERROR! */
    std::for_each(v.begin(), v.end(), print_struct);

    /* WORKAROUND: */
    std::for_each(v.begin(), v.end(), f);

    return 0;
}

// print_struct: Print a struct by its index in vector v.
void print_struct(int idx)
{
    std::cout << v[idx].c << ',' << v[idx].i << '\n';
}

// print_struct: Print a struct by reference.
void print_struct(const struct S& s)
{
    std::cout << s.c << ',' << s.i << '\n';
}

// f: a non-overloaded version of the preceding function.
void f(const struct S& s)
{
    std::cout << s.c << ',' << s.i << '\n';
}

我在 openSUSE 12.2 中使用:

g++-4.7 -ansi -Wall for_each.cpp -o for_each

完整的错误信息是:

for_each.cpp: In function ‘int main()’:
for_each.cpp:31:48: error: no matching function for call to ‘for_each(std::vector<S>::iterator, std::vector<S>::iterator, <unresolved overloaded function type>)’
for_each.cpp:31:48: note: candidate is:
In file included from /usr/include/c++/4.7/algorithm:63:0,
                 from for_each.cpp:5:
/usr/include/c++/4.7/bits/stl_algo.h:4436:5: note: template<class _IIter, class _Funct> _Funct std::for_each(_IIter, _IIter, _Funct)
/usr/include/c++/4.7/bits/stl_algo.h:4436:5: note:   template argument deduction/substitution failed:
for_each.cpp:31:48: note:   couldn't deduce template parameter ‘_Funct’

我在 Stack Overflow 或一般网络上都没有看到任何针对此特定错误的搜索结果。任何帮助,将不胜感激。

4

2 回答 2

7

名称指的是重载集。您需要指定所需的重载:

std::for_each(v.begin(), v.end(), (void (&)(S const&)) print_struct);

另一种方法是使用多态可调用函数对象作为助手:

struct PrintStruct
{
    template <typename T> void operator()(T const& v) const 
        { return print_struct(v); }
};

int main()
{
    PrintStruct helper;

    std::vector<S> sv;
    std::vector<int> iv;

    // helper works for both:
    std::for_each(sv.begin(), sv.end(), helper);
    std::for_each(iv.begin(), iv.end(), helper);
于 2012-09-28T10:29:15.210 回答
5

std::for_each声明如下所示:

template<class InputIter, class Func>
void for_each(InputIter first, InputIter last, Func func);

正如你所看到的,它接受你给它的任何东西作为第三个参数。没有限制它必须是某个签名的可调用类型或完全可调用类型。

在处理重载函数时,它们本质上是模棱两可的,除非你给它们一些上下文来选择正确的函数。在对重载函数的调用中,此上下文是您传递的参数。但是,当您需要指针时,不能将参数用作上下文,并且for_each参数也不算作上下文,因为它需要任何东西。

作为函数参数可以是选择正确重载的有效上下文的示例,请参见:

// our overloads
void f(int){}
void f(double){}

typedef void (*funcptr_type)(int);
void g(funcptr_type){}

// ...
g(&f); // will select 'void f(int)' overload, since that's 
       // the only valid one given 'g's parameter

正如你所看到的,你在这里给出了一个清晰的上下文,帮助编译器选择正确的重载,而不是让它模棱两可。std::for_each的参数没有给出这样的上下文,因为它们接受任何东西。

有两种解决方案:

  • 通过手动提供上下文
    • 强制转换为正确的函数指针类型,或
    • 使用正确类型的中间变量并传递它
  • 使用分派给重载函数的非重载函数(就像你对 所做的那样f

请注意,在 C++11 中,您还可以将 lambda 用于第二个选项:

std::for_each(v.begin(), v.end(), [](const S& s){ print_struct(s); });

关于您的代码的一些注释:

  • (struct S){'a', 1}是复合文字而不是标准 C++
  • struct S你在 C++ 中不需要,S就足够了
于 2012-09-28T10:36:18.497 回答