Two ways, Which one is better?

[talk2amulya]

hey, i was wondering if in the second fragment, its done wrongly. cuz 'this' has been set as the parent of the dialog and as every parent deletes its childs when deleting themselves, when it will try to delete the dialog , the app should crash cuz the dialog is actually created on stack..

[HelloDan]

hey, i was wondering if in the second fragment, its done wrongly. cuz 'this' has been set as the parent of the dialog and as every parent deletes its childs when deleting themselves, when it will try to delete the dialog , the app should crash cuz the dialog is actually created on stack..

You mean that the dialog should be a modal one?

[talk2amulya]

no, that wont make difference..all i mean is widgets and dialogs should always be created on heap..AFAIK

[wysota]

no, that wont make difference..all i mean is widgets and dialogs should always be created on heap..AFAIK

No, that's not true. A modal dialog will always be destroyed before its parent. Unless of course you explicitely shoot yourself in the leg but then the dialog shouldn't have been modal. It really doesn't matter which way you choose here. The stack based version is simply a few statements shorter to code and a tiny bit faster to execute.

[talk2amulya]

so will that concept apply even in the case of a normal dialog(non-modal)?

[lni]

One goes out of scope by itself, one doesn't automatically go out of scope...

If it is a modal dialog, the 2nd one works as good as the 1st one but with less codes... You can't say one is better than the other, it depends on situation...

[HelloDan]

Does this forum has any utilities to save a thread, such add to favouites?

[wysota]

so will that concept apply even in the case of a normal dialog(non-modal)?

No. A non-modal dialog has to be created on heap.

Does this forum has any utilities to save a thread, such add to favouites?

Yeah, there is a "Bookmarks" table at the bottom of the page.

[talk2amulya]

ok, so that means when i have a model dialog on STACK with another widget as parent..thats not wrong? one would think it should be...when parent will try to delete it, it should be crashing time..or am i getting smth really wrong about model dialog??.or maybe u can create a model dialog on stack and not give it a parent..in these examples, 'this' is parent, so be a dialog model or not, dialog creation on stack should be wrong.

[wysota]

ok, so that means when i have a model dialog on STACK with another widget as parent..thats not wrong?

No, it's not. Lifespan of a modal dialog is short - most probably it is born and dies within the same function (method) call. So there is no risk the child will live longer than its parent because it is created on the stack within one of its parent's (sub)methods.

one would think it should be...when parent will try to delete it, it should be crashing time..or am i getting smth really wrong about model dialog??

Yes, it would crash. But the parent won't try to delete it because it opened the dialog for a reason and it wants to react on the choice/action performed by the user within the modal dialog.

.or maybe u can create a model dialog on stack and not give it a parent..

Yes, you can. But then the dialog will not really be modal.

in these examples, 'this' is parent, so be a dialog model or not, dialog creation on stack should be wrong.

No, it's fine :-) The parent lives longer than the child so the child will detach itself from the parent when it goes out of scope and is destroyed thus the parent will not try to delete it when it (the parent this time) goes out of scope.

[talk2amulya]

ok, i can follow this chain of thoughts..but one thing still bugs me..i read on some of the threads here that one should ALWAYS create widgets and dialogs on heap, layouts also..cuz once they go out of scope and parents try to delete them, crash occurs(obviously!).. so when u say child "detaches" itself from parent when goin out of scope..wont that occur in other cases too and parent wont try to delete ths child? and why the recommendation(in multiple threads) that these things SHOULD be created on stack..how and when(in what cases) would a child detach from parent? my main concern is how would a parent know the child has already gone out of scope and not try to delete it?

another thing thats done here is dialog on heap is given a parent and then deleted in the same function..wont the parent try to delete the dialog object or dialog again detaches?i mean once u give a widget a parent, deleting it yourself should be kinda foolish? i can go on with queries but essence of my question is how does a parent know which child to delete and in what cases would a child detach itself from parent?

[wysota]

ok, i can follow this chain of thoughts..but one thing still bugs me..i read on some of the threads here that one should ALWAYS create widgets and dialogs on heap, layouts also..cuz once they go out of scope and parents try to delete them, crash occurs(obviously!)..

Only when both the parent and the child are created on the same frame of the stack or the child is created on the stack, the parent on the heap and you delete the parent while the stack frame holding the child is still valid. This is not the case here and it's one of few exceptions when it's ok to create a widget on stack.

so when u say child "detaches" itself from parent when goin out of scope..wont that occur in other cases too and parent wont try to delete ths child?

The problem is with double deleting the same object. Consider a situation when the parent is on the heap, the child is on the stack and you delete the parent. When this happens, the parent deletes all its children including the one child residing on the stack. So far this is fine. The trouble begins when the stack frame holding the child widget goes out of scope. The compiler calls destructors of all object residing in the frame including the child widget. Unfortunately the child has already been deleted earlier so we're trying to destroy an object that has already been destroyed which results in aborting the process.

and why the recommendation(in multiple threads) that these things SHOULD be created on stack.

With modal dialogs it is easier to create them on the stack because you don't have to worry about deleting them afterwards because the compiler does that for you.

.how and when(in what cases) would a child detach from parent?

When it's deleted. The case we're talking about is not related to not detaching from the parent but to double deletion caused by the compiler. And we can't overcome that so you shouldn't create widgets on the stack if you fear the parent might be deleted before the child is deleted.

my main concern is how would a parent know the child has already gone out of scope and not try to delete it?

The parent (during runtime) knows. The compiler (during compile time) does not.

i mean once u give a widget a parent, deleting it yourself should be kinda foolish?

I'd say it would be foolish if you couldn't delete an object if you wanted to.