next topic »

[on:]

Hello Guys,

I am trying to configure first an Up and down or increment and decrement control for a display to show a "Volume" increasing by one or decreasing by one....I have attached my schematic and code...I can get the "Up" count working but then I tried some  "--" in my code and it didn't work..Please help....

Here is my code:


#include <LiquidCrystal.h>


LiquidCrystal lcd(7,8,9,10,11,12);



const int  buttonPin = 5;    // the pin that the Up pushbutton is attached to

const int  buttonPin1 = 6;    // the pin that the Down pushbutton is attached to

 
// Variables will change:


 int buttonPushCounter = 0;   // counter for the number of button presses
 int buttonState = 0;         // current state of the button
 int lastButtonState = 0;     // previous state of the button
 
void setup() {


   // initialize the button pin as a input:
   
   pinMode(buttonPin, INPUT);
   
   pinMode(buttonPin1, INPUT);
   
   
   lcd.begin(16,2);
   
   lcd.setCursor(0,1);
   
   lcd.print("Volume:");
   
 }
 

void loop() {
 
 
   // read the pushbutton input pin:
   
   buttonState = digitalRead(buttonPin);
   
   
 
  // compare the buttonState to its previous state
 
   if (buttonState != lastButtonState) {
     
     // if the state has changed, increment the counter
     
     if (buttonState == HIGH)
     
     {
       
      buttonPushCounter++;
      lcd.setCursor(7,1);
     
      lcd.print(buttonPushCounter);
     }
     

 
   }
   // save the current state as the last state,
  //for next time through the loop
   lastButtonState = buttonState;
 

}


schematic attached

