代码审查问题记录模板

使用这个模板来记录代码审查中发现的每一个问题。

---

问题：[标题]

严重性

• Critical（阻塞发布）
• High（合并前应修复）
• Medium（尽快修复）
• Low（可选优化）

类别

• Security
• Performance
• Code Quality
• Maintainability
• Testing
• Design Pattern
• Documentation

位置

文件： src/components/UserCard.tsx

行号： 45-52

函数 / 方法： renderUserDetails()

问题描述

是什么： 描述这个问题是什么。

为什么重要： 说明影响，以及为什么需要修复。

当前行为： 展示有问题的代码或行为。

期望行为： 描述理想情况下应该发生什么。

代码示例

当前（有问题）

// 这里展示了 N+1 查询问题
const users = fetchUsers();
users.forEach(user => {
  const posts = fetchUserPosts(user.id); // 每个用户都发一次查询！
  renderUserPosts(posts);
});

建议修复

// 使用 JOIN 查询进行优化
const usersWithPosts = fetchUsersWithPosts();
usersWithPosts.forEach(({ user, posts }) => {
  renderUserPosts(posts);
});

影响分析

方面	影响	严重性
性能	20 个用户就会产生 100+ 次查询	High
用户体验	页面加载缓慢	High
可扩展性	规模变大后会失效	Critical
可维护性	难以排查问题	Medium

相关问题

• AdminUserList.tsx 第 120 行有类似问题
• 相关 PR：#456
• 相关 issue：#789

额外资源

• N+1 Query Problem
• Database Join Documentation

审查者备注

• 这是这个代码库里常见的模式
• 可以考虑把它加入代码风格指南
• 也许值得提取一个辅助函数

作者回复（供反馈）

由代码作者填写：

• 已在提交 abc123 中修复
• 修复状态：Complete / In Progress / Needs Discussion
• 问题或疑虑：（描述）

---

统计信息（供审查者使用）

在审查多个问题时，请记录：

• 发现问题总数： X
• Critical： X
• High： X
• Medium： X
• Low： X

建议： ✅ Approve / ⚠️ Request Changes / 🔄 Needs Discussion

整体代码质量： 1-5 星