
Originally Posted by
Codepoet
...because it went out of scope...
This was an implementation bug
What would you prefer in the following situation?
MyModalDialog modalDlg;
modalDlg.exec()
if(modalDialog.getSomething()==true){
//do something;
}
MyModalDialog modalDlg;
modalDlg.exec()
if(modalDialog.getSomething()==true){
//do something;
}
To copy to clipboard, switch view to plain text mode
or
MyModalDialog *modalDlg = new MyModalDialog;
modalDlg->exec()
if(modalDialog->getSomething()==true){
//do something
}
delete modalDlg;
MyModalDialog *modalDlg = new MyModalDialog;
modalDlg->exec()
if(modalDialog->getSomething()==true){
//do something
}
delete modalDlg;
To copy to clipboard, switch view to plain text mode
Then after code being added by many others...
MyModalDialog *modalDlg = new MyModalDialog;
modalDlg->exec()
if(modalDialog->getSomething()==true){
//do something
}else{
return; //Ouch
}
// Lot's of code that might also return somewhere
delete modalDlg;
MyModalDialog *modalDlg = new MyModalDialog;
modalDlg->exec()
if(modalDialog->getSomething()==true){
//do something
}else{
return; //Ouch
}
// Lot's of code that might also return somewhere
delete modalDlg;
To copy to clipboard, switch view to plain text mode
I'm not saying don't use pointers, but why not avoid them whem they are not needed?
Bookmarks