1

enter image description hereHello guys, this is my code and I am basically doing if-else statement.

int led = D2;   
int but = D1;    

void setup() {
  pinMode(led, OUTPUT);      
  pinMode(but, INPUT);   

}

void loop() {
  if (digitalRead(but) == HIGH){
    digitalWrite(led, HIGH);
  }else{
    digitalWrite(led, LOW);
  }


}

I have also attached the photo so that you can diagnose the situation.

The led show only glow if the button is pressed. But the led keeps glowing for longer time and I don't know why. I think there is current leak somewhere in the circuit or the internal processing is slow. Please help me guys.enter image description here

  • The circuit seems like it's wrong. When a button is pressed it wil fluctuate some before it going to be stable. Try this: " bool state=digitalRead(but) digitalWrite(led, state); " also similar problem: arduino.stackexchange.com/questions/3425/… – Faux_Clef Sep 14 '18 at 13:21
2

Your button and pull-up configuration is not correct, so when the button is not being pressed, the pin D1 is floating, being pulled neither HIGH nor LOW.

Instead, use a pull-up resistor as shown. When the button is not pressed, D1 is connected to GND through a resistor. When the button is pressed, the 3.3V source has a direct path to D1.

schematic

simulate this circuit – Schematic created using CircuitLab

  • why pull-down? it is simpler to use the internal pullup – Juraj Sep 14 '18 at 16:55
  • Because the asker has coded for a pull-up. Yes, it’s trivial to change. – jose can u c Sep 14 '18 at 17:02

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service, privacy policy and cookie policy

Not the answer you're looking for? Browse other questions tagged or ask your own question.