docs(review): 清理过时注释并校正项目记忆中的构建/目录说明
- PlotView.hpp：删除“已迁移到数据层的公式包络缓存”这个指向已删代码的注释块。 - agent_mem.md §2：修正 -Os -> -Oz；补充体积相关开关（NO_EXCEPTIONS / NO_RTTI / LTO / RELEASE_ONLY_OPT）与“改 Makefile 必须删 .o 全量重编”的提醒； 链接库列表补 msimg32；忽略列表补 Temp/、.preflight/、Tools/。 - agent_mem.md §3：目录结构按现状重写（Shared/ 已承载通用控件与 PopupMenu/ ColorPickerPopup；Utilities 现为数据与格式层：DataModel/DataLoader/CsvLoader/ Mf4Loader/MatLoader/MatH5/Deflate/FormulaEval/Utf8Util/MemProbe； 删掉早已不存在的 CsvTable/、DataModel/、PaintUtil/ 等旧条目），并写明依赖方向。 - agent_mem.md 末尾新增第 33 轮（本轮审查）记录。 - Code_Review_Progress.md 追加 P15（注释/文档校正）。
This commit is contained in:
DeepSeek-V4.1-Flash committed
1 parent
8437100586
commit
f13944034a
3 files changed
+121
-26
No files matched your search
@@ -101,5 +101,22 @@ map 中 `cp-demangle.o` 已无 `.text` 段分配。
|
||||
且崩溃会弹模态框），纯逻辑重写无法自证“像素完全一致”。已通过上面的派发收敛
|
||||
消除最容易出错的一类重复，剩余重复属于“几何算式本身”，风险高于收益。
|
||||
|
||||
### P11 拆分 `ApplicationCommands.cpp` —— ✅ 完成（提交 `8437100`）
|
||||
|
||||
原文件 **2,165 行**，混杂六类职责。按职责**纯搬移**（零逻辑改动）：
|
||||
|
||||
| 新文件 | 行数 | 内容 |
|
||||
| --- | --- | --- |
|
||||
| `ApplicationTabs.cpp` | 355 | 标签页新建/关闭/切换/重排、视图同步、窗口标题、标签栏回调、标签页右键菜单 |
|
||||
| `ApplicationDerive.cpp` | 786 | 派生烘焙、重采样、选区截取（新建/就地）、克隆（含带调整） |
|
||||
| `ApplicationMenus.cpp` | 409 | 绘图区右键菜单、菜单数值回写、主题切换、设置菜单、菜单结果派发 |
|
||||
| `ApplicationCommands.cpp` | 642 | 分组/信号增删、移动与重排、文件对话框、工具栏动作、键盘鼠标分发、布局边界绘制 |
|
||||
|
||||
Makefile 按目录通配收集源文件，**无需登记新文件**。
|
||||
|
||||
**验证**：release 零 error 零 warning；产物 793,088 → 794,112 B（+1,024，LTO 内联边界
|
||||
变化，< 0.13%）。
|
||||
|
||||
|
||||
|
||||
|
||||
+104
-19
@@ -20,13 +20,20 @@
|
||||
- 工具链：**MinGW-w64**（`x86_64-win32-seh-rev0`, MinGW-Builds 16.1.0），可直接用 `make`。
|
||||
- 构建入口：根目录 `Makefile`（C / C++ / windres 混合构建）。
|
||||
- 常用命令：
|
||||
- `make release` → `Build/Release/Target.exe`（`-Os`，strip：`-s`）。
|
||||
- `make debug` → `Build/Debug/Target.exe`（`-O0 -g`）。
|
||||
- `make release` → `Build/Release/Target.exe`（`-Oz` + LTO + `-s`，strip）。
|
||||
- `make debug` → `Build/Debug/Target.exe`（`-O0 -g`，**不** strip）。
|
||||
- `make` → 同时构建 Release 与 Debug。
|
||||
- `make clean` → 删除 `Build/`。
|
||||
- 关键编译设置：`-std=c++17`；`-finput-charset=UTF-8 -fexec-charset=GBK -fwide-exec-charset=UTF-16LE`；`-static`；`-mwindows`（GUI）；链接 `gdi32 gdiplus comdlg32 d3d11 dwmapi d3dcompiler`。
|
||||
- **新增组件/工具目录后，必须在 `Makefile` 的 `CXX_INCLUDES_PATHS` 与 `CXX_SOURCES_PATHS` 中登记**（源文件按目录通配自动收集）。
|
||||
- 构建产物统一在 `Build/`，已由 `.gitignore` 忽略；同时忽略 `*.o/*.d/*.lst/*.map/*.exe` 与 `TestData/`。
|
||||
- 关键编译设置：`-std=c++17`；`-finput-charset=UTF-8 -fexec-charset=GBK -fwide-exec-charset=UTF-16LE`；
|
||||
`-static`；`-mwindows`（GUI）；链接 `gdi32 gdiplus msimg32 comdlg32 d3d11 dwmapi d3dcompiler`。
|
||||
- 体积相关开关（Makefile 顶部，**改这些必须删 `*.o` 全量重编，Makefile 变更不会自动触发重编**）：
|
||||
- `NO_EXCEPTIONS = 1` → `-fno-exceptions`。
|
||||
- `NO_RTTI = 1` → `-fno-rtti`（全仓无 `dynamic_cast`/`typeid`；见 `MemProbe.cpp` 里对
|
||||
`__gnu_cxx::__verbose_terminate_handler` 的覆盖，它同时省掉 56 KB 的 `cp-demangle`）。
|
||||
- `LTO = 1` → `-flto -flto-partition=one`。
|
||||
- `RELEASE_ONLY_OPT` = `-fno-asynchronous-unwind-tables`（**只**加在 Release 规则上）。
|
||||
- **新增组件/工具目录后，必须在 `Makefile` 的 `CXX_INCLUDES_PATHS` 与 `CXX_SOURCES_PATHS` 中登记**（源文件按目录通配自动收集；只加 `.cpp` 文件到已登记目录时无需改 Makefile）。
|
||||
- 构建产物统一在 `Build/`，已由 `.gitignore` 忽略；同时忽略 `*.o/*.d/*.lst/*.map/*.exe`、`TestData/`、`Temp/`、`.preflight/`、`Tools/`。
|
||||
|
||||
---
|
||||
|
||||
@@ -34,27 +41,42 @@
|
||||
|
||||
```
|
||||
Components/ 功能组件块（每块 Inc/ 放 .hpp，Src/ 放 .cpp）
|
||||
Application/ 应用主框架：窗口、布局、消息分发、标签页、绘制合成
|
||||
Application/ 应用主框架：窗口、布局、消息分发、绘制合成
|
||||
（分片：Application.cpp / ApplicationTabs.cpp / ApplicationDerive.cpp /
|
||||
ApplicationMenus.cpp / ApplicationCommands.cpp / ApplicationLoading.cpp）
|
||||
Main/ 程序入口（WinMain → Application::Run）
|
||||
Shared/ 与业务无关的通用层：UiComponent（顶层组件协议）、UiWidget（控件树）、
|
||||
UiLayout / UiBox / UiSurface / UiButtonWidget / UiControls、
|
||||
PaintUtil（低层绘制原语）、Theme、TextEdit、PopupMenu、ColorPickerPopup、
|
||||
UiDebugBounds（「显示布局边界」唯一消费者）
|
||||
TabBar/ 顶部多标签栏
|
||||
Toolbar/ 顶部工具栏（导入文件 / 适应视图 / 时间轴自适应 / Y 轴自适应 / 选区放大 / 光标 / 合并·拆分·清空网格）
|
||||
SignalPanel/ 左侧信号面板：按文件分组的信号树、可见性勾选、样式编辑
|
||||
Toolbar/ 顶部工具栏（设置 / 导入文件 / 适应视图 / Y 轴自适应 / 选区放大 / 光标 /
|
||||
合并·拆分·清空网格）
|
||||
SignalPanel/ 左侧信号面板：按分组信号树、可见性勾选、样式与公式编辑、拖拽排序
|
||||
PlotView/ 波形绘图区：网格、时间轴、光标、缩放、拖拽、实验性功能面板
|
||||
Utilities/ 工具与辅助模块（Inc/ + Src/）
|
||||
Application/ ... (见上)
|
||||
CsvTable/ CSV 流式解析（列优先）
|
||||
DataModel/ 数据层：DataSet / DataFile / SignalData / 显示配置
|
||||
PaintUtil/ GDI+ 绘制与文本/控件辅助（含 UiPointInRect、UiMeasureTextWidth 等）
|
||||
PopupMenu/ 主题化自绘弹出菜单（含二级子菜单、内联编辑）
|
||||
ColorPickerPopup/ 浮动拾色器
|
||||
TextEdit/ 内联文本编辑模型
|
||||
Theme/ 主题色板（Solarised Light）与波形默认配色
|
||||
Utf8Util/ 编码转换（UTF-8 ↔ UTF-16 等）
|
||||
（分片：PlotView.cpp / PlotViewPaint.cpp / PlotViewRaster.cpp /
|
||||
PlotViewPanels.cpp / PlotViewCacheKey.cpp / PlotViewFeatures.cpp /
|
||||
PlotViewInteraction.cpp）
|
||||
Utilities/ 与 UI 无关的数据与格式层（Inc/ + Src/）
|
||||
DataModel.* 数据层：DataSet / SignalGroup / TimeAxis / SignalData / SignalStyle / TabConfig
|
||||
DataModelTab.* TabConfig（每标签页的视图配置）实现
|
||||
DataTable.hpp 加载层中间表（TableBlock）
|
||||
DataLoader.* 加载入口（按扩展名分发）
|
||||
CsvLoader.* CSV（多 time 列 = 多时间轴）
|
||||
Mf4Loader.* ASAM MDF v4（.mf4/.mdf）
|
||||
MatLoader.* MATLAB .mat（v5 / v7.3 HDF5）
|
||||
MatH5.* HDF5 只读子集
|
||||
Deflate.* HDF5 分块解压（内置 inflate，不依赖 zlib）
|
||||
FormulaEval.* 「公式映射」表达式编译与求值
|
||||
Utf8Util.* 编码转换 + 数值/时长/频率格式化
|
||||
MemProbe.* 内存预检验与 terminate 兜底
|
||||
Resources/ 图标、清单(.manifest)、版本信息(.rc)、ResourceIds.h
|
||||
TestData/ 本地测试 CSV（**不纳入版本控制**）
|
||||
Documents/ 项目文档（含本文件）
|
||||
Documents/ 项目文档（含本文件、Code_Review_* 系列）
|
||||
```
|
||||
|
||||
- **依赖方向固定**：`Components/*` → `Components/Shared` → `Utilities`（不得反向）。
|
||||
`PopupMenu` / `ColorPickerPopup` 历史上放在 `Utilities`，现已在 `Components/Shared`。
|
||||
- **命名硬性约束**：源文件/头文件**不得含空格**，只能字母与下划线组合。
|
||||
- **头文件后缀**：C 用 `.h`，C++ 用 `.hpp`；C 源 `.c`，C++ 源 `.cpp`。
|
||||
- **防重复包含**：每个头文件必须 `#ifndef __XXX_XX_HPP__ / #define __XXX_XX_HPP__ / ... / #endif`；宏名 = 文件名的双边双下划线 + 全大写。
|
||||
@@ -10283,3 +10305,66 @@ release/debug **零警告**；Release `TargetCheck.exe` **890,880 B**（< 1 MB
|
||||
* 首次 379ms 仍是同步阻塞。若要彻底无感需把公式缓存改成后台线程计算 +
|
||||
未就绪时回退到保序端点映射；当前收益不足以承担该复杂度。
|
||||
|
||||
---
|
||||
|
||||
## 2026-11-04 · 第 33 轮（`Code_Review` 分支）：全项目结构/性能/体积审查
|
||||
|
||||
> 完整节点计划见 `Documents/Code_Review_Plan.md`，逐节点记录见
|
||||
> `Documents/Code_Review_Progress.md`，结论见 `Documents/Code_Review_Report.md`。
|
||||
> 本轮同时删除了过时的 `Documents/UI_Refactor_{Plan,Progress,Report}.md`。
|
||||
|
||||
### 1. 体积：890,880 B → **794,112 B**（−96,768，−10.9%）
|
||||
|
||||
* **`dynamic_cast` 是体积炸弹**：`Toolbar::BuildBoxes` 里唯一的
|
||||
`dynamic_cast<const UiButtonWidget*>` 是**多余强转**（`Button::widget` 本来就是该类型）。
|
||||
最小复现程序实测 39,936 → 115,200 B（+75,264）。
|
||||
* **光关 RTTI 不够**：`cp-demangle.o` 仍在链接里 —— 真凶是 libstdc++ 的 `vterminate.o`
|
||||
（被 `eh_terminate.o` 的 `std::terminate` 拉入，它引用 `__cxa_demangle`）。
|
||||
在 `MemProbe.cpp` 里**自行定义** `__gnu_cxx::__verbose_terminate_handler()`
|
||||
⇒ `vterminate.o` 不再被拉入 ⇒ 整个 demangler 消失（`.text` −49,520、`.rdata` −10,320）。
|
||||
⚠️ 该定义**必须**在全局 `__gnu_cxx` 命名空间，不能嵌在 `jdi` 里。
|
||||
* **Release 关闭异步展开表**（`RELEASE_ONLY_OPT = -fno-asynchronous-unwind-tables`）：
|
||||
`-fno-exceptions` 下 `.pdata` 30,420 → 12,684、`.xdata` 28,904 → 10,156（**−36,352**）。
|
||||
代价：Release 崩溃时 WER 拿不到可靠调用栈（不打印栈回溯的程序功能不受影响）。
|
||||
|
||||
### 2. ⚠️ 修掉一个真实构建缺陷：`DEBUG_OPT` 从未生效
|
||||
|
||||
`DEBUG_OPT = -O0` 定义了却**没有任何规则使用它** —— Debug 的两条编译规则误用
|
||||
`$(RELEASE_OPT)`（`-Oz`），且全程没有 `-g`。所谓 "Debug 构建" 其实只是
|
||||
"不 strip 的 Release"，**根本无法调试**。现已改用 `$(DEBUG_OPT) -g`（链接也加 `-g`）。
|
||||
⚠️ 因此 Debug 产物从 1.73 MB 变成 6.0 MB，属预期（真实 `-O0 -g`）。
|
||||
|
||||
### 3. 架构：面板"按种类分派"收敛为唯一派发点
|
||||
|
||||
`Lane` 里 7 种实验性功能各有 1 份 `LanePanelState` + 1 个滚动偏移。旧实现在
|
||||
**每一处**按种类取用的地方各写一遍嵌套三元式/`switch`（`EdgePanelLayoutFor` 里就有
|
||||
5 条 7 层嵌套三元式），新增一种面板要改 20+ 处，漏一处就是「拖不动 / 关不掉 /
|
||||
滚轮滚不到底」——`MARK`、`RESAMPLE`、`CLIP` 三种都真实踩过。
|
||||
|
||||
现在唯一派发点是 `PlotViewInternal.hpp` 的
|
||||
`LanePanelEnabled` / `LanePanelStateOf` / `LanePanelScrollOf`。
|
||||
⚠️ **这些 `switch` 一律不写 `default:`**，让 `-Wswitch`（属 `-Wall`）在新增
|
||||
`EdgePanelKind` 而漏登记时直接**编译报错**（已用最小程序验证 `-Wswitch` 确实会报）。
|
||||
⇒ **今后新增实验性功能面板，忘记登记会编译失败，而不是运行时静默出 BUG。**
|
||||
|
||||
### 4. 架构：`ApplicationCommands.cpp`（2,165 行）拆成 4 个分片
|
||||
|
||||
`ApplicationTabs.cpp`（355）/ `ApplicationDerive.cpp`（786）/ `ApplicationMenus.cpp`（409）/
|
||||
`ApplicationCommands.cpp`（642）。纯搬移、零逻辑改动；Makefile 按目录通配，无需登记。
|
||||
|
||||
### 5. 审查结论中的"没有发现问题"项（同样重要）
|
||||
|
||||
* **无死代码**：`-Wall` 零警告（未使用的 `static` 函数会报警）；对
|
||||
`PlotView` / `SignalPanel` / `Application` / `TabBar` / `Toolbar` / `DataSet`
|
||||
的 233+ 个 `m_*` 成员做全仓引用统计，**没有"只声明不使用"的字段**。
|
||||
* **无 TODO/FIXME/未完成的临时方案**（源码里的"未完成"全是边沿统计进行中的业务语义）。
|
||||
* **`UiMeasureTextWidth` 有记忆化**（键 = 字体 + 文本），因此布局层的重复测量不是热点。
|
||||
|
||||
### 6. 未做（诚实记录）
|
||||
|
||||
* `EdgePanelLayoutFor`（957 行分支）未改成"每面板一行元数据表"：
|
||||
没有离屏像素对比工具（`.preflight` 已删，崩溃会弹模态框），无法自证像素一致。
|
||||
* `EdgeScanFor` 的累积状态查找是**线性扫描** `m_edgeScans`（上限 2×信号数+余量）。
|
||||
288 信号时每帧约 16 万次比较（~0.3ms 量级）。可改哈希索引，但收益有限、风险不小。
|
||||
* `SceneContentHash` 里 6 段逐面板混入循环未合并（同上：无法验证"仅失效条件不变"）。
|
||||
|
||||
Reference in new issue
Block a user