Proper way to remove items from a QListWidget[Solved]

musimbate

Thanks,SGaist,I really have had a long day!!!

musimbate

Sorry to bother again.This should solve the problem right?
@

int numberOfElements=mListWidget->count();

 for(int i=2;i<numberOfElements;i++)
  mListWidget->takeItem(i);

@

but I am still getting the same results as before.What is going on .....??

dbzhang800

No, your usage is still wrong.

You should always remove the second item, until there is no second item any more, which mean you have only one item now!

or you should remove your item from the last to the second item.

SGaist

To add to 1+1=2, your list size still changes, so your counter is going to grow past the size of your list

andre

General trick in deleting items from a list while iteration over that list, is to iterate backwards. That will make your life much easier...

john_god

Does the takeItem() take care of freeing the memory allocate with new operator ?

SGaist

It's explained in the "documentation":http://qt-project.org/doc/qt-4.8/qlistwidget.html#takeItem

musimbate

Thank you all guys ,was ignoring that the list changed sizes.Its OK now.

? Offline

If you call takeItem you still need to call delete on it, a better way is to just loop over the items and call delete on them without calling takeItem, it works ok because the dtor will remove itself from the ListWidget during destruction.

musimbate

Thanks pwnstar23,

so what you mean is something like this right?
@

//count is the number of items to delete
for(int i=0;i<count;i++)
delete(mListWidget->takeItem(indexLocal+1));

@

It works wonders.

SGaist

Rather

@
while (!mListWidget.isEmpty()) {
QListWidgetItem *item ' mListWidget->takeItem();
delete item;
}
@