LED闪烁模式控制程序的控制结构优化方案咨询

原代码的核心问题 #

• 递归调用main()：每次选择模式后重新进入主函数，反复执行wiringPiSetup()和引脚配置，不仅冗余还会持续消耗栈内存，最终导致程序崩溃
• 多分支else if结构：模式数量增加时，代码可读性和维护性会快速下降
• 无输入校验：用户输入非0-3的数字时，程序直接退出，没有错误提示和重新输入的逻辑

优化后的实现方案 #

核心改进点 #

• 用循环结构替代递归，彻底避免栈溢出问题
• 用switch-case替代多else if，分支逻辑更清晰
• 初始化代码（wiringPi设置、引脚配置）只执行一次，放在循环外部
• 增加输入合法性校验，处理非法输入和非数字输入的情况

优化代码示例 #

#include <stdio.h>
#include <wiringPi.h>

// 假设以下功能函数已实现
void turnOn();
void turnOff();
void blinking();
void pattern();

int main() 
{
    // 初始化仅执行一次
    if (wiringPiSetup() == -1) {
        printf("WiringPi初始化失败！\n");
        return 1;
    }
    pinMode(23, OUTPUT);
    pinMode(26, OUTPUT);
    
    int select;
    while (1) { // 循环接收输入，可按需添加退出逻辑
        printf("\n请选择LED模式：\n");
        printf("0 - 关闭LED\n");
        printf("1 - 开启LED\n");
        printf("2 - LED闪烁\n");
        printf("3 - 低占空比模式\n");
        printf("输入模式编号：");
        
        // 处理非数字输入
        if (scanf("%d", &select) != 1) {
            printf("输入无效，请输入数字！\n");
            // 清空输入缓冲区的无效数据
            while (getchar() != '\n');
            continue;
        }
        
        switch(select) {
            case 0:
                turnOff();
                break;
            case 1:
                turnOn();
                break;
            case 2:
                blinking();
                break;
            case 3:
                pattern();
                break;
            default:
                printf("无效编号，请输入0-3之间的数字！\n");
        }
    }
    return 0;
}

额外优化建议 #

• 可添加退出选项（比如输入4退出程序），避免程序一直处于循环状态
• 将模式编号定义为宏，提升代码可维护性：

  #define MODE_OFF 0
  #define MODE_ON 1
  #define MODE_BLINK 2
  #define MODE_LOW_DUTY 3
  #define MODE_EXIT 4
  

• 如果blinking()和pattern()是阻塞式函数，可考虑用线程实现，避免执行模式时无法响应新的输入

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