Simple request about core::map

[teto]

Hi,

When iterating through a core::map, there are some items I need to check before deleting them and I can't find any simple way to delete a Node* or an Iterator from map. remove only accepts a key which means I will look twice for the same item as I already have the correct Node*.

Code: Select all

 
    TJoystickMap::Iterator itr = Joysticks.getIterator();
 
    for( ;!itr.atEnd(); ++itr)
    {
        TJoystickMap::Node* n = itr.getNode();
        CJoystickDirectInput* joystick = n->getValue();
 
        // Copy Save state
        const SJoystickState savedState = joystick->getState();
        if( joystick->update() )
        {
 
            // generate events
            IInputManager::compareStates(savedState, joystick);
        }
        else
        {
            sendJoystickEvent(EJE_UNPLUGGED, joystick );
            Joysticks.remove ( n->getKey() );         //!< Here is the "problem", I don't need to search the node again. I wish I could do: Joysticks.remove(n);
        }
 
    }

This is very simple to do and effective.

Matt

[hendu]

Either way, I don't think you should remove nodes while iterating. Removal triggers rebalancing, how's the iterator supposed to keep track of things?

[teto]

In the std, you retrieve a valid iterator upon deletion. Also std::map doesn't move values.

[aanderse]

In the std

it sounds like you've already found a suitable alternative then!
excellent

[hybrid]

Added in SVN/trunk.

[teto]

In the std

it sounds like you've already found a suitable alternative then!
excellent

A bit low, don't you think ?

Added in SVN/trunk.

Thank you !

[Brkopac]

In the std

it sounds like you've already found a suitable alternative then!
excellent

QFT * OVER 9000