State change detection for active LOW inputs

I often point new users to the State Change Detection example. That example is great for teaching about edge detection but uses a momentary switch with an external pull-down resistor. Using the same switch, but with the internal pullup enabled and wired to ground is better practice (in most cases). Enable the internal pullup resistor with:

pinMode(pin, INPUT_PULLUP);

The switch is wired as shown. The resistor is not required if the internal pullup is used, but may be necessary if the switch is at the end of long wires. Stronger pullup with lower resistor values (min. 1K). The cap is optional, but will provide some hardware debounce and if the switch is at the end of long wires can filter noise that can cause false indications. Also shown is the safe way to wire the ever popular 6mm tactile switches.

Here is the code from the example modified to use the internal pullup and ground for an active LOW switch. Active LOW means that the switch is LOW when closed (pressed) and HIGH when open (not pressed).

/*
The circuit:
   - pushbutton attached to pin 2 from ground
   - the internal pullup on pin 2 is enabled
   - LED attached from pin 13 to ground (or use the built-in LED on most
    Arduino boards)
*/

// this constant won't change:
const byte  buttonPin = 2;    // the pin that the pushbutton is attached to
const byte ledPin = 13;       // the pin that the LED is attached to

// Variables will change:
int buttonPushCounter = 0;   // counter for the number of button presses
boolean buttonState = 0;         // current state of the button
boolean lastButtonState = 0;     // previous state of the button

void setup()
{
   // initialize the button pin as a input:
   pinMode(buttonPin, INPUT_PULLUP);
   // initialize the LED as an output:
   pinMode(ledPin, OUTPUT);
   // initialize serial communication:
   Serial.begin(9600);
}


void loop()
{

   static unsigned long timer = 0;
   unsigned long interval = 20;
   if (millis() - timer >= interval)
   {
      timer = millis();
      
      // 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 == LOW)
         {
            // if the current state is LOW then the button went from off to on:
            buttonPushCounter++;
            Serial.println("on");
            Serial.print("number of button pushes: ");
            Serial.println(buttonPushCounter);
         }
         else
         {
            // if the current state is HIGH then the button went from on to off:
            Serial.println("off");
         }
          // save the current state as the last state, for next time through the loop
          lastButtonState = buttonState;
      }
  }

   // turns on the LED every four button pushes by checking the modulo of the
   // button push counter. the modulo function gives you the remainder of the
   // division of two numbers:
   if (buttonPushCounter % 4 == 0)
   {
      digitalWrite(ledPin, HIGH);
   }
   else
   {
      digitalWrite(ledPin, LOW);
   }
}

Please see here for further information on hardware and software debounce.

1 Like

buttonState = digitalRead(buttonPin); In the 21st century, "buttonState" doesn't have global scope (it wouldn't be a Boolean either)

Why not Boolean?

fionning_macara:
Why not Boolean?

Because digitalRead returns an int, and LOW and HIGH are not boolean values.

Fair enough :wink:

I don’t understand the comment about wiring diagonally to avoid a short? Isn't it sometimes more convenient to wire 'laterally', i.e. on same side of the button?

Diagonals are always switched, so it's a good way to guarantee you don't connect two wires permanently as would happen if you accidentally used terminals 3 and 4 in the red box below for example.

square buttons.jpg

Thanks, understood. So avoiding connecting the opposite pins on the longer sides. I still occasionally find it convenient to wire between 2 & 4 or 1 & 3.

@groundFungus: What tool do you use for your neat schematics please?

What tool do you use for your neat schematics please?

I use Express Sketch which comes with the EspressPCB software. I downloaded it years ago. Don't know how the newest is, but I love the older one that I have. I used to use OrCad but since I no longer work for a company I can't afford it.

Thanks. I’m using a program that’s probably even older, CircuitMaker Pro, but I’ll take a close look at Express Sketch.

Staying OT for a minute, do you also use the Fritzing app BTW, that I see mentioned frequently?

I have used Fritzing, but only to make simple PC boards* (I route them on my little CNC router). Fritzing will generate the Gerber files for free. Express PCB charges for Gerber files. I do not use the Fritzing breadboard view or schematic drawing. People don't like Fritzing breadboard views, generally, because they leave out a lot of information that would be on a good schematic.

*I find Eagle and KiCad too complex for the simple boards that I make.

@groundFungus,

Trying your sketch while learning about states.
LED is ON after start or reset. One press takes it OFF. Three more presses (four in total) takes it ON again.

Is that the intended behaviour? How would I change it to be OFF at the start, please, awaiting four presses? I tried taking the LED low during Setup but that didn't do it.

“ I tried taking the LED low during Setup but that didn't do it.”

@ Terrypin

When you try something and still have trouble, always show us ‘what’ you tried.


Use CTRL T to format your code.
Attach your ‘complete’ sketch between code tags

[code]Paste your sketch here[/code]

How would I change it to be OFF at the start, please

All of the LED action takes place in the last few lines of the program.

if (buttonPushCounter % 4 == 0)

Until the first button press, buttonPushCounter = 0. 0 % 4 = 0 so the LED is HIGH.
Unless the buttonPushCounter is greater than 0 do not turn the LED on.

if (buttonPushCounter % 4 == 0 && buttonPushCounter > 0)
   {
      digitalWrite(ledPin, HIGH);
      }
   
   else
   {
      digitalWrite(ledPin, LOW);
   }

groundFungus:
All of the LED action takes place in the last few lines of the program.

if (buttonPushCounter % 4 == 0)

Until the first button press, buttonPushCounter = 0. 0 % 4 = 0 so the LED is HIGH.
Unless the buttonPushCounter is greater than 0 do not turn the LED on.

if (buttonPushCounter % 4 == 0 && buttonPushCounter > 0)

{
     digitalWrite(ledPin, HIGH);
     }
 
  else
  {
     digitalWrite(ledPin, LOW);
  }

Thanks, understood.

groundFungus:
*I find Eagle and KiCad too complex for the simple boards that I make.

I have used Altium for years but since I am trying to avoid Windoze like the plague and I am now using Linux I have installed KiCad and for a freebie it is awesome. Maybe a bit complex for simple stuff but I make some not quite so simple stuff.

TheMemberFormerlyKnownAsAWOL:

buttonState = digitalRead(buttonPin);

In the 21st century, "buttonState" doesn't have global scope (it wouldn't be a Boolean either)

  • reply #3

Would bool buttonState = (bool) digitalRead(buttonPin); be more correct?

bool buttonState = digitalRead(buttonPin) == HIGH; Might be. Or maybe even "== LOW"

Is the issue bad form?

If my sketch has --
bool pbState;
and
pbState = digitalRead(pb);
then

if (pbState)

works [correlating 1, HIGH, true; 0, LOW, false)]