Arduino if语句调用void函数后LED常亮不执行后续代码问题咨询

问题根因 #

你的代码存在4个核心错误，共同导致了LED常亮、后续代码不执行的问题：

• 变量currentTime从未被赋值更新，全局定义后默认值始终为0，currentTime - previousTime的判断逻辑完全失效
• blink2函数的执行逻辑写反：你要的是「按下按键立刻亮LED，3秒后熄灭」，但现有代码是「按下后时间没到3秒就灭LED，时间到了才亮」，而且亮了之后没有任何退出逻辑，会一直卡在亮灯的分支里
• 函数互相递归调用：btnpress()调用blink2()，blink2()又反过来调用btnpress()，没有正常的函数返回流程，会导致栈溢出同时逻辑混乱
• 按键输入未做上下拉处理：使用INPUT模式如果没有外接下拉电阻，引脚状态浮空会触发误判

修复方案 #

1. 逻辑调整 #

去掉递归调用，改用非阻塞的状态记录逻辑：用一个变量记录LED是否需要点亮计时，正常在loop里循环判断按键、判断计时时间即可，不需要函数互相跳转。

2. 修正后代码 #

const int btnpin = 9;
int btnstate = 0;
unsigned long currentTime;
unsigned long previousTime;
const long period = 3000;
// 新增：记录LED是否处于点亮计时状态
bool isLedOn = false;

void setup()
{
    pinMode(LED_BUILTIN, OUTPUT);
    // 改用INPUT_PULLUP内置上拉，按键另一端接GND，按下为LOW，避免浮空
    pinMode(btnpin, INPUT_PULLUP);
    Serial.begin(9600);
    // 初始熄灭LED
    digitalWrite(LED_BUILTIN, LOW);
}

void loop()
{
    currentTime = millis(); // 每次循环都更新当前时间
    btnpress();
    // 检测LED计时是否到点
    if(isLedOn && (currentTime - previousTime >= period)){
        digitalWrite(LED_BUILTIN, LOW);
        Serial.println("Led off");
        isLedOn = false;
    }
}

void btnpress()
{
    Serial.println("Waiting button press");
    delay(500); // 这里的delay可以换成非阻塞消抖，不影响功能
    // 上拉模式下按下为LOW
    btnstate = digitalRead(btnpin);
    // 只有LED没在亮的时候才响应按键，避免重复触发
    if (btnstate == LOW && !isLedOn) {
        previousTime = millis();
        digitalWrite(LED_BUILTIN, HIGH);
        Serial.println("Led on");
        isLedOn = true;
    }
}

额外优化建议 #

• 把btnpress()里的delay(500)换成按键消抖逻辑，避免阻塞程序运行：可以记录上次按键检测的时间，只在间隔20ms以上的时候才读取按键状态
• 如果需要保留原有的函数跳转逻辑，不要用递归，blink2执行完直接return回到调用处即可，不要在blink2里再次调用btnpress

内容的提问来源于stack exchange，提问作者Type 111