88 行
6.4 KiB
Markdown
88 行
6.4 KiB
Markdown
# T040 游戏架构完善审查报告
|
||
|
||
> 审查人：ClaudeMiMo
|
||
> 日期：2026-07-06
|
||
> 审查范围：TDD-01~TDD-10（技术设计文档全量）、UI-00 §4.5（战斗界面规范）、GDD-17 §十一（技能程序化生成引擎）
|
||
> 任务背景：新增"城镇/野外实时位置同步"能力 + 战斗表现层从"纯文字战报"升级为"骨骼动画+特效"
|
||
|
||
---
|
||
|
||
## 一、审查范围
|
||
|
||
| 文档 | 版本 | 状态 |
|
||
|------|------|------|
|
||
| TDD-01 服务端整体架构设计 | v1.0 → v1.1 | ✅ 已修复 |
|
||
| TDD-02 客户端热更新技术方案 | v1.0 | ✅ 无需修改 |
|
||
| TDD-03 客户端架构设计 | v1.1 → v1.2 | ✅ 已修复（主要改动） |
|
||
| TDD-04 数据库表结构设计 | v1.6 | ✅ 已由前序 Agent 对齐 |
|
||
| TDD-05 API接口设计 | v1.3 | ⚠️ 需后续同步（RoundLog 新增字段） |
|
||
| TDD-06 离线挂机结算系统设计 | v1.1 | ⚠️ 需后续同步（RoundLog 新增字段） |
|
||
| TDD-07 反作弊与安全设计 | v1.0 | ✅ 无需修改 |
|
||
| TDD-08 GM与运营后台设计 | v1.0 | ✅ 无需修改 |
|
||
| TDD-09 推送与通知系统设计 | v1.0 | ✅ 无需修改 |
|
||
| TDD-10 数据埋点与分析设计 | v1.1 | ✅ 无需修改 |
|
||
| TDD-11 实时位置同步系统设计 | 不存在 | ℹ️ 另一 Agent 正在编写 |
|
||
| UI-00 界面设计规范 §4.5 | v1.2 | ✅ 作为审查基准 |
|
||
|
||
---
|
||
|
||
## 二、发现的缺口与矛盾清单
|
||
|
||
### 已修复（本轮直接修改）
|
||
|
||
| # | 严重度 | 问题描述 | 修复方式 | 涉及文件 |
|
||
|---|--------|----------|----------|----------|
|
||
| 1 | **P0** | TDD-03 缺少"战斗动画播放时间轴系统"设计：UI-00 §4.5 定义了骨骼动画+特效+ATB 速度条+战报滚动区的完整战斗界面，但 TDD-03 §5 仅有"文字战报渲染引擎"，未描述如何将服务端战报转换为动画播放时间轴 | 新增 TDD-03 §5.5（8 个子节）：战报→时间轴转换算法、AnimationEvent 数据结构、Spine 4.x 骨骼动画选型、特效模板池+参数化配置方案、BattleAnimController 播放控制器、与 UI-00 §4.5 的映射表、性能预算 | TDD-03 |
|
||
| 2 | **P0** | TDD-03 §5.1 RoundLog 数据结构缺少 `targetId`（目标ID）和 `effectId`（特效模板ID）字段，无法支持多目标扩展和特效模板关联 | 在 RoundLog 接口中新增 `targetId: string` 和 `effectId?: string` 字段 | TDD-03 |
|
||
| 3 | **P1** | TDD-01 §2.2 微服务职责边界表缺少"实时位置同步子系统"，架构总览不完整（城镇/野外实时 Match 房间能力未声明） | 在 §2.2 表末新增一行"实时位置同步子系统"，引用 TDD-11；§2.3 调用链路表新增一条降级策略 | TDD-01 |
|
||
| 4 | **P1** | TDD-03 §4.2 频道订阅对照表缺少实时位置同步频道 | 新增 `position:{region_id}` 频道行，附 TDD-11 引用说明 | TDD-03 |
|
||
| 5 | **P2** | TDD-01 §4.1 声明 Kafka 使用 KRaft 模式（去 ZooKeeper），但 §2.1 拓扑图仍标注"3 Broker / 3 ZK"，§2.4 端口表仍列出"2181 ZooKeeper" | 端口表 ZooKeeper 行补充说明"仅本地开发兼容旧镜像，生产使用 KRaft 模式无需 ZooKeeper" | TDD-01 |
|
||
| 6 | **P2** | TDD-03 §1 核心约束仍写"文字战报"，与 UI-00 §4.5 骨骼动画战报设计不一致 | 改为"文字战报+骨骼动画战报" | TDD-03 |
|
||
|
||
### 未修复（需后续处理）
|
||
|
||
| # | 严重度 | 问题描述 | 建议处理方式 |
|
||
|---|--------|----------|-------------|
|
||
| 7 | **P1** | TDD-05 §3.3.2 战报响应示例中的 RoundLog 字段未包含本轮新增的 `targetId` 和 `effectId` | 建议 TDD-05 维护者在下一版本同步更新战报响应示例 |
|
||
| 8 | **P1** | TDD-06 离线结算战报数据结构（§3.3 或相关章节）未包含 `targetId` 和 `effectId` | 建议 TDD-06 维护者同步更新离线战报数据结构 |
|
||
| 9 | **P1** | TDD-03 新增的 Spine 骨骼动画方案（§5.5.4）需要客户端实际集成验证，当前为架构设计层面的选型建议 | 建议在客户端原型阶段验证 Spine 4.x + Cocos 3.8.8 的兼容性和性能 |
|
||
| 10 | **P2** | TDD-01 §2.1 拓扑图中的 ASCII 未包含"实时位置同步子系统"的节点框 | 因 ASCII 拓扑图修改复杂度高且已有文字说明，建议后续统一重绘拓扑图时一并添加 |
|
||
| 11 | **P2** | TDD-01 版本记录仅 v1.0/v1.1，但 SYNC.md 显示多个 Agent 曾编辑本文档，历史版本记录不完整 | 建议 TDD-01 维护者补全历史 changelog |
|
||
| 12 | **Info** | TDD-11 尚不存在（另一 Agent 正在编写），本轮无法验证 TDD-01/TDD-03 对 TDD-11 的引用是否与实际内容一致 | TDD-11 完成后需做一次交叉验证 |
|
||
|
||
---
|
||
|
||
## 三、已修复文件清单
|
||
|
||
| 文件 | 修改类型 | 修改摘要 |
|
||
|------|----------|----------|
|
||
| `docs/技术文档/TDD-03-客户端架构设计.md` | 新增章节 + 字段扩展 + 版本更新 | v1.1→v1.2：新增 §5.5 战斗动画播放时间轴系统（Spine 4.x + 特效模板池）；RoundLog 新增 targetId/effectId；§4.2 新增 position 频道；§1 核心约束更新；决策记录新增 ✅C21-✅C24 |
|
||
| `docs/技术文档/TDD-01-服务端整体架构设计.md` | 新增行 + 注释 | v1.0→v1.1：§2.2 新增实时位置同步子系统行；§2.3 新增调用链路；§2.4 ZooKeeper 端口补充 KRaft 说明 |
|
||
|
||
---
|
||
|
||
## 四、设计决策摘要
|
||
|
||
本轮审查确认/新增以下技术决策：
|
||
|
||
| 编号 | 决策 | 来源 |
|
||
|------|------|------|
|
||
| ✅C21 | 战斗表现层采用 Spine 4.x 骨骼动画方案 | TDD-03 §5.5.4 |
|
||
| ✅C22 | 技能特效走"特效模板池 + 参数化配置"（20 个预制体覆盖 300+ 组合） | TDD-03 §5.5.5，呼应 GDD-17 §十一 |
|
||
| ✅C23 | 战报数据 RoundLog 新增 targetId 和 effectId 字段 | TDD-03 §5.1 |
|
||
| ✅C24 | 战斗动画播放采用 AnimationEvent 时间轴模型 | TDD-03 §5.5.1 |
|
||
| — | 实时位置同步子系统作为独立微服务模块，详见 TDD-11 | TDD-01 §2.2 |
|
||
|
||
---
|
||
|
||
## 五、后续建议
|
||
|
||
1. **TDD-11 完成后交叉验证**：确认 TDD-11 中的 AOI 算法、同步频率、Nakama Match 房间生命周期与 TDD-01 §2.2 的一句话职责描述、TDD-03 §4.2 的 position 频道定义一致。
|
||
2. **TDD-05/TDD-06 同步更新**：将 RoundLog 新增的 `targetId`/`effectId` 字段同步到 API 响应示例和离线结算数据结构中。
|
||
3. **Spine 原型验证**：在客户端原型阶段验证 Spine 4.x + Cocos Creator 3.8.8 的集成兼容性、内存占用和帧率表现。
|
||
4. **TDD-01 拓扑图重绘**：下次统一更新时将"实时位置同步子系统"加入 ASCII 拓扑图。
|
||
|
||
---
|
||
|
||
*T040 游戏架构完善审查报告 | 2026-07-06 | ClaudeMiMo*
|
