vector & remove_if

[mickey]

sorry,
what I meant was:
This seems work:

vector<double>::iterator ivalid = validValue.end() - 1;
vector<bool>::iterator imask = _indexMissedMask.end() - 1;
 for ( ; imask != _indexMissedMask.begin(); --imask) { 
if ( *imask ) 
     validValue.erase(ivalid--);
}

vector<double>::iterator ivalid = validValue.end() - 1;
vector<bool>::iterator imask = _indexMissedMask.end() - 1;
 for ( ; imask != _indexMissedMask.begin(); --imask) { 
if ( *imask ) 
     validValue.erase(ivalid--);
}

To copy to clipboard, switch view to plain text mode 

This crashes:

vector<double>::iterator ivalid = validValue.end() - 1;
vector<bool>::iterator imask = _indexMissedMask.end() - 1;
 for ( ; imask != _indexMissedMask.begin(); --imask, --ivalid) {
if ( *imask ) 
     validValue.erase(ivalid);
}

vector<double>::iterator ivalid = validValue.end() - 1;
vector<bool>::iterator imask = _indexMissedMask.end() - 1;
 for ( ; imask != _indexMissedMask.begin(); --imask, --ivalid) {
if ( *imask ) 
     validValue.erase(ivalid);
}

To copy to clipboard, switch view to plain text mode 

Why, please?

[caduel]

The code above is wrong (twice decrementing of iterator).
The code below is wrong, too: after erase() the iterator ivalid is no longer valid.
You would have to do something like

vector<double>::iterator ivalid = validValue.end() - 1;
vector<bool>::iterator imask = _indexMissedMask.end() - 1;
for ( ; imask != _indexMissedMask.begin(); --imask)
{
   vector<double>::iterator tmp = ivalid;
   --tmp;
   if ( *imask )
      validValue.erase(ivalid); // invalidates ivalid; would return iterator pointing behind it
   ivalid=tmp;
}

vector<double>::iterator ivalid = validValue.end() - 1;
vector<bool>::iterator imask = _indexMissedMask.end() - 1;
for ( ; imask != _indexMissedMask.begin(); --imask)
{
   vector<double>::iterator tmp = ivalid;
   --tmp;
   if ( *imask )
      validValue.erase(ivalid); // invalidates ivalid; would return iterator pointing behind it
   ivalid=tmp;
}

To copy to clipboard, switch view to plain text mode 

(untested, of course)

This is why you should not reinvent the wheel yourself. Use the algorithms.