Arduino 8路LED序列点亮故障排查：3颗LED完全不工作

问题描述 #

希望搭建一个电路，实现8颗LED从左到右每次点亮3颗的序列效果。已编写Arduino代码，但有3颗LED完全无法工作，请求排查原因。

原代码 #

int ledpin[] = {5,6,7,8,9,10,11,12};
int switchPinStatus = 0;
int delay1 = 50;

void setup() {
  pinMode(4, INPUT);
  for (int i = 0; i <= 7; i++)
    pinMode(ledpin[i], OUTPUT);
}

void sequence() {
  for (int i = 0; i <= 8; i++) {
    for (int j = i; j <= i + 2; j++) {
      analogWrite(ledpin[j], 100);
      delay(delay1);
    }
    analogWrite(ledpin[i], 0);
  }
  //for backwards
  for (int i = 7; i >= -1; i--) {
    for (int j = i; j >= i - 2; j--) {
      analogWrite(ledpin[j], 100);
      delay(delay1);
    }
    analogWrite(ledpin[i], 0);
  }
}

void loop() {
  switchPinStatus = digitalRead(4);
  while (switchPinStatus == 1) {
    for (int i = 0; i < 3; i++) {
      sequence();
      switchPinStatus = digitalRead(4);
    }
  }
}

故障原因分析 #

• 数组越界访问：sequence()函数中正向循环终止条件为i <= 8，当i=7、8时，j = i+2会超出ledpin数组的有效索引范围（0-7），访问到无效内存地址；反向循环终止条件为i >= -1，当i=-1时，j = i-2会出现负索引，同样属于越界操作。这种越界会导致程序操作非目标引脚，甚至引发异常，直接造成部分LED无法正常工作。
• 循环逻辑错误：正向循环应保证每次点亮的3颗LED都在8颗范围内，正确的终止条件应为i <= 5（此时i+2=7，对应最后一个LED）；反向循环应从i=7开始，到i=2结束（此时i-2=0，对应第一个LED），避免越界。

修复后的代码 #

int ledpin[] = {5,6,7,8,9,10,11,12};
int switchPinStatus = 0;
int delay1 = 50;

void setup() {
  pinMode(4, INPUT);
  for (int i = 0; i <= 7; i++)
    pinMode(ledpin[i], OUTPUT);
}

void sequence() {
  // 正向序列：从左到右每次亮3颗
  for (int i = 0; i <= 5; i++) {
    // 同步点亮当前起始位置的3颗LED
    for (int j = i; j <= i + 2; j++) {
      analogWrite(ledpin[j], 100);
    }
    delay(delay1);
    // 熄灭最左边的LED
    analogWrite(ledpin[i], 0);
  }
  
  // 反向序列：从右到左每次亮3颗
  for (int i = 7; i >= 2; i--) {
    // 同步点亮当前起始位置的3颗LED
    for (int j = i; j >= i - 2; j--) {
      analogWrite(ledpin[j], 100);
    }
    delay(delay1);
    // 熄灭最右边的LED
    analogWrite(ledpin[i], 0);
  }
  
  // 序列结束后关闭所有LED，避免残留点亮状态
  for (int i = 0; i <=7; i++) {
    analogWrite(ledpin[i], 0);
  }
}

void loop() {
  switchPinStatus = digitalRead(4);
  while (switchPinStatus == 1) {
    for (int i = 0; i < 3; i++) {
      sequence();
      switchPinStatus = digitalRead(4);
      // 中途开关断开则立即停止
      if(switchPinStatus != 1) break;
    }
  }
}

额外优化说明 #

• 调整延迟时机为同步点亮3颗LED后再延迟，保证视觉上的“同时点亮3颗”效果，原代码嵌套延迟会导致LED逐个点亮，不符合需求。
• 添加序列结束后的全LED关闭逻辑，避免残留点亮状态干扰下一次序列执行。
• 增加循环内的开关状态检查，确保开关断开时能及时终止序列。

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