Здравейте,
имам въпрос отностно итератора на листа - защо не се инкрементира ?
for(std::list<Company*>::iterator it = companies.begin(); it != companies.end(); )
{
if (условие)
{
it=companies.erase(it);
}
else{
it++;
}
}
break;
Че бъркам някъде, бъркам, но с часове се боря вече и незнам.
Здравей,
На мен ми изглежда добре тази част от кода, сигурен ли си, че точно тук не се инкрементира? Този break долу какво break-ва? Предполагам има външен цикъл, а ако правиш два вложени цикъла и във вътрешния триеш елементи, които външния обхожда - това не е особено безопасно и най-вероятно ще инвалидираш външния цикъл, защото той работи със стари итератори. Изобщо за тази задача не ти трябва втори цикъл, един цикъл е достатъчно за да обходиш елементите и да махнеш всеки излишен - не е нужно за всеки елемент да обхождаш всички останали, което би станало ако имаш вложен цикъл. Когато махаш развалени ябълки от кофа с ябълки, просто вземаш една по една ябълките и ако някоя се окаже развалена, я махаш - няма нужда да погледнеш всички останали ябълки преди да махнеш една конкретна.
Направи си извеждане на всяка стъпка и виж какво точно се случва - струва ми се, че проблемът ти е друг, не е в инкементирането на итератора.
Поздрави,
Жоро
Да наистина, проблема беше другаде. Сега работи, но не изпълнявам изискванията за memory limit, но нищо.
thanks Жорка
Да, текущото ти решение е вярно, но имаш memory leak (затова и не ти стига паметта) - помисли къде е leak-ът.