FastLED与简易计数器配合异常问题求助

我来帮你排查这个问题——你遇到的计数器跳变、LED随机触发等异常，根本原因不是FastLED的bug，而是代码里的数组越界访问和按键防抖缺失导致的内存破坏，咱们一步步来解决：

问题复现与症状 #

你遇到的具体异常表现：

• 按下按键无反应
• LED状态切换正常，但计数器出现多种故障：
  • 跳至-255，之后在1和-255之间反复切换
  • 计数器无法正常递增
  • 未触碰按键时，计数器/LED随机触发状态变化

附上你提供的原始代码：

#include <FastLED.h>
#define AnalogIn A0
#define SwIn 2
#define LED_Out 12
#define NUM_LEDS 100
int pushCounterz = 0;
int buttonState;
int lastButtonState; // the previous reading from the input pin
int ledPin = 13;
int ledState = HIGH;
CRGB leds[NUM_LEDS];
void setup() {
 // put your setup code here, to run once:
 FastLED.addLeds<WS2812, LED_Out, GRB>(leds, NUM_LEDS);
 //pinMode(SwIn, INPUT);
 pinMode(LED_Out, OUTPUT);
 pinMode(ledPin, OUTPUT);
 //Turn Off strip
 for (int i = 0; i <= NUM_LEDS; i++) {
 leds[i] = CRGB ( 255, 0, 0 );
 FastLED.show();
 }
 digitalWrite(ledPin, ledState);
 Serial.begin(115200);
 Serial.println(pushCounterz);
 lastButtonState = digitalRead(SwIn); // Set the button state to the startup state
}
void loop() {
 buttonState = digitalRead(SwIn);
 if (buttonState == LOW && buttonState != lastButtonState) {
 ledState = !ledState;
 }
 if (buttonState == LOW && buttonState != lastButtonState) {
 if (pushCounterz > 3) {
 Serial.println("Reset to 0: ");
 pushCounterz = 0;
 } else {
 pushCounterz = pushCounterz + 1;
 Serial.println("Incerment");
 }
 Serial.println(pushCounterz);
 switch (pushCounterz) {
 case 0:
 for (int i = 0; i <= NUM_LEDS; i++) {
 leds[i] = CRGB (255, 0, 0);
 }
 break;
 case 1:
 for (int i = 0; i <= NUM_LEDS; i++) {
 leds[i] = CRGB ( 0, 255, 0);
 }
 break;
 case 2:
 for (int i = 0; i <= NUM_LEDS; i++) {
 leds[i] = CRGB ( 0, 0, 255);
 }
 break;
 case 3:
 // theaterChaseRainbow(1,50);
 break;
 default:
 for (int i = 0; i <= NUM_LEDS; i++) {
 leds[i] = CRGB ( 0, 0, 0);
 }
 break;
 }
 }
 FastLED.show();
 digitalWrite(ledPin, ledState);
 lastButtonState = buttonState;
}

核心原因分析 #

1. 数组越界访问（最关键）：
   你定义了CRGB leds[NUM_LEDS];（NUM_LEDS=100），数组的有效索引范围是0~99，但所有LED循环都用了i <= NUM_LEDS，这会让i跑到100，访问leds[100]——这是数组外的内存区域，刚好可能覆盖到pushCounterz的内存空间，导致计数器数值被破坏（比如变成-255，这是内存乱码的典型表现）。这也是为什么禁用switch块后恢复正常：因为switch里的循环不再越界写内存了。

2. 按键防抖缺失：
   机械按键按下/松开时会产生抖动，导致digitalRead多次触发高低电平变化，造成计数器被多次递增或异常切换。

3. 重复的条件判断：
   你写了两个完全相同的if (buttonState == LOW && buttonState != lastButtonState)判断，虽然逻辑上没问题，但会增加代码冗余，也容易引发误触发。

修复步骤与完整代码 #

针对以上问题，我修改了代码，每处修改都加了注释：

#include <FastLED.h>
#define AnalogIn A0
#define SwIn 2
#define LED_Out 12
#define NUM_LEDS 100
// 改用unsigned int避免负数值，更符合计数器使用场景
unsigned int pushCounterz = 0;
int buttonState;
int lastButtonState;
int ledPin = 13;
int ledState = HIGH;
CRGB leds[NUM_LEDS];

void setup() {
  FastLED.addLeds<WS2812, LED_Out, GRB>(leds, NUM_LEDS);
  // 设置按键为INPUT_PULLUP，无需外部上拉电阻，默认高电平，按下变LOW
  pinMode(SwIn, INPUT_PULLUP);
  pinMode(LED_Out, OUTPUT);
  pinMode(ledPin, OUTPUT);
  
  // 修复数组越界：i < NUM_LEDS（有效索引0~99）
  for (int i = 0; i < NUM_LEDS; i++) {
    leds[i] = CRGB(255, 0, 0);
  }
  FastLED.show(); // 将show移到循环外，避免重复刷新浪费资源
  
  digitalWrite(ledPin, ledState);
  Serial.begin(115200);
  Serial.println(pushCounterz);
  lastButtonState = digitalRead(SwIn);
}

void loop() {
  buttonState = digitalRead(SwIn);
  
  // 按键防抖：检测到状态变化后延时消抖，再次读取确认
  if (buttonState != lastButtonState) {
    delay(50); // 50ms足够过滤机械抖动
    buttonState = digitalRead(SwIn);
  }
  
  // 合并重复的条件判断，只在按键按下且状态变化时执行逻辑
  if (buttonState == LOW && buttonState != lastButtonState) {
    ledState = !ledState;
    
    if (pushCounterz > 3) {
      Serial.println("Reset to 0: ");
      pushCounterz = 0;
    } else {
      pushCounterz++; // 简化递增写法
      Serial.println("Increment"); // 修正拼写错误
    }
    Serial.println(pushCounterz);
    
    switch (pushCounterz) {
      case 0:
        for (int i = 0; i < NUM_LEDS; i++) {
          leds[i] = CRGB(255, 0, 0);
        }
        break;
      case 1:
        for (int i = 0; i < NUM_LEDS; i++) {
          leds[i] = CRGB(0, 255, 0);
        }
        break;
      case 2:
        for (int i = 0; i < NUM_LEDS; i++) {
          leds[i] = CRGB(0, 0, 255);
        }
        break;
      case 3:
        // theaterChaseRainbow(1,50);
        break;
      default:
        for (int i = 0; i < NUM_LEDS; i++) {
          leds[i] = CRGB(0, 0, 0);
        }
        break;
    }
    FastLED.show(); // 只在状态变化时刷新LED，提升运行效率
  }
  
  digitalWrite(ledPin, ledState);
  lastButtonState = buttonState;
}

关键修改点说明： #

1. 修复数组越界：所有循环条件从i <= NUM_LEDS改为i < NUM_LEDS，确保只访问数组的有效内存区域。
2. 添加按键防抖：检测到按键状态变化后，延时50ms再次读取，过滤机械抖动的干扰。
3. 优化计数器类型：把int pushCounterz改为unsigned int，避免出现负数值，更符合计数器的使用场景。
4. 合并重复判断：将两个相同的if条件合并，减少冗余，逻辑更清晰。
5. 优化LED刷新时机：把FastLED.show()移到状态变化的代码块内，避免每次loop都刷新，提升运行效率。
6. 修正拼写错误：把Incerment改为正确的Increment。

验证效果 #

修改后，你应该不会再遇到计数器跳变到-255、随机触发的问题，按键按下时计数器会稳定递增，LED状态也能正常切换。

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