KAmeleon-STM32L496ZGT6开发板LED跑马灯程序main函数优化问询

当然可以！我帮你把main函数优化得更清晰、易读，同时保持原有功能完全一致，而且完全不改动你写的setSingleLED函数。先看看优化后的代码，再给你拆解改动点：

int main(void) {
    HAL_Init();
    SystemClock_Config();
    MX_GPIO_Init();

    int current_led = 0;
    int direction = 1; // 1表示正向（0→7），-1表示反向（7→0）
    int joystick_triggered = 0; // 摇杆触发标志，防止单次按压多次切换方向

    while (1) {
        // 点亮当前指定的LED
        setSingleLED(current_led);

        // 延时期间检测摇杆状态（保留原循环逻辑，优化内部判断）
        joystick_triggered = 0;
        for (unsigned int i = 0; i < 5000; i++) {
            // 检测摇杆是否按下（GPIO_PIN_15为低电平表示触发）
            if (!HAL_GPIO_ReadPin(GPIOE, GPIO_PIN_15) && !joystick_triggered) {
                joystick_triggered = 1;
                direction *= -1; // 一键切换方向：1变-1，-1变1
            }
        }

        // 更新下一个LED编号，同时处理边界自动反转
        current_led += direction;
        if (current_led > 7) {
            current_led = 6;
            direction = -1;
        } else if (current_led < 0) {
            current_led = 1;
            direction = 1;
        }
    }
}

主要优化点说明： #

• 变量语义化：把current_diode改成current_led、dir改成direction、ischange改成joystick_triggered，变量用途一目了然，可读性大幅提升。
• 移除冗余变量：删掉了逻辑冗余的zmienna变量，直接通过current_led的边界值判断是否需要反转方向，逻辑更直接。
• 简化方向切换：摇杆按下时用direction *= -1替代原有的if-else判断，一行代码完成方向切换，简洁高效。
• 修正边界逻辑：原代码中出现current_diode >8的判断（但LED最大编号是7），优化后直接针对有效范围0-7做边界处理，避免无效LED编号出现。
• 结构分层清晰：把「点亮LED」「延时+按键检测」「更新LED编号」三个核心步骤拆分明确，后续维护或调整逻辑时更方便。

如果想要更精准的延时（原循环延时精度依赖CPU频率），可以换成HAL_Delay(10)这类硬件延时函数，同时在延时周期内定时检测按键——不过我保留了你原有的循环逻辑，只优化了内部判断逻辑。

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