嵌入式C中中断驱动变量通用Setter/Getter函数实现方案咨询

问题背景 #

我在嵌入式项目中通过全局变量实现中断功能：中断检测PWM信号占空比，若数值正常则将bool变量设为true，主程序通过该变量控制指定引脚输出高低电平。为模块化功能并限制变量作用域，我在独立.c文件中模拟“对象”，定义静态变量及访问、修改变量的方法，具体代码如下：

static bool var1 = false;
static bool var2 = false;

bool *var1_get(void) {
  return &var1;
}

bool *var2_get(void){
  return &var2;
}

void var_set(bool *var_ok) {
  /* signal PWM is good */
  *var_ok = true;
}
void var_reset(bool *var_ok) {
  /* signal PWM is bad */
  *var_ok = false;
}

中断服务程序中调用set/reset函数修改变量：

// capture_val is the PWM signal duty cycle value
if ((capture_val > 88) && (capture_val < 92)) {
  var_set(var1_get());
} else {
  var_reset(var1_get());
}

主程序while(1)循环中检查变量值以控制引脚输出：

if (*var1_get() == false) {
  HAL_GPIO_WritePin(GPIOC, OUT_OK_Pin, GPIO_PIN_RESET);
} else {
  HAL_GPIO_WritePin(GPIOC, OUT_OK_Pin, GPIO_PIN_SET);
}

我通过_get函数获取变量地址，在中断中修改其值，同时在主循环中读取值，存在以下疑问：

1. 当前实现方案是否合理？还是应改用main.c中的静态全局变量并舍弃该模块？
2. 我定义的*var_get()函数写法是否合规？有无更标准的写法或具备相同模块化功能的更优实现方案？

问题1：方案合理性分析 #

• 当前模块化方案的优势：
  • 实现了封装隔离：静态变量被限制在独立.c文件内部，避免了全局变量的命名污染，也防止其他模块直接篡改变量值，符合嵌入式开发的模块化设计原则。
  • 逻辑解耦：把PWM检测的状态管理逻辑独立出来，后续扩展更多检测通道（如var2）时，只需在该模块内添加对应逻辑，主程序和中断的调用逻辑无需大幅改动，维护性更强。
• 不建议改用main.c静态全局变量的原因：
  • 会导致状态管理与主程序逻辑耦合，后续修改PWM检测规则或添加新通道时，都要改动main.c，违反单一职责原则。
  • main.c作为程序入口，通常负责硬件初始化、主循环调度等核心逻辑，加入状态变量会让代码臃肿，可读性和可维护性下降。
• 结论：当前模块化方案更合理，应当保留。

问题2：var_get()写法及更优实现 #

• 现有写法的合规性与隐患：
  语法上var1_get()返回静态变量指针是合法的，但存在安全漏洞：任何拿到指针的模块都可以直接修改变量值，绕过var_set()和var_reset()的封装逻辑。若后续要在状态变更时添加临界区保护、日志记录等扩展逻辑，直接修改变量的操作会让这些逻辑失效。

• 更标准的实现方案：

  1. 完全封装状态，提供专属读写接口：
     不再返回变量指针，而是为每个状态变量提供独立的get/set/reset函数，彻底屏蔽内部实现。示例：

     // 添加volatile修饰，防止编译器优化导致读写不同步
     static volatile bool var1 = false;
     static volatile bool var2 = false;
     
     bool var1_get(void) {
       return var1;
     }
     
     void var1_set(void) {
       var1 = true;
     }
     
     void var1_reset(void) {
       var1 = false;
     }
     
     // 同理实现var2的接口
     bool var2_get(void) {
       return var2;
     }
     
     void var2_set(void) {
       var2 = true;
     }
     
     void var2_reset(void) {
       var2 = false;
     }
     

     中断调用简化为：

     if ((capture_val > 88) && (capture_val < 92)) {
       var1_set();
     } else {
       var1_reset();
     }
     

     主循环调用简化为：

     if (!var1_get()) {
       HAL_GPIO_WritePin(GPIOC, OUT_OK_Pin, GPIO_PIN_RESET);
     } else {
       HAL_GPIO_WritePin(GPIOC, OUT_OK_Pin, GPIO_PIN_SET);
     }
     

     这种写法的核心优势是：外部只能通过指定接口操作状态，保证了状态修改的可控性，后续扩展逻辑（如临界区、回调）都可以在接口内实现，不影响调用方。

  2. 添加跨上下文访问保护：
     由于变量在中断（中断上下文）和主循环（线程上下文）中跨域访问，即使是bool类型，为了保证兼容性和原子性，建议在状态修改接口中添加临界区：

     void var1_set(void) {
       __disable_irq(); // 关闭全局中断
       var1 = true;
       __enable_irq();  // 恢复全局中断
     }
     

     若使用RTOS，可替换为互斥锁，但裸机环境下开关中断是最直接的方案。

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