<![CDATA[Cannot delete items of a QVBoxLayout]]>Hi!
this is in the constructor:

myGroupBox->setLayout(myBoxLayout);
myBoxLayout->setAlignment(Qt::AlingTop);

and then I call this:

if(myBoxLayout->takeAt(0) != NULL){
    QLayoutItem *item;
    while((item = myBoxLayout->takeQt(0)) != NULL){
        delete item->widget();
        delete item;
    }
}

to remove all widgets from the layout.
The program just crashes. What did I miss?
This code came from this thread, but I can't find a answer for my problem there:
https://stackoverflow.com/questions/4272196/qt-remove-all-widgets-from-layout
Thanks for answers

]]>https://forum.qt.io/topic/82341/cannot-delete-items-of-a-qvboxlayoutRSS for NodeFri, 10 Apr 2026 11:46:09 GMTSun, 13 Aug 2017 14:07:02 GMT60<![CDATA[Reply to Cannot delete items of a QVBoxLayout on Sun, 13 Aug 2017 15:15:06 GMT]]>Ahh about 1300 lines, unfortunately not small.
It works now... aaand yes, I think the deleting of the builddirectory worked.
What happened:
I deleted the build directory and then the program crashed at an other point before the setting up of the layout. I found a thing that I would not call a mistake, but it surely is in some way, because of my endless function calling, the function, where I set up the layouts got called twice. I don't really know, why this is a mistake, but after I fixed it, it worked and then the program was able to say that it is a nullpointer. So I changed it back to new QVBoxLayout and everything works.
The weird thing is, that I really don't know why it was a problem to setting up the layout as a nullpointer twice.
But, anyway, thanks for your help!

]]>https://forum.qt.io/post/410146https://forum.qt.io/post/410146Sun, 13 Aug 2017 15:15:06 GMT<![CDATA[Reply to Cannot delete items of a QVBoxLayout on Sun, 13 Aug 2017 15:18:14 GMT]]>Super :)
Im not sure what calling the function twice could be issue but it did do something it seems.

]]>https://forum.qt.io/post/410148https://forum.qt.io/post/410148Sun, 13 Aug 2017 15:18:14 GMT<![CDATA[Reply to Cannot delete items of a QVBoxLayout on Sun, 13 Aug 2017 15:15:06 GMT]]>Ahh about 1300 lines, unfortunately not small.
It works now... aaand yes, I think the deleting of the builddirectory worked.
What happened:
I deleted the build directory and then the program crashed at an other point before the setting up of the layout. I found a thing that I would not call a mistake, but it surely is in some way, because of my endless function calling, the function, where I set up the layouts got called twice. I don't really know, why this is a mistake, but after I fixed it, it worked and then the program was able to say that it is a nullpointer. So I changed it back to new QVBoxLayout and everything works.
The weird thing is, that I really don't know why it was a problem to setting up the layout as a nullpointer twice.
But, anyway, thanks for your help!

]]>https://forum.qt.io/post/410146https://forum.qt.io/post/410146Sun, 13 Aug 2017 15:15:06 GMT<![CDATA[Reply to Cannot delete items of a QVBoxLayout on Sun, 13 Aug 2017 15:06:56 GMT]]>I agree @Wieland , the big question is just Where :)

@Niagarer
If its a small project, can you zip and link it here ?

]]>https://forum.qt.io/post/410145https://forum.qt.io/post/410145Sun, 13 Aug 2017 15:06:56 GMT<![CDATA[Reply to Cannot delete items of a QVBoxLayout on Sun, 13 Aug 2017 15:01:20 GMT]]>I say "unitialized memory somewhere".

]]>https://forum.qt.io/post/410142https://forum.qt.io/post/410142Sun, 13 Aug 2017 15:01:20 GMT<![CDATA[Reply to Cannot delete items of a QVBoxLayout on Sun, 13 Aug 2017 15:00:34 GMT]]>@mrjj
Yep, this project works. So it has to be my bad.

]]>https://forum.qt.io/post/410141https://forum.qt.io/post/410141Sun, 13 Aug 2017 15:00:34 GMT<![CDATA[Reply to Cannot delete items of a QVBoxLayout on Sun, 13 Aug 2017 14:58:23 GMT]]>Ok, I did and now I have a mistake at a point in the code before the setting up of the layout.
I add a Tab of a tabWidget and there it crashes.,.. This was never a problem, this line works sonce weeks, I did not change anything there.
I deleted the build directory, opened creator again, ran qmake, rebuilt and that's it, no errors, it is just crashing

]]>https://forum.qt.io/post/410140https://forum.qt.io/post/410140Sun, 13 Aug 2017 14:58:23 GMT<![CDATA[Reply to Cannot delete items of a QVBoxLayout on Sun, 13 Aug 2017 14:46:55 GMT]]>Close Creator, delete the build directory and start Creator again. Sometime strange things happen.

]]>https://forum.qt.io/post/410139https://forum.qt.io/post/410139Sun, 13 Aug 2017 14:46:55 GMT<![CDATA[Reply to Cannot delete items of a QVBoxLayout on Sun, 13 Aug 2017 14:45:34 GMT]]>@mrjj
Now, it does not print any qDebugs() anymore... what the hell!?

]]>https://forum.qt.io/post/410138https://forum.qt.io/post/410138Sun, 13 Aug 2017 14:45:34 GMT<![CDATA[Reply to Cannot delete items of a QVBoxLayout on Sun, 13 Aug 2017 14:44:42 GMT]]>@Niagarer
Hi, no worries, we all do stupid mistakes sometimes.
Its a sign of a productive programmer ;)

