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

[Franzk]

Store isChecked() in a setting. You can read it as a boolean:

@settings.setValue("myCheckBox", myCheckBox->isChecked());

myCheckBox->setChecked(settings.value("myCheckBox", false /* default value */).toBool());@

If you use the tristate version you should use checkedState.

[loladiro]

QSettings uses QVariant and QVariant supports bool.

[Leon]

Ok thanks a lot! It works!
Should i change toString() to toBool() when importing the settings?

[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!