ISR中使用结构体封装全局变量的可读性问题及替代方案咨询

如果不想调整现有架构，只需要简化嵌套成员的访问，可以在访问的函数内定义局部指针别名：

/* SOURCE FILE 内的按键处理代码 */
state *btn_state = &DB.buttonState;
btn_state->new = BTN_PORT;
btn_state->change = btn_state->new ^ btn_state->old;
btn_state->old = btn_state->new;

不需要修改结构体定义，只需要在频繁访问嵌套结构体的代码段加一行别名定义，就能达到和直接访问变量差不多的简洁度，编译后不会产生额外开销。

你当前的封装方式并没有实现“防止其他无关源文件修改全局变量”的效果，因为结构体实例DB定义在头文件中，所有引用该头文件的源文件都可以直接修改它的成员。
要实现真正的访问限制，调整方式如下：

1. 头文件仅对外暴露操作接口声明，不暴露内部变量和结构体定义：

/*** HEADER FILE ***/
#include <stdint.h>
// 仅对外提供函数声明，不暴露内部数据结构
void debounce_isr_update(void);
uint16_t get_button_change(void);
uint32_t get_debounce_tick(void);

2. 源文件内将所有内部变量定义为static，限制仅当前文件可访问：

/*** SOURCE FILE ***/
#include "debounce.h"

typedef struct {
    uint16_t old;
    uint16_t new;
    uint16_t change;
} state;

static state buttonState;
static uint16_t isPortStable;
static uint16_t countStableStates[BTN_COUNT];
static uint32_t tick;

void debounce_isr_update(void) {
    // 这里直接访问static变量，不需要嵌套结构体，代码非常简洁
    buttonState.new = BTN_PORT;
    buttonState.change = buttonState.new ^ buttonState.old;
    buttonState.old = buttonState.new;
    // 其余消抖逻辑
}

uint16_t get_button_change(void) {
    // 外部要读状态只能通过这个接口，你可以在这里加合法性校验、临界区保护
    return buttonState.change;
}

// 其余接口实现

这种方式从编译层面限制了其他源文件无法直接访问消抖模块的内部变量，完全避免了意外修改的可能，同时内部代码不需要嵌套访问结构体，可读性更高。

后续新增ISR时，每个ISR对应的逻辑单独做成一个模块，每个模块自己管自己的static内部变量，仅对外暴露必要的操作接口。全局作用域不会出现任何全局变量，完全避免命名冲突和意外修改的问题。

非常建议结合使用：

• static用来限制变量的作用域，是嵌入式场景下避免全局变量滥用的核心手段，只要不需要跨文件访问的变量都应该定义为static。
• 指针可以用来做局部别名简化长路径变量的访问，也可以用来在多个模块间传递数据句柄，只要注意访问安全即可。

额外注意：所有ISR和主程序都会访问的变量，需要加volatile修饰，同时访问时要做好临界区保护（比如关中断），避免出现数据一致性错误。

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