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::it erator itr = observers_.begi n();
while(itr != observers_.end( )){
if(itr->second == observer){
ObserverMap::it erator tmpItr = ++itr;
--itr;
observers_.eras e(itr);
itr = tmpItr;
continue;
}

++itr;
}

- minglei

[Victor Bazarov]

ke****@gmail.co m wrote:

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::it erator itr = observers_.begi n();
while(itr != observers_.end( )){
if(itr->second == observer){
ObserverMap::it erator tmpItr = ++itr;
--itr;
observers_.eras e(itr);
itr = tmpItr;

OK, maybe I'm stating the obvious here, but the above FOUR lines
can be rewritten as just one:

observers_.eras e(itr++);

continue;

Drop the 'continue' and add 'else' below.

}

add
else

here.

>
++itr;
}

- minglei

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

ObserverMap::it erator itr = observers_.begi n();
while (itr != observers_.end( )) {
if (itr->second == observer)
observers_.eras e(itr++);
else
++itr;
}

How's that?

V
--
Please remove capital 'A's when replying by e-mail
I do not respond to top-posted replies, please don't ask

[moonrie]

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

ObserverMap::it erator itr = observers_.begi n();
while (itr != observers_.end( )) {
if (itr->second == observer)
observers_.eras e(itr++);
else
++itr;
}

How's that?

V

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

for(ObserverMap ::iterator itr = observers_.begi n(); itr !=
observers_.end( ); )
if (itr->second == observer)
observers_.eras e(itr++);
else
++itr;

- minglei

[Frank Birbacher]

Hi!

Victor Bazarov schrieb:

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

ObserverMap::it erator itr = observers_.begi n();
while (itr != observers_.end( )) {
if (itr->second == observer)
observers_.eras e(itr++);
else
++itr;
}

Now we consolidate the increment ++itr:

ObserverMap::it erator itr = observers_.begi n();
while (itr != observers_.end( )) {
const ObserverMap::it erator current = itr++;
if (current->second == observer)
observers_.eras e(current);
}

This is less confusing to me.

Regards, Frank