Thread: One Button program for different uses.

Hi friends,

Being a newbie means I have to be humble and ask for help from time to time.
I'm trying to write code for a two button set up, high/low, which are used to change their values in memory. I press either button 3 seconds to enter setup. Once I'm in set up, the same two buttons are inputs to change their respective values in memory. I press the high button 3 seconds enter setup, then the high button increments the H value, and the low button decrements the H value. Likewise, when I press the low button 3 seconds, I enter low value setup, high button increments the low value while low button decrements the low value. If I hold a button down, I want the value to increase/decrease until I reach the desired value.

(forgive me for not following protocol with brackets, I'm still not too good at it)

Code:

if (button4 == HIGH)
{    
 tick1 = tick1++; //wait 3 seconds then enter setup. 
   if(tick1 = 65000){  
      delay(300);
      tick1 == 0; //reset timer 
      if(button4 == HIGH){ 
       lset == lset--; //decrement low value, write it in memory 
       EEPROM.write(11, lset);
      //here I want to loop back to button4 to decrement again as needed.    

    }
    else if(button3 == HIGH){
        lset == lset++; //increment low value and write it down
        delay(300);
        EEPROM.write(11, lset);
       //here I want to loop back to button3 to increment again as needed. 
    }
 }
}

You've got everything turned around in the first few lines.

Code:

 tick1 = tick1++; //wait 3 seconds then enter setup. 
   if(tick1 = 65000){  
      delay(300);
      tick1 == 0; //reset timer

Should be

Code:

   tick1++; //wait 3 seconds then enter setup. (note: no reassignment here)
   if(tick1 == 65000){   // == for comparison
      delay(300);
      tick1 = 0; //reset timer,  = for assignment

Correcting the indentation.

Code:

if (button4 == HIGH)
      {    
      tick1++; //wait 3 seconds then enter setup. 
            if(tick1 == 35000)
                 {  
                 delay(300);
                 tick1 = 0; //reset timer 

  if(button4 == HIGH)
       { 
       lset == lset--; //decrement low value, write it in memory 
       EEPROM.write(11, lset);
       delay(300);
       //here I want to loop back to button4 to decrement again as needed.    
       }

  else if(button3 == HIGH)
       {
        lset = lset + 1; //increment low value and write it down
        EEPROM.write(11, lset);
        delay(300);
        //here I want to loop back to button3 to increment again as needed. 
       }
   }
}

Code:

    if (button4 == HIGH)
    {
        tick1++; //wait 3 seconds then enter setup.
        if(tick1 == 35000)
        {
            delay(300);
            tick1 = 0; //reset timer

            if(button4 == HIGH)

Now that the indentation is clear, what does this mean?
The second if seems to be trivially true, if the first one happened.

Are your buttonx variables directly linked to some external hardware?
Are they declared as being 'volatile'?

I have two buttons on hardware, 3 and 4, and I use them both to enter setup and to enter the high and low value in memory, so if I press button 3 for three seconds, I enter setup, then I press 3 again, and this time it increments the high value in memory, while button 4 decrements the value stored in the high memory. Conversely, if I press 4 for three seconds, I set up the low memory with both buttons. That's why I have button4 HIGH again, this time, in setup situation, where both buttons serve to raise/lower the value in high memory. It's a dual use of the same button, first to enter setup, second to increment the value.

And if I let it sit for 5 seconds without changing values, it exits setup.

PS Will read tutorials.

Thanks for your help, I will work on this tomorrow. I have to finish this soon, I have customers waiting.

Code:

 if (button3 == HIGH)
      {    
      tick1++; //wait 3 seconds then enter setup. 
      if(tick1 == 35000)
            {  
            digitalWrite(backLightPin, LOW); //flash screen two times.
            delay(300);
            digitalWrite(backLightPin, HIGH); 
            delay(300);
            digitalWrite(backLightPin, LOW);  
            delay(300);
            digitalWrite(backLightPin, HIGH); 
            delay(300);
            tick1 = 0;                           //reset timer 
            hset = EEPROM.read(10);      //read and display last setup value
            lcd.print(hset, DEC);  
                 while(button4 == HIGH)
                    { 
                    delay(300);  
                    hset--;                        //decrement high value, write. 
                    EEPROM.write(11, hset);
                    lcd.print(hset, DEC);  
                    }
                     
                  while(button3 == HIGH)
                    {
                    delay(300);  
                    hset++;                         //increment high value, write.
                    EEPROM.write(11, hset);
                    lcd.print(hset, DEC);  
                    }
                        //exit setup
                    }
             }
      }