radar_data_process/archived_docs/BUG_REPORT.md

# 雷达数据处理 DLL 逻辑 BUG 分析报告

分析范围：`data_process_class_dll/` 下全部源码（不含 `Eigen/`），以迁移后的当前代码行号为准。
严重级别定义：

- **P0**：可能导致崩溃、越界读写、内存破坏或长时间运行内存泄漏，建议优先修复。
- **P1**：在常见异常时序/边界输入下会出错，或属于明显算法逻辑错误。
- **P2**：健壮性、数值稳定性、状态一致性、功能缺失等问题。
- **P3**：代码质量/可维护性问题。

---

## 1. 内存与生命周期

### BUG-01 [P0] 基类没有虚析构函数，工厂按基类指针 delete 派生对象
- 位置：`data_process_class_dll.h:168`、`data_process_class_dll.cpp:25-32`
- 说明：`Data_process_class_dll` 没有声明虚析构函数，而 `Data_Process` 内部包含多个 `std::vector` 和若干成员对象。`Data_Process_Factory::Destroy()` 执行 `delete p` 时静态类型是基类指针，只会调用基类析构函数，派生类成员（`Data_buffer`、`trust_track`、`temp_track`、Eigen 相关对象等）的析构不会执行，属于未定义行为并造成内存泄漏。
- 建议：在基类中增加 `virtual ~Data_process_class_dll() {}`；同时让工厂支持重复销毁、销毁后返回 nullptr。
- 暂不修复

### BUG-02 [P1] `Hight_smooth` 高度平滑缓存只增不减，长时间运行内存持续增长
- 位置：`track_asso.cpp:890`、`track_asso_tas.cpp:722`、`track_asso_direct_tracking.cpp:639`
- 说明：每次高度更新都执行 `Hight_smooth.push_back(...)`，从未裁剪或清空。航迹存活时间越长，该 vector 越大；500 条航迹长时间运行时内存会持续增长。
- 建议：仅保留最近 `height_win_length` 个高度值（如 `resize`/`erase(begin)` 后再 push），或改用固定长度 `std::deque`；航迹消亡时随结构体释放。
- 暂不修复

### BUG-03 [P2] 工厂单例创建/销毁非线程安全
- 位置：`data_process_class_dll.cpp:15-32`
- 说明：`GetB()` 和 `Destroy()` 对静态指针 `p` 无任何同步。若宿主在多线程环境调用，可能创建两个实例（泄漏一个）或对同一对象重复销毁。
- 建议：用 C++11 `static Data_Process instance;` 返回地址，或对工厂方法加锁；明确 DLL 接口的线程模型。
- 暂不修复

---

## 2. 数组/向量越界

### BUG-04 [P0] TAS 输入点迹未限制 `Point_Sum <= 150`
- 位置：`data_process.cpp:66`
- 说明：TWS 分支使用 `min(Point_Sum, 150)`，但 TAS 分支直接 `data_num = Data_Input[0].Point_Sum` 并循环读取 `Data_Input[i]`。当协议传入 `Point_Sum > 150` 时，会越界读取 `Data_Input[150]`。
- 建议：与 TWS 分支一致，使用 `data_num = min(max(Point_Sum,0), 150)`，并校验 `Point_Sum >= 0`。
- 按建议修复

### BUG-05 [P0] `Track_to_start.size()-1` 在空容器时下溢导致越界
- 位置：`track_init.cpp:534`
- 说明：与已发现示例一致。`size()` 返回 `size_t`，空容器减 1 得到极大值，随后 `Track_to_start[i]` 越界。
- 建议：`if (Track_to_start.empty()) return;`，或改为 `for (size_t i = 0; i + 1 < Track_to_start.size(); ++i)`。
- 按建议修复，改为 `for (size_t i = 0; i + 1 < Track_to_start.size(); ++i)`

