360 Secure SkillHub
360 安全技能中心

code-review-cn

按功能正确性、代码质量、性能、安全和可维护性五个维度审查代码,使用模板生成包含问题分级、修改建议和评分的 Markdown 中文审查报告。

guyulong
热度 0.00
最近更新 2026-10-18 23:40:10
分类:开发依赖:纯提示词应用场景:代码审查 / 报告撰写

安装技能

复制下面的话术发给你的 Agent,它会通过 secure-skills 检索并安装该技能。

执行命令安装此 skill: npx secure-skills-cli add guyulong/cn-agent-skills/code-review-cn
Metadata
name
code-review-cn
description
中文代码审查,生成中文审查报告
category
代码质量
tags
["代码审查","质量","中文"]
tools
["claude-code","cursor","copilot","windsurf"]

中文代码审查

使用场景

对代码进行全面审查,生成中文审查报告。

审查维度

1. 功能正确性

  • 逻辑是否正确
  • 边界条件是否处理
  • 异常情况是否覆盖
  • 返回值是否一致

2. 代码质量

  • 命名是否清晰(变量、函数、类)
  • 函数是否过长(建议 < 50行)
  • 是否有重复代码
  • 圈复杂度是否过高

3. 性能

  • 是否有 N+1 查询
  • 是否有不必要的循环
  • 内存是否有泄漏风险
  • 是否有可以缓存的计算

4. 安全

  • 输入是否做了校验
  • SQL是否使用参数化查询
  • 敏感信息是否硬编码
  • 权限是否正确检查

5. 可维护性

  • 是否有必要的注释
  • 是否有单元测试
  • 错误处理是否完善
  • 日志是否足够排查问题

报告模板

# 代码审查报告

## 概述
- 审查文件:xxx.py
- 审查日期:2026-06-06
- 总体评分:B+(良好)

## 发现的问题

### 🔴 严重问题(必须修复)
1. **SQL注入风险** (第45行)
   - 问题:直接拼接SQL字符串
   - 建议:使用参数化查询
   - 示例:`cursor.execute("SELECT * FROM users WHERE id = %s", (user_id,))`

### 🟡 建议改进
1. **函数过长** (第100-250行)
   - 问题:process_data 函数超过150行
   - 建议:拆分为多个小函数

### 🟢 优点
1. 错误处理完善
2. 日志记录详细

评分标准

  • A+:优秀,可直接合并
  • A:良好,小问题可忽略
  • B+:良好,建议修改后合并
  • B:一般,需要修改
  • C:较差,需要重构
  • D:严重问题,需要重写