[Reply #1 on:]

Just looked at my schematic......the pull up resistors are tied to high but they are connected to GND here....disregard that mistake....

[Reply #2 on:]

Please somebody help.....I have been racking my brain over how easy this is suppose to be but everything I try doesn't work

[Reply #3 on:]

If I were you, I would add a Serial.println() for every LCD.print() you have in your code plus the counter and state registers. If after test your volume buttons (dec/inc) everything goes right serially, then you have a hardware issue. These are just some ideas.

[Reply #4 on:]

I just can't get over how easy this seems but its kicking my ass......I mean, I have the BUtton state and all that...and with the ++ it works great incrementing...I even took the wire from the other button I'm trying to use as the 'Down counter" and it make the count go up...so somewhere in my code there needs to have some "--" decrementing, but I can't figure it out...hardware is good...its my code I need to fix....somebody out there has got to know..

[Reply #5 on:]

Try this:

Code:

#include <LiquidCrystal.h>


LiquidCrystal lcd(7,8,9,10,11,12);



const int  buttonPin = 5;    // the pin that the Up pushbutton is attached to

const int  buttonPin1 = 6;    // the pin that the Down pushbutton is attached to

 
// Variables will change:


 int buttonPushCounter = 0;   // counter for the number of button presses
 int buttonState5 = 0;         // current state of the button
 int buttonState6 = 0;         // current state of the button
 int lastButtonState = 0;     // previous state of the button

 
void setup() {


  // initialize the button pin as a input:
   
   pinMode(buttonPin, INPUT);
   
   pinMode(buttonPin1, INPUT);
   
   
   lcd.begin(16,2);
   
   lcd.setCursor(0,1);
   
   lcd.print("Volume:");
   
 }
 

void loop() {
 
 
   // read the pushbutton up input pin:
   
   buttonState5 = digitalRead(buttonPin);

  // compare the buttonState to its previous state
 
   if (buttonState5 != lastButtonState) {
     
     // if the state has changed, increment the counter
     
     if (buttonState5 == HIGH)
     
     {
       
      buttonPushCounter++;
      lcd.setCursor(7,1);
           
      lcd.print(buttonPushCounter);
     }
     
     
 
   }
   // save the current state as the last state,
  //for next time through the loop
   lastButtonState = buttonState5;
   
     // read the pushbutton down input pin:
   
   buttonState6 = digitalRead(buttonPin1);

  // compare the buttonState to its previous state
 
   if (buttonState6 != lastButtonState) {
     
     // if the state has changed, decrement the counter
     
     if (buttonState6 == HIGH)
     
     {
       
      buttonPushCounter--;
      lcd.setCursor(7,1);
         
      lcd.print(buttonPushCounter);
     }
     
             
   }
   // save the current state as the last state,
  //for next time through the loop
   lastButtonState = buttonState6;
   
   
}

[Reply #6 on:]

"--" decrementing, but I can't figure it out

instead of -- use -=1 or -=2 or any decrement you like, sam for incrementing +=1

[Reply #7 on:]

Thanks to both of you....I used both of the information given...my code now is shown below...


Its working...I just have to tweak it a little with some delays etc...



#include <LiquidCrystal.h>


LiquidCrystal lcd(7,8,9,10,11,12);



const int  buttonPin = 5;    // the pin that the Up pushbutton is attached to

const int  buttonPin1 = 6;    // the pin that the Down pushbutton is attached to

 
// Variables will change:


 int buttonPushCounter = 0;   // counter for the number of button presses
 int buttonState5 = 0;         // current state of the button
 int buttonState6 = 0;         // current state of the button
 int lastButtonState = 0;     // previous state of the button

 
void setup() {


  // initialize the button pin as a input:
   
   pinMode(buttonPin, INPUT);
   
   pinMode(buttonPin1, INPUT);
   
   
   lcd.begin(16,2);
   
   lcd.setCursor(0,1);
   
   lcd.print("Volume:");
   
 }
 

void loop() {
 
 
   // read the pushbutton up input pin:
   
   buttonState5 = digitalRead(buttonPin);

  // compare the buttonState to its previous state
 
   if (buttonState5 != lastButtonState) {
     
     // if the state has changed, increment the counter
     
     if (buttonState5 == HIGH)
     
     {
       
      buttonPushCounter++;
      lcd.setCursor(7,1);
           
      lcd.print(buttonPushCounter);
     }
     
      delay(50);
 
   }
   // save the current state as the last state,
  //for next time through the loop
   lastButtonState = buttonState5;
   
     // read the pushbutton down input pin:
   
   buttonState6 = digitalRead(buttonPin1);

  // compare the buttonState to its previous state
 
   if (buttonState6 != lastButtonState) {
     
     // if the state has changed, decrement the counter
     
     if (buttonState6 == HIGH)
     
     {
       
      buttonPushCounter-=1;
      lcd.setCursor(7,1);
         
      lcd.print(buttonPushCounter);
     }
     
         delay(50);   
         
         
         if (buttonPushCounter < 10)
         
         {
             lcd.setCursor(8,1);
         
              lcd.print("  ");
         }
         
         
         if (buttonPushCounter <= 0)
         
         {
           
           
            lcd.setCursor(7,1);
           
            lcd.print("OFF");
           
         }
         
         
         
         if (buttonPushCounter >= 25)
         
         {
           
           lcd.setCursor(7,1);
           
           lcd.print("Max");
           
         }
         
       
   }
   
   
   // save the current state as the last state,

  //for next time through the loop

   lastButtonState = buttonState6;
   
   
}

[Reply #8 on:]

Please edit your post, select the code, and put it between [code] ... [/code] tags.

You can do that by hitting the # button above the posting area.


Please use code tags in future, it makes your code easier to follow.

[Reply #9 on:]

Also the double-spacing is distracting. This isn't an essay.

[Reply #10 on:]

Hi .. i'm a newb


i tried this code ... on my arduino (gift) and it seems to go beyond  the "OFF" & "MAX"

how do i make it count from OFF to 5(MAX)  ... because this code its prints  OFF and MAX on display .. but he goes beyond that with -1 -2 -3 -4 -5 ..and when i press my +button ... i have to go from -2 or how many times i pushed the botton after OFF

and for MAX the same ..goes after my  if (buttonPushCounter >= 5)

AND i want to modify this code to control a blinking led ...

something like :
if OFF = closed led
if 1    = 1/min
if 2    = 2/min
if 3    = 3/min
if MAX= 4/min

+ to display this


PS i don't have any coding knowledge  sry  but im triend to learn but i got this 2 days ago ..i i dont want to trow it away because i dont know how this is done
I have browsed the forum for something similar ..cant find anything ...

Thanks to both of you....I used both of the information given...my code now is shown below...


Its working...I just have to tweak it a little with some delays etc...



#include <LiquidCrystal.h>


LiquidCrystal lcd(7,8,9,10,11,12);



const int  buttonPin = 5;    // the pin that the Up pushbutton is attached to

const int  buttonPin1 = 6;    // the pin that the Down pushbutton is attached to

 
// Variables will change:


 int buttonPushCounter = 0;   // counter for the number of button presses
 int buttonState5 = 0;         // current state of the button
 int buttonState6 = 0;         // current state of the button
 int lastButtonState = 0;     // previous state of the button

 
void setup() {


  // initialize the button pin as a input:
   
   pinMode(buttonPin, INPUT);
   
   pinMode(buttonPin1, INPUT);
   
   
   lcd.begin(16,2);
   
   lcd.setCursor(0,1);
   
   lcd.print("Volume:");
   
 }
 

void loop() {
 
 
   // read the pushbutton up input pin:
   
   buttonState5 = digitalRead(buttonPin);

  // compare the buttonState to its previous state
 
   if (buttonState5 != lastButtonState) {
     
     // if the state has changed, increment the counter
     
     if (buttonState5 == HIGH)
     
     {
       
      buttonPushCounter++;
      lcd.setCursor(7,1);
           
      lcd.print(buttonPushCounter);
     }
     
      delay(50);
 
   }
   // save the current state as the last state,
  //for next time through the loop
   lastButtonState = buttonState5;
   
     // read the pushbutton down input pin:
   
   buttonState6 = digitalRead(buttonPin1);

  // compare the buttonState to its previous state
 
   if (buttonState6 != lastButtonState) {
     
     // if the state has changed, decrement the counter
     
     if (buttonState6 == HIGH)
     
     {
       
      buttonPushCounter-=1;
      lcd.setCursor(7,1);
         
      lcd.print(buttonPushCounter);
     }
     
         delay(50);   
         
         
         if (buttonPushCounter < 10)
         
         {
             lcd.setCursor(8,1);
         
              lcd.print("  ");
         }
         
         
         if (buttonPushCounter <= 0)
         
         {
           
           
            lcd.setCursor(7,1);
           
            lcd.print("OFF");
           
         }
         
         
         
         if (buttonPushCounter >= 25)
         
         {
           
           lcd.setCursor(7,1);
           
           lcd.print("Max");
           
         }
         
       
   }
   
   
   // save the current state as the last state,

  //for next time through the loop

   lastButtonState = buttonState6;
   
   
}

[Reply #11 on:]

This appears to be a new member picking up an old (> 6 months) thread.
The quote was unnecessary however.

@siko:
You need to learn some more basics.
If you have a range you want to use, you are the one to tell your Arduino how to handle that range.
So if off is 0, then do not allow the value to go below 0, and if on = 255 do not allow the value to be more than that.
Or if the value is already "out of range", force it back in range before you do anything else with it.
There are multiple ways of handling this.
Everything you need for that is already in the sketch, use your imagination to extend it to fit your needs.
(You just need to add 2 lines).