50 lines
991 B
Markdown
50 lines
991 B
Markdown
# 代码评审流程
|
|
|
|
## 概述
|
|
|
|
架构设计师负责评审前端和后端工程师提交的代码，确保代码质量和需求满足度。
|
|
|
|
## 评审触发
|
|
|
|
代码提交到 Git 仓库后，架构设计师自动收到评审通知。
|
|
|
|
## 评审标准
|
|
|
|
### 架构整洁性 (40%)
|
|
- 代码结构清晰度
|
|
- 设计模式遵循
|
|
- 模块解耦程度
|
|
- 命名规范
|
|
|
|
### 需求满足度 (40%)
|
|
- 功能实现完整性
|
|
- 边界情况处理
|
|
- 错误处理
|
|
- 性能表现
|
|
|
|
### 安全性 (20%)
|
|
- 输入验证
|
|
- 权限控制
|
|
- 数据安全
|
|
- 漏洞防护
|
|
|
|
## 评审流程
|
|
|
|
```
|
|
提交代码 → 架构师评审 → 通过/不通过 → 存档
|
|
↓
|
|
发起讨论会议 → 修复 → 重新评审
|
|
```
|
|
|
|
## 输出
|
|
|
|
每次评审生成 `architecture/code-reviews/YYYY-MM-DD-模块.md` 报告。
|
|
|
|
## 讨论会议
|
|
|
|
如评审不通过，架构师需组织讨论会议：
|
|
1. 邀请相关开发人员
|
|
2. 讨论问题解决方案
|
|
3. 记录会议结论
|
|
4. 安排修复时间
|