итераторы и vector.erase(

[Teksa]

Здравствуйте, есть вот такой вот фрагмент кода:

// serv->clients определен как vector< CVoIPServClient *>
for( vector< CVoIPServClient *>::iterator  l = serv->clients.begin(); l != serv->clients.end(); l++)
		    {
			
				if (!(*l)->GetState()) // отдает информацию об активности клиента
				dataset.push_back( (*l)->GetData()); //некая полезная работа
				else //если состояние нам не нравится - удаляем клиент
				{
					printf("Client count: %d \n", serv->clients.size());
					printf("delete client...ID  %d \n ",(*l)->GetID());
					delete (*l);
					serv->clients.erase(l); // удаляем из списка указатель
					printf("Client count: %d", serv->clients.size());
				}
			}

где serv->clients - коллекция подключенных клиентов, имеющих поле состояния и доступ к нему через GetState как только состояние изменяется клиента необходимо удалить ( delete (*l); ). Так же необходимо исключить "мертвую душу" из serv->clients. И вот тут начинаются проблемы: для удаления элемента вектора вызываем erase(текущий итератор), удаляется... но попытка инкремента итератора вызывает ошибку , то есть итератор не желает переходить на следующйи элемент, пробовал после erase присваиваить итератору begin(), не помогло, все равно дохнет ... с STL только знакомлюсь, поэтому кажется собираю все возможные грабли...

[ozo]

Нужны новые итераторы, ибо при удалении элемента вектор может перестроится.
PS на сколько помню, erase( iterator ) возвращает итератор на следующий элемент

[Teksa]

Спасибо, пробовал: l=serv->clients.erase(l); - толку ноль.
break цикла после удаления - спасает, но нежелателен...

[ozo]

delete (*l);
serv->clients.erase(l); // удаляем из списка указатель

если после этого сделать:
l = serv->clients.begin();
тоже не работает ? Странно всё это =) На вашем месте я бы еще убрал строчку delete (*l); на время, а то мало ли )

[Teksa]

пробовал убирать , не влияет В итоге модифицировал схемку с break цикла в конце, а то в мелочах завязну. l = serv->clients.begin() - попробовал до того как на форуме отписался, все равно на l++ слетает спасибо за помощь.

[ozo]

std::vector<Quastion>::iterator i = quastions->begin();
while(i != quastions->end() ){
  ...;
  if( некое условие ){
    quastions->erase(i);
    ...;
    i = quastions->begin();
  } else 
	++i;
}

Я где-то так делал, вроде бы работает исправно