### BUG-06 [P0] `track_start_point_num` 未校验，取值 0/1/2 或大于 9 时多处越界
- 位置：`track_init.cpp:533,544-545,588-594`、`track_init_direct_tracking.cpp:316-322`
- 说明：重复航迹比较固定访问 `Track_to_start[i][1]`、`[2]`，三点初始化访问 `[L-3]/[L-2]/[L-1]`。该参数由外部 `RadarPara` 传入，若未初始化或配置为 0/1/2，会出现负下标或越界；若大于 9，`tmp_track_die` 会在 `n>=10` 时删除临时航迹，逻辑也无法起批。
- 建议：在 `track_process_parameters_initial/modify` 中校验 `3 <= track_start_point_num <= 9`；使用 `Track_to_start[i].size()` 作为实际长度并在访问前检查。
- 按建议修复

### BUG-07 [P0] 禁止区域个数未限制在 30 以内，外部配置过大时数组越界
- 位置：`track_init.cpp:711`（`track_prohibite_area_num`）、`tas_ctrl.cpp:123`（`TAS_prohibite_area_num`）
- 说明：循环上界直接使用外部传入的计数，而对应数组固定为 `[30]`。配置大于 30 时越界读。
- 建议：循环上界改为 `min(count, 30)`，并在参数初始化/修改时拒绝非法计数或截断。
- 暂不修复

### BUG-08 [P1] 消亡航迹号输出无数组容量检查
- 位置：`track_die.cpp:24`、`track_die_tas.cpp:26`
- 说明：`Track_die_Index_Output[*Track_die_num_Output-1] = ...` 不检查数组容量。正常流量下最多 500 条航迹，与 `MAX_TRACK_NUM` 一致，但接口没有把容量传入，一旦宿主传入较小数组、或计数被异常修改，就会越界写。
- 建议：接口增加 `die_array_capacity` 参数，或内部保证 `*Track_die_num_Output < MAX_TRACK_NUM` 后再写。
- 暂不修改

### BUG-09 [P1] `Beam_Ctrl` 路径中输出计数未初始化/无上界，可能越界写航迹输出数组
- 位置：`data_process.cpp:193-198`、`tas_ctrl.cpp:74-78`
- 说明：`Beam_Ctrl()` 直接使用宿主传入的 `Trust_track_num_Output`，而 `tas_ctrl_process` 不会先清零，`tas_target_add` 会基于旧值自增并写 `Trust_Track_Output[*Trust_track_num_Output-1]`。若调用方未清零或旧值接近 `MAX_TRACK_NUM`，会越界写。
- 建议：`tas_ctrl_process` 内部保存 `*count = 0` 或在每次写入前检查 `*count < MAX_TRACK_NUM`；同时校验指针非空。
- 按建议修改

### BUG-10 [P1] 航迹号直接作为数组下标，未校验范围
- 位置：`track_index_mangement.cpp:29`
- 说明：`List[(*trust_track)[i].Track_Index-1] = 1`，若航迹号不在 `[1, MAX_TRACK_INDEX]` 内（异常数据、内存损坏、外部修改），立即越界。
- 建议：写前检查 `Track_Index >= 1 && Track_Index <= MAX_TRACK_INDEX`，异常航迹号返回错误或跳过。
- 按建议修改

---

## 3. 未初始化变量

### BUG-11 [P1] `Track_Asso_Tas::model_filter` 在找不到 TAS 目标时使用未初始化变量
- 位置：`track_asso_tas.cpp:228-232`，随后在 `271/276/280/282` 等使用
- 说明：`X1/X2/X3/P1/P2/P3/T_track/v_track/r_track/h_track` 只在 `Track_Index == tas_track_idx` 时赋值。若 `tas_track_idx` 不存在（例如目标已被消亡、主程序传入失效批号），后续仍用这些未初始化值计算 IMM、距离门限和外推，结果不可预测。MSVC `/W4` 已报 C4701。
- 建议：函数开头初始化这些变量，并在找不到目标时直接 return；上层也应处理“TAS 目标不存在”的返回值。
- 按建议修改

### BUG-12 [P1] `tas_beam_output` 在队列目标不在航迹表中时使用未初始化变量
- 位置：`tas_ctrl.cpp:242-271`
- 说明：`H_track`、`X_now[]` 只在找到匹配航迹时赋值；若 `track_clear_all()` 后队列未清空、或目标已被删除而队列未同步，循环找不到目标，随后 `asin(H_track/range)`、`X_now[...]` 使用未初始化数据。Cppcheck 和 MSVC `/W4` 均报 C4701。
- 建议：找不到目标时立即 `open_flag=0` 并 return；变量声明时初始化。
- 按建议修改

