- 1
- 2
- 3
- 4
- 5
- 6
- 7
- 8
vector<int> :: iterator itBegins = array.begin();
vector<int> :: iterator itEnds = array.end();
while(itBegins != itEnds)
{
pVec.pop_back();
it++;
}-
C++ / Говнокод #2092
+69.8
Сей артефакт остался в коде ввиду лености переписывать его в 100 файлах сразу после удаления того, ради чего собственно затевался while(!pVec.empty()) ... прямо скажем - отлов ЭТОГО занял немало времени и нервов... тк на первый взгляд оно должно... а что ты видишь в этом коде, %юзернейм% ?
(исправленная версия XD )Запостил:
Zeks,
02 Ноября 2009
Follow us!
И ещё тут написано, что выбивают элемент из pVec, а итераторы на какой-то array...
vector сложный контейнер, думаю он легко отследит, что pop_back выполняется на пустом векторе, и просто будет в цикле крутиться до морковкиного заговенья.
:)))
"Ну ничего. Я готов ждать Сяо-Феня хоть до морковкина заговенья" (с) =]
Просто, стандарт предоставляет относительную "вольность". Например стандарт не определяет размер буфера для vector и даже не определяет, что этот буфер должен быть. А valarray может предполагать, что объект, в нём лежащий, всегда можно побитово скопировать. Но если valarray поддерживает глубокое копирование и не только POD объекты, то это не делает stl не стандартной.
Я, пока лазил и искал что-то по vector.clear(), нашёл, что pop_back() ничего никому не обязан... И на пустом векторе может произойти что угодно... Так что крутилка эта может иметь ещё и последствия...
Тем более на Ваш случай можно воспользоваться, к примеру, sed-ом :)
Про sed знаю - этот кусок кода активно генерится с помощью него в том числе, но это надо было выгружаться в линукс тк програмлю я на работе под виндой... :) Вот в результате лености лишний раз перезагрузиться и применения "Заменить в файлах" вместо требовавшегося перегенерирования cpp`шников и появился сей артефакт)
И действительно по идее всё удаляется.
Мы же не разименовываем итераторы, потому указывающие не известно куда итераторы, которые, вероятно, нигде больше не используются, не страшны.
После pVec.pop_back() a.end() не равняется itEnds. Это и называется invalidate iterators.
>я боюсь что я сам запутался
Именно по этому не стоит забивать себе голову итераторами, когда можно проще. =]