[prog.c++] Еще в качестве рекламы современного C++ :)

Совсем маленький пример того, как современный C++ защищает от элементарных ошибок разработчика.

Простая задачка -- пройтись по небольшому std::vector и удалить из него все элементы, которые удовлетворяют конкретному условию.

Начал было писать так:

for( auto it = m_events.begin(); it != m_events.end(); ++it )    if( mbox_id == it->m_mbox->id() && msg_type == it->m_msg_type )       it = m_events.erase( it );

Но потом поймал себя на мысли о том, что таки XXI-вый век на дворе, надо бы заюзать функции из <algorithm> и лямбду в качестве предиката. Написал:

m_events.erase(       remove_if( begin( m_events ), end( m_events ),             [mbox_id, &msg_type]( const info_t & i ) {                return i.m_mbox->id() == mbox_id && i.m_msg_type == msg_type; } ),       end( m_events ) );

Подумалось, что XXI-ый век -- это хорошо, но писать больше кода...

А потом обратил внимание, что в варианте со старым-добрым for есть баг. Правильная версия должна была бы выглядеть так:

for( auto it = m_events.begin(); it != m_events.end(); )    if( mbox_id == it->m_mbox->id() && msg_type == it->m_msg_type )       it = m_events.erase( it );    else       ++it;

А это уже совсем другой расклад. Чуть-чуть больший по объему код, в котором трудно допустить ошибку, все-таки получше, чем более короткий, в котором ошибка есть, да еще и не сразу ее заметишь.