Increse/Decrese a value with a button

[dbemowsk]

@dzjr Can you edit your post to put your code in a code block. When you edit the post, highlight your code section and then click on the </> icon. Also, your code seems a little odd to me. You have for example

if (SetHigher == HIGH) {
(Setpoint2 ++);}

I would write that like this

if (SetHigher == HIGH) {
    Setpoint2 ++;
}

No need to put parenthesis around your "Setpoint2 ++".

@mfalkvidd He did say "input both with digitalread and with a debouncer, both have the same effect.", though I'm not sure what kind of debouncer he tried.

[dzjr]

@dbemowsk i did edited the original post as you asked.

[dbemowsk]

@dzjr Thank you. Many here will appreciate that as it makes it much easier to read.

[electrik]

You now increase as long as the button is pressed, you should only increase on pressing the button. So you should check if the button has changed, and if it does increase the thermostat value

So something like

button = digitalRead (pin);
if (button != OldButton)
{
  Value++;
  OldButton = button;
}

[dzjr]

@electrik

I even tried that, and it's still sky high

the code i made is :

int SetHigher == digitalRead(Higher_PIN);
int SetHigherOld ;

if (SetHigher != SetHigherOld)
{
Heating_set_High ++;
SetHigherOld = SetHigher;
}

Also tried boolean for digitaRead.

!!! EDIT, changed the code !!!

[zboblamont]

@dzjr Delay incrementing via a while loop with 50-500ms sleep/wait, to check the button has changed state before proceeding. Modify delay value to suit.

[electrik]

@dzjr

you are reading the pin into SetHigher first, which is good.
Then you are setting SetHigherOld to a value which is negative, so always different from SetHigher. So that is always true. I also don't see how a negative value can fit into a boolean.

I now see I made a mistake in my previous post. I'll update it.

[dzjr]

@electrik
Thank you all for the help!

It's working!!!'

I have done the following;

at the top:

float Heater_set_High = 17;
float Change_Degree = 0.5;
unsinged long SetHigherOld = 0;
unsinged long SetLowerOld = 0;

i used 0,5 becouse than the change of de setpoint is changed by 1 when you press the button.

The reading from te button i do with the debouncer (SetHigher & SetLower)

The sketch:

if (SetHigher !=SetHigherOld){
Heater_set_High += ChangeDegree;
SetHigherOld = SetHigher;
}

if (SetLower !=SetLowerOld){
Heater_set_High -= ChangeDegree;
SetLowerOld = SetLower;
}

// To protect te setpoint going to high i made this:

if (Heater_set_High >= 25){
Heater_set_High = 25;
}
else if (Heater_set_High <=7){
Heater_set_High = 7;
}

//to reset the High setpoint after High temperature time or switching it off:

if (Heater_High == LOW){
Heater_set_High = 17;
}

I also can change it in the controller by sending a temperature message to the node.

When i finished the project i will make a post of this project.

[electrik]

It is now still activated twice per button press, as this code will be executed on both pressing and releasing the button.
You could check if it is high so you can change your setpoint by 0.5 degrees

[dzjr]

@electrik I already dit set it at 0,5 so it is changing 1 degree.

I now have made it even 0.05 so i can change it by 0,1 degree.

Thanks for all the help!