### BUG-13 [P2] `Data_Process::Work_Parameter` 及若干成员在构造后未初始化
- 位置：`data_process.h:28-31,78-95`
- 说明：构造函数为空，`Work_Parameter`、`Beam_num`、`data_num`、`TAS_track_idx` 等未初始化。若宿主在调用 `track_process_parameters_initial` 前就调用 `data_preprocess/track_process/Beam_Ctrl`，`Work_Parameter.track_start_point_num`、`Sys_delay`、`V_MIN/V_MAX` 等是垃圾值，可能导致起批越界、除零或异常门限。
- 建议：构造函数中对 `Work_Parameter` 进行 `memset`/值初始化并设置安全默认参数；在处理函数入口检查“参数是否已初始化”。
- 暂不修复

### BUG-14 [P3] `Track_Ind_Mangement::lastest_index` 未初始化
- 位置：`track_index_mangement.h:17`、`track_index_mangement.cpp:33`
- 说明：只有空航迹表分支会赋值为 1；如果首次调用时航迹表非空，就会读取未初始化的 `lastest_index`。
- 建议：声明为 `int lastest_index = 0;` 或增加构造函数初始化。
- 按建议修改

---

## 4. 时间戳/时序处理

### BUG-15 [P1] TWS 关联在 `delta_T <= 0` 时仍把点迹标记为已关联并刷新航迹状态
- 位置：`track_asso.cpp:610-691`
- 说明：滤波和航迹信息更新在 `if (delta_T > 0)` 内，但 `Extrapolate_round=0`、`point_flag=1`、`associate_point_number++`、点迹 `Use_Flag=1` 都在 if 之外。重复/乱序时间戳的点仍会“占用”点迹、让航迹看起来已更新，实际状态未更新。
- 建议：`delta_T <= 0` 时直接跳过该关联候选（或作为无效量测处理），不要标记点迹已使用、不要刷新航迹新鲜度。
- 按建议修改

### BUG-16 [P1] TAS 关联完全没有 `delta_T > 0` 检查
- 位置：`track_asso_tas.cpp:271-335,385-452`
- 说明：TAS 路径计算 `delta_T` 后直接进入统计距离和滤波，即使 `delta_T <= 0` 也会生成 F/Q 并更新航迹，可能把航迹时间更新到过去。
- 建议：与 TWS 一致，只有 `delta_T > 0` 才允许关联；否则跳过该点。
- 按建议修改

### BUG-17 [P1] TAS 外推使用未校验的 `latest_timestamp - T_track`，负时间差导致反向预测
- 位置：`track_asso_tas.cpp:483-506`
- 说明：若 `latest_timestamp < T_track`（乱序/重复时间戳），`delta_T` 为负，`T_track += delta_T*1000` 会回退航迹时间，IMM F/Q 也按负时间生成。
- 建议：`delta_T = max(0, latest_timestamp - T_track)/1000.0`；若为 0 则直接返回或保持原状态。
- 按建议修改

### BUG-18 [P1] 卡尔曼初始化及统计距离函数对零/负时间差没有保护
- 位置：`kalman.cpp:13-46,126-130,158-168`；调用点 `track_init.cpp:275,450,593-594`、`track_init_direct_tracking.cpp:135,271,321-322`
- 说明：两点/三点初始化直接除以 `T/T1/T2`。同一 CPI 重复点、时间戳相等或乱序会产生除零、inf/NaN，随后污染航迹协方差和模型概率。
- 建议：调用前统一校验时间差大于最小阈值（如 >0 或 >1 ms）；`kalman_filter_init_2dots/3dots` 内部对非法时间返回错误。
- 暂不修改

### BUG-19 [P1] `Bind_speed` 对 `prt == 0` 无保护，而接口说明 PRI 可给 0
- 位置：`kalman.cpp:766-770`、`data_process_class_dll.h:29`
- 说明：`return 150000.0/(freq*prt)`，当 `PRI=0` 时除零；结果传给 `Round()` 会把 inf/NaN 转换为未定义整型。
- 建议：`prt <= 0` 时返回固定安全值或直接返回无效距离；统一约定 PRI 单位与默认值。
- 按建议修改，同时修改接口说明：PRI不可为0

