Remove column from QGridLayout

[high_flyer]

i use qlayout::removeitem() but it doesn't get removed.

Oh, I see now.
But you "are doing it without doing it".
What I mean is, you ask the layout for the items.
Instead, you must have the pointers to the objects somewhere right, so use these pointers in remoevItem() - don't ask the layout for them.

To do it the way you are doing it, you need to keep track of everything you add and remove to the layout, but why would you want to do that - that is what the layout it for!
Asking the layout for the viewer in a given cell only guarantees you that you get a pointer to a viewer (if any) which is in that cell - but if your bookkeeping is wrong, you are not getting the viewer you expect, or, not any viewer at all.

[enricong]

Oh, I see now.
To do it the way you are doing it, you need to keep track of everything you add and remove to the layout, but why would you want to do that - that is what the layout it for!

I think I see what you mean.
I was was not tracking the pointers after they are added to the layout.
I was depending on the ability to find the pointer by asking the layout.

Therefore, what would you propose is the best way I can accomplish this.

I can try tracking all my pointers and deleting directly but as you say "thats what a layout is for"
I can also ask the layout for the items, but that doesnt seem to be working right for me.

[high_flyer]

Hmm... after thinking about it a bit more:

I am not sure what's QLayout policy in a case when you add a viewer to an already occupied cell.
Does it put the widgets on top of each other or creates a new row/column (cell).
I'd expect the later which is the reason I thought working with the layout position is not reliable.
However if the layout stacks the widgets, then your approach should work, but the question then is, which of the stacked widgets (in a cell) do you get?

Test it.


At any rate, you don't really need to track or manage anything, you just need to store the pointers of your widgets.
A QVector, or a QList is ok.
You don't need the layout position if you have the pointer.
Unless you allocate your items as children, you need these pointers to delete them, otherwise you get a memory leak.