# Code Review

> 审查代码中的错误、风格问题和最佳实践 Use when this capability is needed.

- Skill: `tomevault-io/code-review-237` (Agent Skill, multi-file: 2 files)
- Install (CLI): `npx skillmds@latest add tomevault-io/code-review-237`
- Raw SKILL.md: https://api.skillmd.com/api/skills/tomevault-io/code-review-237/raw
- Safety review: pending (external: skill-scanner PASS, skillspector PASS)
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: AI & ML
- Author: tomevault-io (https://skillmd.com/u/tomevault-io)
- Updated: 2026-09-17
- Page: https://skillmd.com/skills/tomevault-io/code-review-237

---


# 代码审查技能

## 目标
审查代码变更中的潜在错误、风格违规和最佳实践违规。

## 审查检查清单

### 1. 功能性
- 代码是否实现了所需功能？
- 是否存在明显的错误或逻辑错误？
- 边界情况是否得到妥善处理？

### 2. 代码质量
- 代码是否可读且结构良好？
- 变量和函数名称是否具有描述性？
- 是否存在不必要的复杂性？

### 3. 最佳实践
- 代码是否遵循特定语言的最佳实践？
- 设计模式的使用是否恰当？
- 代码是否符合DRY原则（不要重复自己）？

### 4. 安全性
- 是否存在任何安全漏洞？
- 用户输入是否得到适当验证？
- 敏感数据是否得到妥善处理？

### 5. 性能
- 是否存在明显的性能问题？
- 算法和数据结构是否合适？
- 是否存在不必要的计算？

## 审查流程

1. **阅读差异**：理解变更内容和原因
2. **检查功能**：验证实现是否按预期工作
3. **检查风格**：确保格式和命名一致
4. **检查测试**：验证测试覆盖率和测试质量
5. **提供反馈**：提供建设性、具体的反馈

## 输出格式

审查代码时，请按以下结构组织反馈：

```
## 总结
[审查的简要总结]

## 发现的问题
### 严重
- [列出必须修复的严重问题]

### 重要
- [列出应该修复的重要问题]

### 轻微
- [列出轻微问题和风格建议]

## 优点
- [突出好的实践和实现良好的部分]

## 建议
- [具体的改进建议]
```

## 示例

```java
// 修改前
public void process(List<String> items) {
    for (int i = 0; i < items.size(); i++) {
        String item = items.get(i);
        System.out.println(item);
    }
}

// 修改后 - 使用增强型for循环
public void process(List<String> items) {
    for (String item : items) {
        System.out.println(item);
    }
}
```

## 提示
- 建设性：专注于改进代码，而不是批评作者
- 具体：指出确切的行并解释为什么存在问题
- 友善：记住代码审查是每个人的学习机会
- 优先级：首先关注严重和重要问题

---
> Source: [HOPPINZQ/hoppinai-agent](https://github.com/HOPPINZQ/hoppinai-agent) — distributed by [TomeVault](https://tomevault.io).
<!-- tomevault:4.0:skill_md:2026-06-26 -->