[Solved] How to save checkbox/radiobutton states with QSettings?

Franzk

If you expect a boolean, yes you should.

andre

You mind me asking why on earth you are storing the buttons text propery to QSettings?

Leon

Andre i have a button that when you press it you select an audio file.. When you click ok the text of the button changes to the path of the audio file.. So when you close the app and reopen i need the button to have the path of the sound that was previously selected ! :)

P.S Thanks Franzk

andre

That sounds like a weird interface to me... What does a button with a file name on it do? Play it? No, in your case, it allows to (presumably) select another one. Not very intuitive IMHO. Also, from a programming perspective, not the nicest separation on concerns. Why not store the file name name where it makes sense (some class variable or better yet, property) at runtime and store that to QSettings when you need to. From that, you can simply re-set the text for the button. If you ever decide you don't want that file name on the button after all, there is only one piece of code to change...

Leon

I Just think this is the right thing to do..! And it looks perfect at my app! So i will keep it like this.

Leon

What if the .conf file doesn't exist? ( First time running the program )
Even if the default check state of a checkbox is checked and the .conf doesn't exist
@ui->preview->setChecked(settings.value("preview", false).toBool());@
will make it unchecked...

So should i check for file existence?

goetz

If the file does not exist, you will have to provide a default anyways. So just adjust the default value of the value("name", default) method call. If you change it from false to true your checkbox is checked by default.

Leon

So you mean something like:
@ui->preview->setChecked(settings.value("preview", true).toBool());@
? I am not sure...

goetz

Oh come on! Just try it out! That's quicker than writing a comment here and waiting for an answer!

Leon

So for the checkboxes that are checked by default i should have true and for the ones that are not checked false? Right?

EDIT: I checked before adding the comment

goetz

[quote author="Leon" date="1309953483"]So for the checkboxes that are checked by default i should have true and for the ones that are not checked false? Right?

EDIT: I checked before adding the comment[/quote]

Sounds logical? no?

Leon

It works for me so propably yes! :)
Out of topic.. What about a slider?
What if i want to have as default value 6?
@ui->timerSlider->setValue(settings.value("timeSlider").toInt());@

EDIT: If the file doesn't exist the value of the slider will be set to 1

goetz

Then pass 6 as the default. Surprising?

Leon

I edited my answer..
If the file doesn’t exist the value of the slider will be set to 1 even if the default one is 6

Leon

What i did is this:
@QFile timeslide ( QDir::homePath()+"/.config/Program/MainWindow.conf" );
if (timeslide.exists()) ui->timerSlider->setValue(settings.value("timeSlider").toInt());@

If there is another way tell me!

andre

Let's reformat that code a bit to make it more readable:

@
QFile timeslide ( QDir::homePath()+"/.config/Program/MainWindow.conf" );
if (timeslide.exists()) {
ui->timerSlider->setValue(settings.value("timeSlider").toInt());
}
@

That makes it obvious that line 3 is not executed if the file does not exist. Hence, the value of the slider will not change. If you want to set the slider to some value indepent of the existance of the file, then you need to make sure you execute that line in a code path that doesn't only execute if the file exists.

Leon

I am reformating it at my code not here at the forum!
No i just want for the first time the default value to be 6.. After that the file will exist so if the user change it eg to value 4 the next time it will be value 4..
Ok thanks again!

goetz

This works perfectly for me:

@
QSettings testSettings;
int sliderVal = testSettings.value("slider/value", 6).toInt();
qDebug() << "sliderVal=" << sliderVal;
ui->horizontalSlider->setValue(sliderVal);
@

Be sure NOT to call or trigger any slot connected to signal valueChanged() before you have set the default value.