Question about using for_each

[Yan]

Hi,

I don't seem to be able to use for_each if it should replace a 'for'
loop in a method (constructor in my case) and inside that 'for' loop a
class member variable is being accessed. The presence of this member
variable prevents me from using a static or global method to be passed
as a third parameter to for_each, and mem_fun doesn't seem to work for
me either as I am not going to execute a method of an iterator but
pass an iterator as a parameter. I am not sure that explanation makes
a lot of sense, so below is a sample code. I would like to be able to
replace the 'for' loop inside C class constructor with a for_each.
Please don't pay attention to what is actually happening with these
two vectors of ints, it's just for illustration purposes. Thanks.

-----------------------------------------------------------

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

using namespace std;

typedef vector<intvecto rInts;
typedef vector<int>::it erator vectorIntsIter;

class C {
public:

C(pair<vectorIn tsIter, vectorIntsItera Pair) {
for (vectorIntsIter iter = aPair.first; iter != aPair.second; +
+iter) {
// do something that involves member variable
// which seems to prevent using a static method
// such as for example this:
ints.push_back( *iter + 10);
}
}

void printAllElement s() {
// here a static method 'print' is successfully used
for_each(ints.b egin(), ints.end(), print);
}

private:
static void print(int number) {
cout << number << endl;
}
private:
vectorInts ints;
};

int main() {
vectorInts ints;

ints.push_back( 0);
ints.push_back( 1);
ints.push_back( 2);
ints.push_back( 3);

pair<vectorInts Iter, vectorIntsItera Pair(ints.begin (),
ints.end());
C c(aPair);
c.printAllEleme nts();

return 0;
}

[int2str]

On Jul 11, 2:24 pm, Yan <yvinogra...@gm ail.comwrote:

Hi,

I don't seem to be able to use for_each if it should replace a 'for'
loop in a method (constructor in my case) and inside that 'for' loop a
class member variable is being accessed. The presence of this member
variable prevents me from using a static or global method to be passed
as a third parameter to for_each, and mem_fun doesn't seem to work for
me either as I am not going to execute a method of an iterator but
pass an iterator as a parameter. I am not sure that explanation makes
a lot of sense, so below is a sample code. I would like to be able to
replace the 'for' loop inside C class constructor with a for_each.
Please don't pay attention to what is actually happening with these
two vectors of ints, it's just for illustration purposes. Thanks.

[snipped original code using for()]

How about something like this:

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

using namespace std;

class push_back_doubl e
{
public:
push_back_doubl e( vector<int& ints ) : p_ints( &ints ) {}

void operator() ( const int & i )
{
p_ints->push_back( i*2 );
}

private:
vector<int*p_in ts;
};

class Test
{
public:
Test( vector<int>::it erator first, vector<int>::it erator last )
{
for_each( first, last, push_back_doubl e(ints));
}

void Dump()
{
copy( ints.begin(), ints.end(),
ostream_iterato r<int>(cout,"\n "));
}

private:
vector<intints;
};

int main()
{
vector<inttest;
test.push_back( 1 );
test.push_back( 2 );
test.push_back( 3 );

Test t( test.begin(), test.end() );
t.Dump();
}

Though I'm pretty sure this could be done cleaner with mem_fun...
I'll try that next.

Cbeers,
Andre

[Barry Ding]

On 7 12 , 5 54 , "int2...@gmail. com" <int2...@gmail. comwrote:

On Jul 11, 2:24 pm, Yan <yvinogra...@gm ail.comwrote:

Hi,

I don't seem to be able to use for_each if it should replace a 'for'
loop in a method (constructor in my case) and inside that 'for' loop a
class member variable is being accessed. The presence of this member
variable prevents me from using a static or global method to be passed
as a third parameter to for_each, and mem_fun doesn't seem to work for
me either as I am not going to execute a method of an iterator but
pass an iterator as a parameter. I am not sure that explanation makes
a lot of sense, so below is a sample code. I would like to be able to
replace the 'for' loop inside C class constructor with a for_each.
Please don't pay attention to what is actually happening with these
two vectors of ints, it's just for illustration purposes. Thanks.

[snipped original code using for()]

How about something like this:

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

using namespace std;

class push_back_doubl e
{
public:
push_back_doubl e( vector<int& ints ) : p_ints( &ints ) {}

void operator() ( const int & i )
{
p_ints->push_back( i*2 );
}

private:
vector<int*p_in ts;

};

class Test
{
public:
Test( vector<int>::it erator first, vector<int>::it erator last )
{
for_each( first, last, push_back_doubl e(ints));
}

void Dump()
{
copy( ints.begin(), ints.end(),
ostream_iterato r<int>(cout,"\n "));
}

private:
vector<intints;

};

int main()
{
vector<inttest;
test.push_back( 1 );
test.push_back( 2 );
test.push_back( 3 );

Test t( test.begin(), test.end() );
t.Dump();

}

Though I'm pretty sure this could be done cleaner with mem_fun...
I'll try that next.

Cbeers,
Andre

the operator() in the functor is always better a const member function

[Greg Herlihy]

On 7/11/07 2:24 PM, in article
11************* *********@r34g2 00...legr oups.com, "Yan"
<yv*********@gm ail.comwrote:

I don't seem to be able to use for_each if it should replace a 'for'
loop in a method (constructor in my case) and inside that 'for' loop a
class member variable is being accessed. The presence of this member
variable prevents me from using a static or global method to be passed
as a third parameter to for_each, and mem_fun doesn't seem to work for
me either as I am not going to execute a method of an iterator but
pass an iterator as a parameter. I am not sure that explanation makes
a lot of sense, so below is a sample code. I would like to be able to
replace the 'for' loop inside C class constructor with a for_each.
Please don't pay attention to what is actually happening with these
two vectors of ints, it's just for illustration purposes. Thanks.

-----------------------------------------------------------

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

using namespace std;

typedef vector<intvecto rInts;
typedef vector<int>::it erator vectorIntsIter;

class C {
public:

C(pair<vectorIn tsIter, vectorIntsItera Pair) {
for (vectorIntsIter iter = aPair.first; iter != aPair.second; +
+iter) {
// do something that involves member variable
// which seems to prevent using a static method
// such as for example this:
ints.push_back( *iter + 10);
}
}

A std::for_each() is not the best choice for implementing the loop in the
constructor. I would use std::copy() instead:

copy( aPair.first, aPair.second, back_inserter(i nts) );

Greg