code-review

ck0123's avatarfrom ck0123

系统化的代码审查方法，帮助发现问题并提供改进建议

⭐1stars🔀0forks📁View on GitHub🕐Updated Dec 10, 2025
[Code Review]

When & Why to Use This Skill

This Claude skill provides a systematic and structured framework for comprehensive code reviews. It is designed to enhance software quality by identifying logic errors, security vulnerabilities, and architectural flaws while ensuring adherence to industry best practices and project-specific coding standards.

Use Cases

  • Pull Request Auditing: Automate the initial review of code changes to ensure they meet quality benchmarks and architectural requirements before merging.
  • Security & Bug Detection: Proactively identify potential security leaks, edge-case failures, and improper error handling to reduce production incidents.
  • Performance Optimization: Analyze code for inefficient algorithms, redundant computations, and suboptimal data structure choices to improve system speed.
  • Technical Debt Reduction: Receive actionable suggestions for refactoring complex code, removing duplication, and improving overall maintainability.
  • Developer Mentorship: Provide junior developers with constructive, objective feedback and explanations of best practices during the development lifecycle.
namecode-review
description系统化的代码审查方法，帮助发现问题并提供改进建议
dependencies[]
priority8

代码审查技能

这个技能提供了一个结构化的方法来进行全面的代码审查，帮助发现潜在问题并提供改进建议。

何时使用此技能

当你需要：

  • 审查新提交的代码变更
  • 检查代码质量和最佳实践
  • 发现潜在的 bug 或安全问题
  • 提供代码改进建议

审查流程

第一步：理解上下文

  1. 了解变更目的 - 阅读相关的 PR 描述或提交信息
  2. 查看相关文件 - 理解代码在项目中的位置和作用
  3. 确定审查范围 - 明确需要审查的文件和功能

第二步：逐层审查

按以下顺序检查代码：

2.1 架构和设计

  • 代码结构是否合理？
  • 是否遵循现有的设计模式？
  • 是否有不必要的复杂性？

2.2 功能正确性

  • 代码是否实现了预期功能？
  • 边界条件是否处理正确？
  • 错误处理是否完善？

2.3 代码质量

  • 命名是否清晰、一致？
  • 是否有重复代码？
  • 注释是否必要和准确？

2.4 性能考虑

  • 是否有明显的性能问题？
  • 数据结构选择是否合适？
  • 是否有不必要的计算？

2.5 安全性

  • 输入验证是否充分？
  • 是否有潜在的安全漏洞？
  • 敏感数据处理是否安全？

第三步：提供反馈

反馈格式：

## 代码审查报告

### 概述
[简要总结代码变更和整体评价]

### 问题发现

#### 严重问题 🔴
- 问题描述
  - 位置: `文件名:行号`
  - 建议: 具体的修改建议

#### 建议改进 🟡
- 问题描述
  - 位置: `文件名:行号`
  - 建议: 具体的修改建议

#### 小建议 🟢
- 问题描述

### 优点 ✨
[列出代码中做得好的地方]

### 总结
[是否建议合并，需要哪些修改]

检查清单

通用检查项

  • 代码是否可读且易于理解？
  • 是否遵循项目的编码规范？
  • 是否有适当的错误处理？
  • 是否有必要的测试？
  • 文档是否更新？

语言特定检查项

Python:

  • 是否使用了类型提示？
  • 是否遵循 PEP 8？
  • 导入是否有序？

JavaScript/TypeScript:

  • 是否有适当的类型定义？
  • 是否处理了异步错误？
  • 是否避免了常见陷阱？

最佳实践

  • 保持客观 - 关注代码，而不是人
  • 提供具体建议 - 不只指出问题，还要给出解决方案
  • 区分优先级 - 明确哪些是必须修复的，哪些是建议
  • 肯定优点 - 也要指出做得好的地方
  • 保持简洁 - 避免过长的反馈