first
QVBoxLayout *myBoxLayout; -> QVBoxLayout *myBoxLayout=nullptr.
in .h
then do NOT
myBoxLayout = nullptr; before the check. ( you kill it )

also , it must print something ???
myBoxLayout = nullptr;
if(myBoxLayout == nullptr){
qDebug() << "layout is null";
}
else something is really, really wrong.

Can you test this small sample ?
https://www.dropbox.com/s/tkl1hdik6q0qwl2/myreuselayout.zip?dl=0
Uses your code to remove buttons from layout.

]]>https://forum.qt.io/post/410137https://forum.qt.io/post/410137Sun, 13 Aug 2017 14:44:42 GMT<![CDATA[Reply to Cannot delete items of a QVBoxLayout on Sun, 13 Aug 2017 14:45:41 GMT]]>@mrjj
In the .h file:

    QVBoxLayout *myBoxLayout;

and in the .cpp file:

myBoxLayout = nullptr;
if(myBoxLayout == nullptr){
    qDebug() << "layout is null";
}

and it prints nothing...
I really don't understand that too.
I just hope that I am not doing a totally dumb mistake here...

]]>https://forum.qt.io/post/410136https://forum.qt.io/post/410136Sun, 13 Aug 2017 14:45:41 GMT<![CDATA[Reply to Cannot delete items of a QVBoxLayout on Sun, 13 Aug 2017 14:38:19 GMT]]>@Niagarer
what ??
myBoxLayout = nullptr;
if(myBoxLayout != nullptr){
qDebug() << "layout is not null";
}

that code should say nothing. :)
I assume that
myBoxLayout = nullptr; <<< in .h yes ?

Sorry, i have really no idea.
Its does work that way and i have used takeAt many times.

]]>https://forum.qt.io/post/410135https://forum.qt.io/post/410135Sun, 13 Aug 2017 14:38:19 GMT<![CDATA[Reply to Cannot delete items of a QVBoxLayout on Sun, 13 Aug 2017 14:35:57 GMT]]>@mrjj
Ok, I tried it.
I did not make the classic mistake but if I define it with a nullpointer

myBoxLayout = nullptr;

and then asking if it's nullptr where I try to delete all content, it says, it is not null
But I only defined it once...
And if I ask that this way:

myBoxLayout = nullptr;
if(myBoxLayout != nullptr){
    qDebug() << "layout is not null";
}

it says, it is not null...

]]>https://forum.qt.io/post/410134https://forum.qt.io/post/410134Sun, 13 Aug 2017 14:35:57 GMT<![CDATA[Reply to Cannot delete items of a QVBoxLayout on Sun, 13 Aug 2017 14:29:04 GMT]]>@Niagarer
If it crashes there, something must be wrong with the pointer
myBoxLayout->count();

You did show real code, yes ?

A classic is to define in .h
and then do
QVBoxLayout *myBoxLayout = new QVBoxLayout; ( in ctror)
and make local variable instead.

set it to null where u define it
QVBoxLayout *myBoxLayout=nullptr;

and check if its NON null before using it on the count()

]]>https://forum.qt.io/post/410131https://forum.qt.io/post/410131Sun, 13 Aug 2017 14:29:04 GMT<![CDATA[Reply to Cannot delete items of a QVBoxLayout on Sun, 13 Aug 2017 14:26:12 GMT]]>@mrjj
Thanks, I tried and it crashes at exactely this line you gave me.
I can not do anything with the layout
Seems to be a nasty one...

]]>https://forum.qt.io/post/410129https://forum.qt.io/post/410129Sun, 13 Aug 2017 14:26:12 GMT<![CDATA[Reply to Cannot delete items of a QVBoxLayout on Sun, 13 Aug 2017 14:22:01 GMT]]>@Niagarer
Cant guess then.

try
qDebug() << "items:" << myBoxLayout->count();
just before the If.

]]>https://forum.qt.io/post/410127https://forum.qt.io/post/410127Sun, 13 Aug 2017 14:22:01 GMT<![CDATA[Reply to Cannot delete items of a QVBoxLayout on Sun, 13 Aug 2017 14:19:49 GMT]]>@mrjj
Nope, this is a class attribute and in the constructor (all in the same class) I do

myBoxLayout = new QVBoxLayout;

and there is also no error, that works.

]]>https://forum.qt.io/post/410126https://forum.qt.io/post/410126Sun, 13 Aug 2017 14:19:49 GMT<![CDATA[Reply to Cannot delete items of a QVBoxLayout on Sun, 13 Aug 2017 14:17:22 GMT]]>Hi
It crashes at
if(myBoxLayout->takeAt(0) .. ?

did you forget to new myBoxLayout ?

]]>https://forum.qt.io/post/410125https://forum.qt.io/post/410125Sun, 13 Aug 2017 14:17:22 GMT<![CDATA[Reply to Cannot delete items of a QVBoxLayout on Sun, 13 Aug 2017 14:15:49 GMT]]>@mrjj
on the line with the if statement

]]>https://forum.qt.io/post/410124https://forum.qt.io/post/410124Sun, 13 Aug 2017 14:15:49 GMT<![CDATA[Reply to Cannot delete items of a QVBoxLayout on Sun, 13 Aug 2017 14:12:42 GMT]]>Hi
Seems ok but there some notes


if(myBoxLayout->takeAt(0) != NULL){ << you take first out first here ( and do not delete it)
    QLayoutItem *item;
    while((item = myBoxLayout->takeQt(0)) != NULL){
        delete item->widget();
        delete item;
    }
}

IF you single step , what lines does it crash on ?

]]>https://forum.qt.io/post/410123https://forum.qt.io/post/410123Sun, 13 Aug 2017 14:12:42 GMT