Помощь - Поиск - Пользователи - Календарь
Полная версия: Список. Удаление
Форум «Всё о Паскале» > Современный Паскаль и другие языки > Ада и другие языки
Fanat
Список содержит указатели на элементы.
Необходимо удалить i-ый элемент списка, при этом удалить и сам элемент.
Я делаю так

it += Index;
delete *it;
m_List.erase(it);


Всё работает если Index = 0, или если нет строки delete *it.
А если удалить элемент с индексом больше 0 (меньше размера списка), то при следующем проходе по списку
вылетает ошибка по доступу к памяти...как правильно сделать?..
volvo
Если список - это std::list, то достаточно одного erase(it)... delete здесь ни при чем совершенно.
Fanat
Цитата(volvo @ 23.01.2008 13:31) *

Если список - это std::list, то достаточно одного erase(it)... delete здесь ни при чем совершенно.


Да это std::list, но вылетают потери памяти если удалять просто erase(it)...а если с delete *it то нет...
Michael_Rybak
Цитата
Если список - это std::list, то достаточно одного erase(it)... delete здесь ни при чем совершенно.


volvo, там не объекты, а указатели, Fanat хочет и объект удалить, и указатель на него.

Fanat, попробуй заменить it += index циклом. т.е. пробегай по списку вручную до нужного элемента.
Fanat
Цитата(Michael_Rybak @ 23.01.2008 13:52) *

Fanat, попробуй заменить it += index циклом. т.е. пробегай по списку вручную до нужного элемента.


Так на самом деле работает...но был перегружен оператор + таким же самым образом...
Тут...
Ну и пишу я соответственно не += ,а в нужном месте (it + Index).
volvo
Fanat, это твой тип, или что-то готовое (я про то, на что указывает указатель)?
Fanat
Цитата(volvo @ 23.01.2008 14:20) *

Fanat, это твой тип, или что-то готовое (я про то, на что указывает указатель)?


Мой тип.
Michael_Rybak
Цитата
Так на самом деле работает...но был перегружен оператор + таким же самым образом...

Тогда тести для index = 1, сравнивай с перегрузкой и без (одновременно), смотри адреса и выясняй, где расхождение.
volvo
Цитата
Мой тип.
Можно посмотреть, как заполняется список?
Fanat
Цитата(volvo @ 23.01.2008 16:24) *

Можно посмотреть, как заполняется список?



int __fastcall TTrfObjectList::add(ILXListItem* aItem)
{
m_List.push_back(aItem);
return m_List.size();
};




for(int i = 0; i < c_nNumberOfStartElems; i++)
{
ILXListItem* pItem = new TTrfListItem(i*10*pow(-1,i));
m_pList->add(pItem);
}



Вот так он заполняется...этого хватит или ещё чтото выложить?..хотя я уже решил везде заменить перегруженную операцию на пробег по итератору в цикле, так то она работает(далко только перегрузку конечно...=()...так что можно не искать ошибку...
volvo
Fanat, я не совсем понимаю, откуда у тебя берутся leak-и... Вот смотри:

class TMyList: public list<int*>
{
friend
TMyList::iterator&
operator + (TMyList::iterator &it, int index) {
for(; index; --index) ++it;
return it;
}

public:
void erase_element(int index) {
TMyList::iterator it = begin();
delete *(it = (it + index));
erase(it);
}

~TMyList() {

for(TMyList::iterator
it = begin(); it != end(); delete *it++) {
}
}
};

void __fastcall TForm1::Button2Click(TObject *Sender)
{
TMyList myList;

for(int i = 0; i < 20; i++) {
int *p = new int(i);
myList.push_back(p);
}

myList.erase_element(5);

for(TMyList::iterator it = myList.begin();
it != myList.end(); it++)
{
Memo2->Lines->Add(IntToStr(*(*it)));
}
}

Код аналогичен твоему, правда? Ну, за исключением некоторых мелочей... Только почему-то у меня CodeGuard молчит... Попробуй у себя прогнать это, будет ругаться?
Fanat
Цитата(volvo @ 23.01.2008 17:47) *

Ну, за исключением некоторых мелочей...


У себя прогнал...В общем мелочью оказалась следующее:
Я писал

delete *(it + aIndex);
m_List.erase(it + aIndex);



А ты

delete *(it = (it + aIndex));
m_List.erase(it);



Вот в твоём варианте всё работает хорошо...а в моём даже до ликов не доходит...
ошибка вылетает...
Это текстовая версия — только основной контент. Для просмотра полной версии этой страницы, пожалуйста, нажмите сюда.