Removing QWidgets from a QGridLayout

Question

I have a QGridLayout where I add my custom QWidgets.

When I try to delete all of them they are supposedly removed from the layout (as the function layout.count() returns 0) but they are still shown in the interface and I can interact with them.

Here you have the way I add widgets:

void MyClass::addCustomWidget(CustomWidget *_widget, int r, int c)
{
    layout->addWidget(_widget, r, c);
    _widget->show();
}

And here the way I delete them:

void MyClass::clearLayout()
{
    qDebug() << "Layout count before clearing it: " << layout->count();

    int count = layout->count();
    int colums = layout->columnCount();
    int rows = layout->rowCount();

    int i=0;
    for(int j=0; j<rows; j++)
    {
        for(int k=0; k<colums && i<count; k++)
        {
            i++;

            qDebug() << "Removing item at: " << j << "," << k;
            QLayoutItem* item = layout->itemAtPosition(j, k);

            if (!item) continue;

            if (item->widget()) {
                layout->removeWidget(item->widget());
            } else {
                layout->removeItem(item);
            }
            qDebug() << "Removed!";
        }
    }

    qDebug() << "Layout count after clearing it: " << layout->count();
}

Any kind of help or tip to delete items/widgets correctly from a QGridLayout?

P.D. : I have seen on the internet that a lot of people deletes the widget directly (delete _widget) after removing them from the layout. In my case it is not possible as I need to mantain that widgets in memory.

possible duplicate of [Removing widgets from QGridLayout](http://stackoverflow.com/questions/5395266/removing-widgets-from-qgridlayout) — emkey08, Oct 08 '13 at 20:01

score 3 · Answer 1 · answered Mar 12 '17 at 10:35

3

You can also use deleteLater() to avoid issue with maintaining child count during iterations:

for (int i = 0; i < gridLayout.count(); i++)
{
   gridLayout.itemAt(i)->widget()->deleteLater();
}

answered Mar 12 '17 at 10:35

vdudouyt

843
7
14

1

Thanks for trying to solve the issue 5 years later ;) – azorrozua Mar 13 '17 at 10:27

score 2 · Accepted Answer · edited Jul 11 '17 at 03:00

Just to be clear. You didn't "delete" the widgets. You only removed them from layout. Removing from layout means only that widget will be no more managed (resized/positioned) by this layout BUT it doesn't mean that widget will be "deleted" (in C++ way). Also widget won't be magically hidden. Your widget after removing from layout still leaves in widget it was created / managed in. So owner of this layout still has this widget as child (visible child).

You have to

hide widget or if you're sure it will not be used anymore
delete widget with "delete" keyword

Also you don't need to call removeWidget(item->widget()); removeItem(item) will be enough for all layout items (even those with widget inside)

My intent is to remove the widgets from the layout (and don't show them anymore) but mantain them in memory. I tried hiding the widgets (_widget.hide) before I remove them from the layout and it worked, thanks. — azorrozua, May 23 '12 at 08:49

score 1 · Answer 3 · answered May 23 '12 at 08:44

Try

QLayoutItem *child;
while ((child = layout->takeAt(0)) != 0);

It is supposed to be safe. If for any reasons it doesn't work, you can use a collection of widgets or layoutitems, which is updated every time you add a widget. Then to delete you loop on the collection and remove each element from the layout.

score 0 · Answer 4 · answered Feb 27 '14 at 10:10

Header:
class Grid : QGridLayout
{
public:
    Grid(QWidget* parent);
    void addElement(QWidget* element, int x, int y);
    void delElement(int x, int y);
    void resize(int width, int height);
    void clear();
protected:
    QSize size;
};

void Grid::clear(){
    for(int i = 0;i<size.width();i++){
        for(int j = 0;j<size.height();j++){
            QLayoutItem* item = takeAt(i*size.width() + j);
            if(item != NULL) delete item;
        }
    }
}

score 0 · Answer 5 · answered Oct 01 '20 at 21:43

None of these answers worked for me. In my situation, I have several object each with their own QChartView. The idea is that the user selects which object they want to view and a common area central in the main window is updated with the QChartView of the user-selected object. This should have been straightforward, simply remove the widget from the chart area, add a new one. What ended up working for me was this:

while( QLayoutItem* item = ui->mPlotArea->layout()->takeAt(0) )
{
    item->widget()->setVisible(false);
    ui->mPlotArea->layout()->removeItem(item);
}

ui->mPlotArea->layout()->addWidget( pv );
pv->setVisible(true);

Where mPlotArea is a QFrame and pv is a derived class of QChartView. I can't explain why the other answers did not worked, but I spent a couple of hours trying different widgets and different ways of removing without deleting, organizing, etc...

Removing QWidgets from a QGridLayout

5 Answers5