---

## 5. 关联门限与滤波算法逻辑

### BUG-20 [P1] 统计距离 d 已经是平方形式，门限中又平方了一次
- 位置：`track_asso.cpp:544-546`、`track_asso_tas.cpp:335`、`track_asso_direct_tracking.cpp:286`、`track_init.cpp:221`
- 说明：`d_cal_EKF` 返回的是 `delta_z^T S^{-1} delta_z`（马氏距离平方），后续门限却写成 `d*d < THRESHOLD*THRESHOLD`。例如阈值 3 时本意是 `d < 9`，实际变成 `d < 3`，波门明显偏小。
- 建议：统一改为 `d < TRACK_START_THRESHOLD*TRACK_START_THRESHOLD` / `d < ASSO_THORD*ASSO_THORD`。
- 暂不修改

### BUG-21 [P1] 近程模型 3 门限因整数除法恒为 0
- 位置：`track_asso.cpp:544-545`
- 说明：`ASSO_THORD` 是 int 宏，`ASSO_THORD*ASSO_THORD/1000` 和 `/100` 按整数计算，结果均为 0，导致 `d3*d3 < 0` 永远不成立；近距离下模型 3 实际被禁用。
- 建议：写为 `ASSO_THORD*ASSO_THORD/1000.0`、`/100.0`，同时按 BUG-20 修正平方关系。
- 按建议修改，但不修改BUG-20

### BUG-22 [P1] EKF 方位角残差未按 0/2π 环绕处理
- 位置：`kalman.cpp:265-298,457-492,642-680`；凝聚 `dot_coh.cpp:35,44`（`work_mode==0` 分支）、`dot_coh_tas.cpp:33,42`
- 说明：预测方位被归一化到 `[0,2π)`，但量测方位未归一化，`delta_z = Z_mea - Z_pred` 未做 ±π 环绕。目标跨正北时残差会接近 2π，导致错误拒绝/错误滤波。凝聚中 `work_mode==0` 和 TAS 凝聚也直接 `fabs(angle)`，未处理 360° 环绕。
- 建议：方位差统一按 `wrapToPi()` 处理；`work_mode==0` 与 TAS 凝聚复用同一环绕角差函数。
- 按建议修改

### BUG-23 [P1] IMM 模型概率计算缺少零分母/奇异矩阵保护
- 位置：`track_asso.cpp:157-165,627-647`、`track_asso_tas.cpp:122-130,406-431`、`track_asso_direct_tracking.cpp:95-103,352-374`
- 说明：`c[0..2]`、`Possibility1*c[0]+...`、`det_S` 都可能为 0 或负；`1/sqrt(2*PI*det_S)` 对负/零行列式产生 NaN。一旦概率变 NaN，后续 `model_output` 会把整条航迹状态污染。
- 建议：计算前检查 `det_S > eps`、分母 > eps；异常时保持上一拍模型概率或回退为等概率 `{1/3,1/3,1/3}`。
- 按建议修改

### BUG-24 [P2] EKF 似然只取 3x3 新息协方差左上角 2x2 的行列式
- 位置：`kalman.cpp:515-519`（S 为 3x3，但输出只保存 2x2）；调用处 `track_asso.cpp:627-633`、`track_asso_tas.cpp:406-412`、`track_asso_direct_tracking.cpp:352-358`
- 说明：EKF 量测为距离/方位/径向速度三维，新息协方差 S 是 3x3，但接口 `S_filter[2][2]` 和似然计算只使用二维子块，模型似然不完整。
- 建议：将接口改为 3x3，并采用完整 3 维高斯归一化因子 `1/sqrt(pow(2*PI,3)*detS)`（若只比较相对大小，也至少应保持三个模型使用相同维数）。
- 按建议修改

