Fix debounce

src/config.cpp

@@ -11,3 +11,5 @@ Audio audio;
 Preferences preferences;
 
 long lastActionTime = 0;
+byte buttonLastState = HIGH;
+long startPress = NOT_PRESSED;

src/config.h

@@ -21,7 +21,8 @@
 #define BUTTON 33
 #define DEEP_SLEEP_WAKEUP GPIO_NUM_33
 
-#define DEBOUNCE_DELAY_MS 1000
+#define MIN_PRESS_DURATION 500
+#define NOT_PRESSED -1
 #define DEEP_SLEEP_DELAY_MS 3600000  // 1h
 
 // Screen
@@ -57,5 +58,7 @@ extern AsyncWebServer server;
 extern Audio audio;
 extern Preferences preferences;
 extern long lastActionTime;
+extern byte buttonLastState;
+extern long startPress;
 
 #endif

src/main.cpp

@@ -8,8 +8,6 @@
 #include "utils.h"
 #include "config.h"
 
-byte buttonLastState = HIGH;
-long lastDebounceTime = 0;
 bool wasRunning = false;
 
 void setup()
@@ -51,13 +49,7 @@ void setup()
 
 void loop()
 {
-  if ((millis() - lastDebounceTime) > DEBOUNCE_DELAY_MS)
+  checkButtonAndPlay();
-  {
-    byte buttonCurrentState = digitalRead(BUTTON);
-    if (buttonCurrentState == LOW && buttonLastState == HIGH)
-      play();
-    buttonLastState = buttonCurrentState;
-  }
 
   audio.loop();
 

src/utils.cpp

@@ -30,6 +30,21 @@ void displayText(String text)
   display.display();
 }
 
+void checkButtonAndPlay() {
+  byte buttonCurrentState = digitalRead(BUTTON);
+  if (buttonCurrentState == HIGH && buttonLastState == LOW) {
+    startPress = millis();
+  } else if (buttonCurrentState == LOW && buttonLastState == HIGH) {
+    startPress = NOT_PRESSED;
+  }
+  long pressDuration = millis() - startPress;
+  if (startPress != NOT_PRESSED && pressDuration >= MIN_PRESS_DURATION) {
+    play();
+    startPress = NOT_PRESSED;
+  }
+  buttonLastState = buttonCurrentState;
+}
+
 void play()
 {
   updateLastAction();

src/utils.h

@@ -9,6 +9,7 @@ bool fileIsValid(String fileName);
 bool fileExists(String fileName);
 void clearMessageArea();
 void displayText(String text);
+void checkButtonAndPlay();
 void play();
 
 void selectFile(String fileName);