How do I make my own QDialog working like for example QFileDialog

[marcel]

What do you mean when you say the code in main window continues? That is not possible...
Have you tested this in debug?
Do you have other threads running?

Anyway, exec() returns a code that specifies if the user pressed OK or cancel. So you should test for that ( provided that you have OK and cancel buttons ).

regards

[Randulf]

What do you mean when you say the code in main window continues? That is not possible...
Have you tested this in debug?
Do you have other threads running?

I know I thougt it sholdn't, I have no other threads.
But if do like this using the code above.

int r = esw->exec();
	qDebug() << r;
	qDebug() << esw->getSettings();

int r = esw->exec();
	qDebug() << r;
	qDebug() << esw->getSettings();

To copy to clipboard, switch view to plain text mode 

printout is:
1
()
Accept.

Anyway, exec() returns a code that specifies if the user pressed OK or cancel. So you should test for that ( provided that you have OK and cancel buttons ).

I have only impelemented two buttons and connected them to accept and reject slots. Is that enough?

But....Ah...hmmm..."show() returns control to the caller immediately". Maybe I shouldn't use show() but how do I tell it to be visible?

[marcel]

Oh, that's because in your exec() implementation you call show() . I missed that.
You must call QDialog::exec().

Regards

[jpn]

MyDialog::exec() is incorrect. First of all, QDialog::exec() is not virtual so it might be a better idea to rename the method to avoid confusion if you even need a method like that at all. I'd suggest removing it and using QDialog::exec() which should be sufficient for you. QDialog::exec() starts an event loop which blocks while the dialog is open.

The static convenience methods of various QDialog subclasses work more or less like this:

class MyDialog : public QDialog
{
public:
	static QString doSomething(QWidget* parent = 0);
 
private:
	QString info;
};
 
QString MyDialog::doSomething(QWidget* parent)
{
	MyDialog dialog(parent);
	if (dialog.exec() == QDialog::Accepted)
		return dialog.info;
	return QString();
}
 
// usage:
QString blaa = MyDialog::doSomething();
if (!blaa.isNull())
{
	// dialog was accepted, blaa contains the info
}

class MyDialog : public QDialog
{
public:
	static QString doSomething(QWidget* parent = 0);

private:
	QString info;
};

QString MyDialog::doSomething(QWidget* parent)
{
	MyDialog dialog(parent);
	if (dialog.exec() == QDialog::Accepted)
		return dialog.info;
	return QString();
}

// usage:
QString blaa = MyDialog::doSomething();
if (!blaa.isNull())
{
	// dialog was accepted, blaa contains the info
}

To copy to clipboard, switch view to plain text mode 

[marcel]

First of all, QDialog::exec() is not virtual

exec() is a slot in QDialog, therefore it is virtual.

But yes, you are right, it is better to use QDialog::exec directly and won't make any difference in this case.

Regards

[wysota]

exec() is a slot in QDialog, therefore it is virtual.

Only when called as a slot.

[marcel]

Only when called as a slot.

Yes. Damn it!

[wysota]

Yes. Damn it!

It doesn't have to be virtual to reimplement and use it of course... Virtuality is only needed when something else calls the method.

[jpn]

It doesn't have to be virtual to reimplement and use it of course... Virtuality is only needed when something else calls the method.

True, but it's still a bad practice to shadow non-virtual methods.

[wysota]

True, but it's still a bad practice to shadow non-virtual methods.

"It depends" I don't see anything improper in reimplementing QDialog::exec() I know it's possible that someone will want to call it after casting to QDialog, but hey! let's try our luck Of course it's better to avoid such risks...

[Randulf]

Thanks a lot for the help and tips!!
Got it working now.
Removed exec(), accept(), reject() and done().

Now I can just

int r = myDialog->exec()
if(r==1)
    QStringList settings = myDialog->readSettings()
delete myDialog

int r = myDialog->exec()
if(r==1)
    QStringList settings = myDialog->readSettings()
delete myDialog

To copy to clipboard, switch view to plain text mode 

Thanks again!!!