### BUG-25 [P1] TWS 航迹关联/输出没有排除 TAS 航迹
- 位置：`track_asso.cpp:65-133`（输出）、`141-228`（交互
）、`414-570`（滤波）
- 说明：原代码中有 `// if manual_tracking_flag==0` 的过滤逻辑但被注释掉。当前 TWS 处理会遍历并更新所有航迹，包括 `Track_Mode==1` 的 TAS 航迹；TWS 输出循环也会把 TAS 航迹当作 `point_type=0` 输出，造成 TAS 航迹被 TWS 点迹错误更新和重复输出。
- 建议：TWS 的交互、关联、输出统一跳过 `Track_Mode == 1`（或 `manual_tracking_flag == 1`）的航迹；TAS 目标只在 TAS 流程中处理。
- 暂不处理

### BUG-26 [P2] 输出航向角使用 `atan(y/x)` 而不是 `atan2`
- 位置：`track_asso.cpp:100`、`track_asso_tas.cpp:66`、`track_asso_direct_tracking.cpp:59`
- 说明：当 `vx == 0` 时除法结果接近 ±inf，现有象限修正逻辑不完整，某些象限会输出负角度或 90° 偏差。
- 建议：统一使用 `atan2(X[4], X[1])` 并归一化到 `[0,2π)`。
- 按建议修改

### BUG-27 [P2] `asin(height/range)` 缺少定义域和零距离保护
- 位置：`track_asso_tas.cpp:54`、`track_asso_direct_tracking.cpp:48`、`track_init.cpp:646,673`
- 说明：`range` 为 0 或 `height > range` 时，`asin` 参数超出 `[-1,1]` 产生 NaN，并输出到 `Track.Elevation`。
- 建议：计算前钳位 `h/r` 到 `[-1,1]`，并对 `r <= eps` 特殊处理。
- 按建议修改

---

## 6. 状态清理与接口一致性

### BUG-28 [P1] `track_clear_all` 清理不彻底，重连后可能输出幽灵航迹/幽灵波束
- 位置：`data_process.cpp:219-229`
- 说明：只清空 6 个顶层 vector，未清理：
  - `Dot_Coh::data_input_buff`（滑窗凝聚缓存）；
  - `TAS_Ctrl::tas_target_queue` 和 `tas_target_num`；
  - `last_beam_num`（扫描圈判断）；
  - `Track_Ind_Mangement::lastest_index`；
  - 各关联类的 `point_process` 成员。
  清空后若立即调用 `Beam_Ctrl`，TAS 队列仍认为有目标，但 `trust_track` 已空，会触发 BUG-12 的未初始化路径并输出错误波束；下一次 TWS 处理也可能输出清空前的缓存点迹。
- 建议：为 `Dot_Coh`、`TAS_Ctrl`、`Track_Ind_Mangement` 增加 `reset()`，`track_clear_all` 调用所有 reset，并恢复 `last_beam_num=INT_MAX`。
- 按建议修改

### BUG-29 [P2] `tracking_stop` 后 TAS 队列要到下一次 `Beam_Ctrl` 才移除
- 位置：`data_process.cpp:260-270`、`tas_ctrl.cpp:138-162`
- 说明：`tracking_stop` 只清标志位和 `Track_Mode`，不会立即清 `tas_target_queue`。在调用 `Beam_Ctrl` 前，队列仍会输出该目标的跟踪波束。
- 建议：`tracking_stop` 内同步删除 TAS 队列项并更新 `tas_target_num`，或明确接口时序并文档化。
- 暂不修改

### BUG-30 [P2] 工作模式枚举与代码分支不一致
- 位置：`data_process_class_dll.h:140`（注释：0 进程、1 中程、3 远程）、`track_asso.cpp:729-744`（按 0/1/2/else 分支）
- 说明：注释定义远程模式为 3，但代码按 `work_mode==2` 使用远距数据率；当外部按注释传 3 时，会落入 else 使用近程数据率 `DATA_RATE_SHORT`。
- 建议：统一枚举定义，或代码改为 `work_mode==3` 使用 `DATA_RATE_FAR`。
- 注释有误，修改注释为：0 进程、1 中程、2 远程

### BUG-31 [P2] `tracking_point` 和引导跟踪处理是空实现
- 位置：`data_process.cpp:282-286`、`data_process_class_dll.h:200-213`、`track_init_direct_tracking.cpp:11-21`
- 说明：`tracking_point` 直接返回 0；`Data_Process` 没有重写 `direct_tracking_process`，基类默认返回 0；`Track_Init_Direct_Tracking::track_init_process_logic` 也是空函数。若接口已被主程序调用，则相关功能实际未生效。
- 建议：确认这两个接口是否已废弃；若仍需要，补齐实现或至少在文档中明确为未实现并返回错误码。
- 在注释中明确接口未启用，保留空实现

