Thread: for_each: callback function

Originally Posted by Codeplug

You can't use class member function as a for_each functor.

Should be able to do it using mem_fun_ref. I've done it before using different compilers MSVC++ 6.0 and MSVC .NET. Depending on the compiler I've used you need to do different things to get it to work. Under MSVC++ 6.0, the function being called needed to return a value and be non-const, i.e.:

Code:

#include <functional>
#include <algorithm>
#include <iostream>
#include <list>

class MyClass
{
public:
    MyClass();
    ~MyClass();
    std::list<int> l;
    int Print(int i);
};

MyClass::MyClass()
{
    for (int a=0;a<11;++a)
        l.push_back(a);
    std::for_each(l.begin(),l.end(),std::mem_fun_ref(MyClass::Print));
}

MyClass::~MyClass()
{
}

int MyClass::Print(int i)
{
    std::cout<<i<<std::endl;
    return 0;
}

int main(int argc,char* argv[])
{
    MyClass mc;
    return 0;
}

However, MSVC .NET wanted the called function to be a type void, const function:

Code:

#include <functional>
#include <algorithm>
#include <iostream>
#include <list>

class MyClass
{
public:
    MyClass();
    ~MyClass();
    std::list<int> l;
    void Print(int i) const;
};

MyClass::MyClass()
{
    for (int a=0;a<11;++a)
        l.push_back(a);
    std::for_each(l.begin(),l.end(),std::mem_fun_ref(MyClass::Print));
}

MyClass::~MyClass()
{
}

void MyClass::Print(int i) const
{
    std::cout<<i<<std::endl;
}

int main(int argc,char* argv[])
{
    MyClass mc;
    return 0;
}

[edit]Added #include <algorithm> for the for_each function.[/edit]

[edit2]I just realized that this won't work because you are expecting an arugment to be passed to the Print function. You can pass an argument to the function also by using bind2nd as in:

Code:

    std::for_each(l.begin(),l.end(),std::bind2nd(std::mem_fun_ref(MyClass::Print),1));

But as you see, you need a specific value to give as the second parameter, you can't really use what you were trying to print out the members of the list. You could however replace the for_each call with a call to copy and just to this:

Code:

std::copy(l.begin(),l.end(),std::ostream_iterator<int>(cout,"\n"))

[/edit2]

Well, no version of MSVC, or any compiler for that matter, will compile that mem_fun_ref<> code.

std::mem_fun_ref<> is usefull if you want to call member functions of objects within the container.

>> std::for_each(l.begin(),l.end(),std::mem_fun_ref(M yClass::Print));
This says, "call the MyClass member function Print(), for each of the MyClass instances within l".
The compiler will have a hard time doing that since "l" contains int's and not MyClass's.

I like what you're thinking, but I don't know of anything in std:: that will do it.

We can certainly do it ourselves though.....chew on this:

Code:

#include <functional>
#include <algorithm>
#include <iostream>
#include <list>
using namespace std;

// This class represents an adaptor for calling a class member function on 
// a particular class instance as if it were simply a unary functor with void 
// return type.
template<class mem_fun_t,  // member function pointer type
         class instance_t, // class type containing member function
         class param_t>    // parameter type of member function
struct call_mem_fun_t : public unary_function<param_t, void> 
{
    // constructor
    //  pmf - pointer to member function
    //  inst - instance to use when calling member function
    explicit call_mem_fun_t(mem_fun_t pmf, instance_t *inst)
        : m_pmf(pmf), m_inst(inst) {}

    // functor
    void operator()(param_t &val) const
    {
        (m_inst->*m_pmf)(val); 
    }// operator()

private:
    mem_fun_t m_pmf;    // pointer to member function
    instance_t *m_inst; // pointer to instance in order to call member function
};//call_mem_fun_t


// This is a utility template function for constructing and returning a 
// call_mem_fun_t<> object. The actual "utility" is that the compiler will 
// deduce template arguments for us.
// Note that the compiler has a hard time deducing param_t only based on 
// the mem_fun_t type. So it is the first template parameter and should be
// specified.
template<class param_t,    // param_t goes first since compiler can't figure it out
         class mem_fun_t,  // member function pointer type
         class instance_t> // class type containing member function
inline
call_mem_fun_t<mem_fun_t, instance_t, param_t> // return type
call_mem_fun(mem_fun_t pmf, instance_t *inst)
{
    return call_mem_fun_t<mem_fun_t, instance_t, param_t>(pmf, inst);
}//call_mem_fun


// Test class for our adaptors
class MyClass
{
    list<int> m_ilist;

public:
    MyClass()
    {
        for (int n = 0; n < 10; n++)
            m_ilist.push_back(n);

        // use call_mem_fun_t directly
        for_each(m_ilist.begin(), m_ilist.end(),
                 call_mem_fun_t<void (MyClass::*)(int), // member function pointer type
                                MyClass,                // class type containing member function
                                int>                    // parameter type of member function
                               (&MyClass::Print, this));// call Print() using 'this'
        
        cout << "--------" << endl;

        // now use the "utility" function
        for_each(m_ilist.begin(), m_ilist.end(), 
                 call_mem_fun<int>(&MyClass::Print, this)); // I can read this!!
    }//constructor

    void Print(int i)
    {
        cout << i << endl;
    }//Print
};//MyClass


int main()
{
    MyClass mc;

    return 0;
}//main

gg