使用for_each / mem_fun有什么問題

Question

我正在使用C ++ 98。

我有以下代碼：

#include <vector>
#include <algorithm>
#include <functional>

struct Foo {};

void
add_each(std::vector<std::vector<Foo*> > &vv, const Foo *f)
{
    for_each(vv.begin(), vv.end(), std::bind2nd(std::mem_fun(&std::vector<Foo*>::push_back), f));
}

基本上，我喜歡在每個向量的末尾添加一個元素。

G ++抱怨for_each行。

In file included from /usr/lib/gcc/x86_64-redhat-linux/4.4.7/../../../../include/c++/4.4.7/algorithm:62,
                 from 1.c:2:
/usr/lib/gcc/x86_64-redhat-linux/4.4.7/../../../../include/c++/4.4.7/bits/stl_algo.h: In function ‘_Funct std::for_each(_IIter, _IIter, _Funct) [with _IIter = __gnu_cxx::__normal_iterator<std::vector<Foo*, std::allocator<Foo*> >*, std::vector<std::vector<Foo*, std::allocator<Foo*> >, std::allocator<std::vector<Foo*, std::allocator<Foo*> > > > >, _Funct = std::binder2nd<std::mem_fun1_t<void, std::vector<Foo*, std::allocator<Foo*> >, Foo* const&> >]’:
1.c:10:   instantiated from here
/usr/lib/gcc/x86_64-redhat-linux/4.4.7/../../../../include/c++/4.4.7/bits/stl_algo.h:4200: error: no match for call to ‘(std::binder2nd<std::mem_fun1_t<void, std::vector<Foo*, std::allocator<Foo*> >, Foo* const&> >) (std::vector<Foo*, std::allocator<Foo*> >&)’
/usr/lib/gcc/x86_64-redhat-linux/4.4.7/../../../../include/c++/4.4.7/backward/binders.h:146: note: candidates are: typename _Operation::result_type std::binder2nd<_Operation>::operator()(const typename _Operation::first_argument_type&) const [with _Operation = std::mem_fun1_t<void, std::vector<Foo*, std::allocator<Foo*> >, Foo* const&>]
/usr/lib/gcc/x86_64-redhat-linux/4.4.7/../../../../include/c++/4.4.7/backward/binders.h:152: note:                 typename _Operation::result_type std::binder2nd<_Operation>::operator()(typename _Operation::first_argument_type&) const [with _Operation = std::mem_fun1_t<void, std::vector<Foo*, std::allocator<Foo*> >, Foo* const&>]

看到什么地方了嗎？

Answer 1

for_each將取消引用迭代器，並將結果傳遞給其一元函數參數。 在您的情況下，該類型為std::vector<Foo*>& 。 但是， std::mem_fun需要指向包裝對象的指針 ，而不是引用 。 您應該改用std::mem_fun_ref 。

void
add_each(std::vector<std::vector<Foo*> > &vv, const Foo *f)
{
    for_each(vv.begin(), vv.end(), 
             std::bind2nd(std::mem_fun_ref(&std::vector<Foo*>::push_back), f));
}

現場例子

Answer 2

您正在嘗試push_back一個const Foo*為向量Foo* 。 使向量保持const Foo*或將參數f的類型更改為Foo* 。

Answer 3

如果您希望您的代碼也可以使用C ++ 2011的標准庫進行編譯，則建議采用以下解決方案

void ( std::vector<Foo *>:: *pf )( Foo * const &);
pf = &std::vector<Foo *>::push_back;

std::for_each( vv.begin(), vv.end(), 
               std::bind2nd( std::mem_fun_ref( pf ), f ) );

問題在於，在C ++ 2011成員函數中push_back被重載。