erase all pairs with a specified value in a std::map

[kernus]

i am looking for a more elegant/standard way to do this simple
operation,

however, for now, i just coded this beast. any comments will be
grateful,

ObserverMap::iterator itr = observers_.begin();
while(itr != observers_.end()){
if(itr->second == observer){
ObserverMap::iterator tmpItr = ++itr;
--itr;
observers_.erase(itr);
itr = tmpItr;
continue;
}

++itr;
}

- minglei

 

[moonrie]

IOW, you should be able to do what you need like so:

ObserverMap::iterator itr = observers_.begin();
while (itr != observers_.end()) {
if (itr->second == observer)
observers_.erase(itr++);
else
++itr;
}

How's that?

V

cool, that's really cool, a little arrange of code.

for(ObserverMap::iterator itr = observers_.begin(); itr !=
observers_.end(); )
if (itr->second == observer)
observers_.erase(itr++);
else
++itr;

- minglei

 

[Frank Birbacher]

Hi!

IOW, you should be able to do what you need like so:

ObserverMap::iterator itr = observers_.begin();
while (itr != observers_.end()) {
if (itr->second == observer)
observers_.erase(itr++);
else
++itr;
}

Now we consolidate the increment ++itr:

ObserverMap::iterator itr = observers_.begin();
while (itr != observers_.end()) {
const ObserverMap::iterator current = itr++;
if (current->second == observer)
observers_.erase(current);
}

This is less confusing to me.

Regards, Frank