代码审查记录

基本信息

项目	内容
审查编号	CR-YYYY-MM-DD-XXX
审查日期	YYYY-MM-DD
审查人	@reviewer
作者	@author
仓库/项目	repo-name
分支/PR	#PR号 / 分支名
审查范围	文件列表或 commit 范围

---

审查清单

1. 功能与逻辑

• 代码是否实现了需求文档中的功能？
• 边界条件（空值、越界、异常输入）是否已处理？
• 是否存在潜在的逻辑漏洞或竞态条件？
• 算法复杂度是否合理？大数据量下是否有性能隐患？

2. 代码质量

• 命名（变量、函数、类）是否清晰、有意义？
• 函数/方法是否遵循单一职责原则（SRP）？
• 代码重复（DRY）是否已消除或合理抽象？
• 注释是否必要且准确？是否避免了冗余注释？
• 魔法数字/字符串是否已提取为常量？

3. 安全性

• 用户输入是否已验证和转义（防注入、XSS）？
• 敏感数据（密码、密钥、Token）是否未硬编码？
• 权限校验是否到位？是否存在越权访问风险？
• 依赖库是否存在已知漏洞？

4. 性能与资源

• 是否存在 N+1 查询、内存泄漏或资源未释放？
• 异步操作是否正确处理（回调、Promise、async/await）？
• 缓存策略是否合理？缓存失效逻辑是否正确？

5. 可测试性

• 核心逻辑是否有单元测试覆盖？
• 测试用例是否包含正常、异常和边界场景？
• 代码是否易于测试？耦合度是否过高？

6. 规范与风格

• 是否遵循团队代码规范（Lint/Format）？
• 类型定义是否完整（TypeScript/静态语言）？
• 导入/依赖是否整洁？是否存在循环依赖？

---

审查意见

问题列表

🔴 阻塞问题（必须修复）

序号	文件	行号	问题描述	建议修改
1	path/to/file	L42	描述	具体建议
2

🟡 建议优化（推荐修复）

序号	文件	行号	问题描述	建议修改
1	path/to/file	L88	描述	具体建议
2

🟢 肯定与亮点

• 某处设计优雅/性能优化得当/测试覆盖完善...

---

审查结论

• 通过 — 无阻塞问题，可合并
• 有条件通过 — 需修复指定问题后无需再次审查
• 需修改后重审 — 存在阻塞问题，修复后需再次审查

备注： > 其他补充说明...

---

修订记录

日期	修订人	说明