Toggle Button Switch w\ Motor PWM

Hello all,

hope everything is doing well. I believe i need your help on trying to understand what is the best approach to achieve the following:

  • Using the Tutorial Debounce, i want to use the pushbutton as a Toggle Switch to light up the LED and at the same time run a motor using PWM.

Below the code i'm trying to use with minor changes to reflect this PWM feature. however, i'm not able to achieve what i desire.

// constants won't change. They're used here to set pin numbers:
const int buttonPin = 2;    // the number of the pushbutton pin
const int ledPin = 13;      // the number of the LED pin
int motorpin=5;
int vel=0;

// Variables will change:
int ledState = HIGH;         // the current state of the output pin
int buttonState;             // the current reading from the input pin
int lastButtonState = LOW;   // the previous reading from the input pin

// the following variables are unsigned longs because the time, measured in
// milliseconds, will quickly become a bigger number than can be stored in an int.
unsigned long lastDebounceTime = 0;  // the last time the output pin was toggled
unsigned long debounceDelay = 50;    // the debounce time; increase if the output flickers

void setup() {
  pinMode(buttonPin, INPUT);
  pinMode(ledPin, OUTPUT);
  pinMode(motorpin,OUTPUT);
  // set initial LED state
  digitalWrite(ledPin, ledState);
  analogWrite(motorpin,vel);
}

void loop() {
  // read the state of the switch into a local variable:
  int reading = digitalRead(buttonPin);

  // check to see if you just pressed the button
  // (i.e. the input went from LOW to HIGH), and you've waited long enough
  // since the last press to ignore any noise:

  // If the switch changed, due to noise or pressing:
  if (reading != lastButtonState) {
    // reset the debouncing timer
    lastDebounceTime = millis();
  }

  if ((millis() - lastDebounceTime) > debounceDelay) {
    // whatever the reading is at, it's been there for longer than the debounce
    // delay, so take it as the actual current state:

    // if the button state has changed:
    if (reading != buttonState) {
      buttonState = reading;

      // only toggle the LED if the new button state is HIGH
      if (buttonState == HIGH) {
        ledState = !ledState;
       //if i put here the PMW analogWrite, the result is that the motor runs without control once the state is high..
      }
      
    }
  }

  // set the LED:
  digitalWrite(ledPin, ledState);


  // save the reading. Next time through the loop, it'll be the lastButtonState:
  lastButtonState = reading;
}

Hope you can guide me on what is the best approach to solve this.

thanks in advance, Frederico

For button, you should use a pull-up or pull-down resistor to avoid the floating input problem. See Arduino - Button tutorial for detail.