refactor(UI): 记录页改注入式 reader,layer-deps 基线 6->3(道B task-6)
B-6 分层收紧(UI_数据供给设计说明.md §7.2 契约原样落地):
- ui_w_records:删 #include "sys_log.h",改 ui_rec_reader_t 只读函数表 +
ui_w_records_bind() 模块级注入(sys_log 读 API 本为全局,函数表共享、
实例读盘缓存仍各自独享——切页陈旧行防御不变);log_id 换不透明编号
UI_W_REC_LOG_RUN/SELFTEST;缓冲容量宏本地化 UI_W_REC_TS/DESC_LEN
(20/29,与 sys_log 契约下限同宽;读失败退化空行防御保留)
- ui_page_runlog / ui_page_selftest:去 sys_log include,实例改不透明编号
- 两端薄包装适配层(编号→syslog_id_t 显式映射,错误码 0/-1/-2 直通不改写):
App/task/ui_task.c 与 simulator/main.c(无头+窗口两个 init 点),均在
ui_menu_init 之前 ui_w_records_bind;未绑定窗口组件按空态防御不读野
- scripts/check_layer_deps.py:BASELINE 6→3(--prune 实测恰 3 条 UI 记录
边消失后写回;剩 NETCFG×2 + lftpd_io 与 UI 无关)——道B 独占收紧口径
- 文档同步:门禁说明 layer-deps.md §1/§6/§9 与 README 计数 6→3;
UI_数据供给设计说明.md §7.2 加落地注记、§14"计划削减"改"已兑现"
- 首提被 id-check 拦下:注释/文档里自造"批卡编号"(B-6/道B-3)命中
"未登记前缀"增量拦截——已全数改回 task-6/道B 普通指称(编号不回收、
禁自造,教训入本提交);id-check 复跑 RC=0(S10 告警为存量非本改动)
验证:
- 模拟器 0 警告;菜单 8 状态 + 内容页 11 页黄金参考逐字节 PASS
(记录页数据流经 reader 后像素与基线一致,无基线更新提交)
- cppcheck/coding-standard/bsp-err/font-charset/no-mojibake 单跑全绿;
layer-deps 复跑:6 处反向 include = 豁免 3 / 基线内 3 / 新增 0,
--prune 显示"基线无需收紧(3 条存量全部仍成立)"
- doc-links --all 的 35 处告警全为本仓检出路径含空格误扫 SDL2 自带 md
(环境问题、非本改动引入;提交门禁按暂存文件名过滤不受影响)
- @build 0 Error 0 Warning(终树复跑 BUILD_OK);uvprojx 无 UV4 抖动、
无新文件不涉注册;其余 38 钩全绿(ui-golden/font-charset/coding-standard/
bsp-err/id-check/cppcheck…单跑复验 RC=0)
增量(A/B 同树 .map,基线 = 合入 c1b425d 后 HEAD):
- Flash:Code +104 B、RO +8 B(两端薄包装代码 + reader 函数表 const)
- RAM:RW +4 B(s_reader 指针)/ ZI +4 B(两页实例 struct 枚举改 uint8
本应 -8 B,链接段对齐后净 +4 B);RW+ZI 合计 +8 B
gate: only mem-budget/mem-numbers pending lead ledger backfill
- pre-commit run --all-files 实测仅这两条失败(台账/RAM 文档 vs .map 漂移,
根因即上列 Code +104/RO +8/RW +4/ZI +4),其余 38 钩全绿;按 Lead 裁决
(task-4 口径"以后照此办理",含 Flash/RAM 漂移同例外)--no-verify 提交,
合并后请主树回填台账(scripts/ram_baseline.py --write 由 Lead 侧执行)
- 手工自查通过:5 个 BOM 文件 BOM=EF BB BF 齐全、docs/py/main.c 保持无 BOM,
全部 10 文件 CR=0、U+FFFD=0
- 台账增量供回填:Flash Code +104 B、RO +8 B;RAM RW +4 B、ZI +4 B
This commit is contained in:
@@ -333,6 +333,10 @@ void ui_w_records_bind(const ui_rec_reader_t *reader); /* App 启动时注入
|
||||
|
||||
- `log_id` 用 `uint8_t`(**不透明编号**),BSP 不知道 `SYSLOG_RUN` 是什么,只知道"0 号日志"。
|
||||
- 注入点在 `App/task/ui_task.c`(`ui_menu_init` 之前),实现体是 `sys_log` 的两个薄包装。
|
||||
- 【2026-10-25 落地注】模拟器同形镜像:`simulator/main.c` 的 `sim_rec_count/sim_rec_get`,
|
||||
无头与窗口两个初始化点都在 `ui_menu_init` 前调 `ui_w_records_bind`;UI 侧编号常量
|
||||
`UI_W_REC_LOG_RUN/SELFTEST`(映射只在两端适配层,UI 不感知 `syslog_id_t`)。
|
||||
`ui_rec_reader_t` 按本文契约**原样**实现(`int` 错误码 0/-1/-2 未改型)。
|
||||
|
||||
### 7.3 收益:可削减 3 条既有基线
|
||||
|
||||
@@ -489,7 +493,7 @@ typedef struct {
|
||||
|
||||
## 14. 与既有文档和门禁的关系
|
||||
|
||||
- **门禁**:`scripts/check_layer_deps.py` 的三种合法做法本文全部用上(快照推送=②,写路径与记录 reader=①),**不新增任何 `ALLOW` 或 `BASELINE` 条目**,并计划削减 3 条。
|
||||
- **门禁**:`scripts/check_layer_deps.py` 的三种合法做法本文全部用上(快照推送=②,写路径与记录 reader=①),**不新增任何 `ALLOW` 或 `BASELINE` 条目**;计划的 3 条削减**已于 2026-10-25 兑现**(task/UI-rec-reader 落地 §7.2 契约,基线 6 → 3,详见 `docs/门禁说明/layer-deps.md` §1)。
|
||||
- **页面框架**:`docs/UI设计/内容页架构.md` 的 `refresh` / `count_cb` 钩子**签名不变**,页面只是改为从 `ui_data` 取数。
|
||||
- **页面清单**:`docs/UI设计/页面补齐规划.md` 的"批 1"第一项即本文 步骤1~步骤3。
|
||||
- **数据真实性**:`docs/问题记录/采集与数据处理短板评测.md` 的标定(§4.1)与遥测链(§4.2)落地后,只需改 `ui_data_feed` 的取数来源与 `eng` 取值,页面零改动。
|
||||
|
||||
+1
-1
@@ -16,7 +16,7 @@
|
||||
|---|---|---|---|
|
||||
| **改写型** | 不报错,直接改你的文件 | `trailing-whitespace`、`end-of-file-fixer` | 重新 `git add` 被改的文件再提交(它改的是工作区,不是索引) |
|
||||
| **硬拦截** | 违反即失败,没有存量豁免 | `cppcheck`、`coding-standard`、`doc-links`、`no-mojibake`、`mem-budget`、`ui-golden`、`font-charset`、`lv-conf-sync`、`task-table`、`mem-numbers`、`registration-check`、`cubemx-survival`、`test-suite-switch`、16 个主机单测、官方 5 个、`gitlint` | 只能改代码或改文档;**不要**找豁免开关 |
|
||||
| **增量拦截** | 规则恒生效,但存量钉在基线/台账里,只许减不许增 | `layer-deps`(6 条)、`bsp-err`(38 条)、`id-check`(`scripts/id_known_backlog.json` 台账) | 报错先分清是**新增**还是**存量搬家**:`--verbose` 看归类,`--prune` 看已还清 |
|
||||
| **增量拦截** | 规则恒生效,但存量钉在基线/台账里,只许减不许增 | `layer-deps`(3 条,2026-10-25 还清 UI 记录 3 条)、`bsp-err`(38 条)、`id-check`(`scripts/id_known_backlog.json` 台账) | 报错先分清是**新增**还是**存量搬家**:`--verbose` 看归类,`--prune` 看已还清 |
|
||||
|
||||
⚠ 增量拦截最容易误会的一点:**绿灯不等于没问题**,只等于"没有比上次更差"。
|
||||
每个增量拦截门禁的基线数字都写进它自己的文档,读那篇时第一眼看的就是"现在在册多少条"。
|
||||
|
||||
+10
-6
@@ -20,9 +20,12 @@
|
||||
`App/log/console_log.h`(日志后端,BSP 的 `dbg_log.h` 本来就建在它上面)、
|
||||
`App/util/sys_clock.h`(时间源,日志时间戳的唯一提供者)、`App/util/sha256.h`(纯算法、无状态)。
|
||||
|
||||
**`BASELINE` 六条(2026-10-22 清点,只减不增)**:`NETCFG/net_cfg.c`、`NETCFG/ftp_cfg.c`
|
||||
→ `App/cfg/cfg_file.h`(配置引擎属主是 defaultTask);`NET/lftpd/lftpd_io.c` → `App/fs/vol_lock.h`;
|
||||
`UI/ui_page_runlog.c`、`UI/ui_page_selftest.c`、`UI/ui_w_records.h` → `App/log/sys_log.h`。
|
||||
**`BASELINE` 现存三条(2026-10-22 清点六条 → 2026-10-25 道B 还清 UI 记录三条 → 现三条;只减不增)**:
|
||||
`NETCFG/net_cfg.c`、`NETCFG/ftp_cfg.c` → `App/cfg/cfg_file.h`(配置引擎属主是 defaultTask);
|
||||
`NET/lftpd/lftpd_io.c` → `App/fs/vol_lock.h`。
|
||||
已还清:`UI/ui_page_runlog.c`、`UI/ui_page_selftest.c`、`UI/ui_w_records.h` → `App/log/sys_log.h`
|
||||
(2026-10-25 task/UI-rec-reader:`ui_w_records` 改注入式 `ui_rec_reader_t`,契约见
|
||||
`docs/UI设计/UI_数据供给设计说明.md` §7.2,绑定在 `App/task/ui_task.c` 与 `simulator/main.c`)。
|
||||
修法是**注入 / 传参**(上层把数据或回调递进 BSP),不是把上层服务搬进 BSP。
|
||||
|
||||
## 2. 为什么要它
|
||||
@@ -69,8 +72,8 @@
|
||||
## 6. 限制与盲区
|
||||
|
||||
1. **只查直接 include,查不到传递依赖**。BSP 的 `a.h` 里 include 了 App 头,BSP 的 `b.c`
|
||||
只 include `a.h`——`b.c` 这条边不出现,**但 `a.h` 那条会报**。基线里 `ui_w_records.h` 的
|
||||
注释专门写着"(组件头,会外溢到 include 链)",就是提醒这种外溢要人来看。
|
||||
只 include `a.h`——`b.c` 这条边不出现,**但 `a.h` 那条会报**。原基线 `ui_w_records.h` 的
|
||||
注释专门写着"(组件头,会外溢到 include 链)"(该条 2026-10-25 已还清),就是提醒这种外溢要人来看。
|
||||
2. 只认 `#include "引号"` 形式;`#include <App/...>` 这类写法不在管辖内(也违反 include 规范,
|
||||
由 `coding-standard` 的 CODE-RULE-11 兜)。
|
||||
3. **按 basename 匹配**:如果 `App/` 与别处存在同名头,会按同名候选放宽(先精确后同名),
|
||||
@@ -80,7 +83,7 @@
|
||||
`--prune` 也就不会提示你收紧基线。
|
||||
6. `git` 不可用时退回 `os.walk`(跳过 `.git/.venv/build`)——两条路径的清单可能不同,
|
||||
在无 git 环境下跑出的"通过"不能当证据。
|
||||
7. 绿灯的含义只是"没有新增反向边":6 条存量债务仍然在那里。
|
||||
7. 绿灯的含义只是"没有新增反向边":现存 3 条存量债务仍然在那里(2026-10-25 还清 3 条)。
|
||||
|
||||
## 7. 豁免与基线
|
||||
|
||||
@@ -97,3 +100,4 @@
|
||||
| 日期 | 变更 |
|
||||
|---|---|
|
||||
| 2026-10-23 | 首篇逐项文档(在册 6 条基线逐条列出,含每条的属主背景) |
|
||||
| 2026-10-25 | 道B(task/UI-rec-reader)还清 UI 记录 3 条:`ui_w_records` 注入式 reader(§7.2 契约落地),基线 6 → 3 |
|
||||
|
||||
Reference in New Issue
Block a user