---

## 7. 其他逻辑与健壮性

### BUG-32 [P2] 凝聚内层循环没有跳过已标记使用的点
- 位置：`dot_coh.cpp:25,148`、`dot_coh_tas.cpp:25`
- 说明：内层只检查当前基准点 `loop_of_point` 的 `Use_Flag`，没有检查被比较点 `i` 的 `Use_Flag`。已被凝聚掉的点仍可参与后续比较，甚至反过来把高幅度基准点标记掉。
- 建议：内层同样判断 `(*data_input)[i].Use_Flag != 1`（以及 `data_tmp[i]`）。
- 按建议修改

### BUG-33 [P2] 起批/临时航迹关联未校验 `delta_T > 0`，重复时间戳可能产生非法卡尔曼初始化
- 位置：`track_init.cpp:189,381,450`、`track_init_direct_tracking.cpp:67,221,271`
- 说明：点迹与临时航迹、点迹与航迹头的关联均使用 `T_point - T_track_head`，未要求正时间差。`kalman_filter_init_2dots` 遇到 0/负 T 会产生除零。
- 建议：在关联条件中显式要求 `T_point > T_track_head`，且时间差需大于最小步长。
- 暂不修改

### BUG-34 [P2] 临时航迹消亡函数在空向量时直接取 `[n-1]`
- 位置：`track_init.cpp:694-695`、`track_init_direct_tracking.cpp:417-418`
- 说明：`int n = (*Iter).size();` 后立即 `(*Iter)[n-1]`。正常流程每条临时航迹至少 1 个点，但没有防御；一旦出现空内层向量，`n-1` 为负并越界。
- 建议：先判断 `n > 0`，空向量直接删除。
- 按建议修改

### BUG-35 [P3] 航迹号复用逻辑漏掉 `MAX_TRACK_INDEX`
- 位置：`track_index_mangement.cpp:44,56`
- 说明：回绕查找循环写作 `i < MAX_TRACK_INDEX`，因此 500 号航迹永远不会被复用；与第一段 `i <= MAX_TRACK_INDEX` 不一致。
- 建议：两处回绕循环改为 `i <= MAX_TRACK_INDEX`。
- 按建议修改

### BUG-36 [P3] TAS 波束关闭时只清 `open_flag`，其他字段保留旧值
- 位置：`tas_ctrl.cpp:277-284`
- 说明：当 `tas_target_queue[0].empty_flag==0` 时，仅设置 `Tracking_beam->open_flag=0`，`Range/Azi/Elev/type/TAS_track_index` 保留上一拍内容。主程序若只按 `open_flag` 判断则无问题，但字段语义不清晰。
- 建议：关闭时同时清零 `type/Range/Azi/Elev/TAS_track_index`。
- 暂不修改

### BUG-37 [P3] 部分结构体局部变量未完整初始化
- 位置：`track_init.cpp:80,598`、`track_init_direct_tracking.cpp:325`
- 说明：`Temp_track temp_track_tmp`、`Trust_Track trust_track_tmp` 的 `P` 矩阵等字段未显式初始化。当前流程部分字段未被读取，但依赖调用顺序；后续维护容易读到垃圾值。
- 建议：使用值初始化 `Temp_track tmp = {};` 或为结构体提供构造函数/`init()` 统一初始化所有字段。
- 按建议修改

---

## 建议修复顺序

1. 先修 P0：BUG-01、BUG-04、BUG-05、BUG-06、BUG-07、BUG-11、BUG-12。
2. 再修 P1 时间与关联逻辑：BUG-15~BUG-23、BUG-25、BUG-28。
3. 最后处理数值稳定性、状态一致性和未启用功能：BUG-24、BUG-26~BUG-37。

> 说明：`track_asso_direct_tracking.cpp` 和 `track_init_direct_tracking.cpp` 当前未加入 CMake 构建（与旧 .pro 一致），其中问题与主流程同类问题重复，修复主流程时可同步修改或暂缓。