Modified Code Review

Polar Sponsor
爱发电 赞助
.NET 9.0

审查用户修改的代码(diff/PR),提供最佳实践建议,分析方案性价比并输出代码评分。使用...

修改代码评审(Modified Code Review)

功能概述

修改代码评审(Modified Code Review)是一项面向实际任务的技能,主要用于对用户修改的代码进行评审,给出最佳实践建议、修改方案性价比分析,并输出代码评分;用户要求「review 我修改的代码」「审查修改」「代码评审」;

核心要点

  • 用户需要「最佳实践方案」「修改方案性价比」「代码评分」;
  • 用户提供 git diff、PR 或指定文件/改动范围;
  • 1. 获取改动范围;

使用与执行

若用户未指定:先看 git status。它将相关步骤、工具调用和结果整理方式集中到统一流程中,帮助使用者更快完成目标并减少重复操作。从功能定位来看,该技能强调把分散的操作要求整理成清晰、可复用的处理流程,使用户能够围绕既定目标快速准备输入、选择执行方式并获得结构化结果。

结果检查与注意事项

实际使用前应先确认任务范围、数据来源、运行环境、必要权限和关键参数,再依据技能说明逐步执行;若输入条件不完整,应先补齐信息或采用保守配置,避免因错误假设导致结果偏离需求。执行过程中需要关注工具调用是否成功、接口或依赖是否可用、输出格式是否符合预期,并对异常提示、缺失字段和边界情况进行处理;涉及批量任务时,还应保存进度,避免中断后重复操作。

修改代码评审(Modified Code Review)

对用户修改的代码进行评审,给出最佳实践建议、修改方案性价比分析,并输出代码评分。

触发场景

  • 用户要求「review 我修改的代码」「审查修改」「代码评审」
  • 用户需要「最佳实践方案」「修改方案性价比」「代码评分」
  • 用户提供 git diff、PR 或指定文件/改动范围

执行流程

1. 获取改动范围

  • 若用户未指定:先看 git status / git diff 或当前打开/选中的文件
  • 明确本次评审的文件与改动内容(新增、修改、删除)

2. 评审维度

按以下维度分析修改,并给出结论与建议:

维度 关注点
正确性 逻辑是否正确、边界与异常是否考虑、是否有明显 bug
可读性 命名、结构、注释是否清晰,是否符合项目风格
可维护性 是否易扩展、是否过度耦合、重复代码
性能 是否有不必要的计算、内存/请求是否合理
安全 输入校验、敏感数据、XSS/注入等
测试 关键路径是否有测试、用例是否合理

3. 最佳实践建议

  • 每条建议标明:必须改 / 建议改 / 可选优化
  • 若与项目约定(如 CLAUDE.md、ESLint、现有模式)冲突,以项目约定为准并说明
  • 建议附带简短理由和(如适用)示例代码片段

4. 修改方案性价比

对主要修改方案做「性价比」评估:

  • 收益:可维护性、性能、可读性、风险降低等
  • 成本:改动量、影响范围、回归风险、时间预估
  • 结论:高性价比(建议优先)/ 中性价比(可选)/ 低性价比(暂不推荐),并简述原因

5. 代码评分

先列 Blockers,再给评分——有 Blocker 时评分仅供参考,不能用分数掩盖必须修的问题。

分数区间 等级 含义
90–100 A 优秀 可直接合并,仅有个别小建议
75–89 B 良好 整体达标,有若干改进点
60–74 C 合格 能用但建议按优先级修复/优化
40–59 D 需改进 存在明显问题,建议修改后再合
0–39 E 不达标 存在严重问题,需重构或重做

评分需结合:正确性权重最高,其次可维护性/可读性,再考虑性能与安全。

输出模板

按以下结构组织回复(可用中文):

## 修改代码评审报告

### 一、改动概览
- 涉及文件与变更类型
- 主要改动摘要(1–3 句)

### 二、评审结论(按维度)
- 正确性:…
- 可读性:…
- 可维护性:…
- 性能:…
- 安全:…
- 测试:…

### 三、最佳实践建议
- [必须改] …
- [建议改] …
- [可选优化] …

### 四、修改方案性价比
- 方案/建议 1:收益 vs 成本 → 性价比结论
- (其他主要方案按需)

### 五、代码评分
- **Blockers(必须改,不改不能合)**:…(无则写"无")
- **分数**:XX / 100(等级:X)
- **一句话总结**:…

注意事项

  • 评审基于用户提供的 diff/文件/上下文,不臆测未展示的代码
  • 与项目技术栈、既有风格一致(如 Next.js、TypeScript、Zustand、Tailwind 等)
  • 性价比与评分需简要说明依据,便于用户决策

相关专题

更多
Vite 项目如何打包和部署
Vite 项目如何打包和部署

Vite项目打包部署专题汇集生产构建、产物预览、输出目录、资源路径、Nginx与GitHub Pages部署及常见白屏和路由404问题的已审核文章,帮助读者完成上线流程。

2026.10.28

0

16

Vite 如何搭配 Vue.js
Vite 如何搭配 Vue.js

Vite搭配Vue.js专题汇集创建Vue 3项目、配置开发环境、路径别名、Vue插件、代理、多环境构建与调试的已审核文章,帮助读者完成从初始化到日常开发的配置。

2026.10.28

0

15

Vite 项目目录怎么看
Vite 项目目录怎么看

Vite项目目录专题汇集入口文件、src源码、public静态资源、路径别名、项目根目录和构建输出目录相关的已审核文章,帮助读者理解各文件夹的用途与引用方式。

2026.10.28

0

10

Gradle插件开发入门教程
Gradle插件开发入门教程

本专题介绍Gradle插件开发的基本流程,涵盖插件类编写、自定义任务注册、扩展参数设计和插件应用方法,帮助你理解插件如何组织构建逻辑,减少多个项目之间的重复配置。

2026.10.22

0

11

Gradle Wrapper配置教程
Gradle Wrapper配置教程

本专题介绍Gradle Wrapper的基本作用,涵盖文件组成、生成方法、版本配置和任务调用,帮助你理解项目如何使用指定的Gradle版本,减少不同开发环境带来的构建差异。

2026.10.22

0

10

Golangci-lint检查器使用教程
Golangci-lint检查器使用教程

本专题整理Golangci-lint常用Linters的用途与配置方法,覆盖错误处理、无效代码、安全、复杂度、命名和依赖检查,帮助项目合理选规则。

2026.10.22

0

15

Gradle怎么构建Java项目
Gradle怎么构建Java项目

本专题介绍Gradle Java项目构建方法,涵盖Java插件配置、依赖声明、源码编译、测试执行和打包检查,帮助你理解各阶段的任务关系,逐步完成项目构建。

2026.10.22

0

10

PHP时间戳漏洞修复教程
PHP时间戳漏洞修复教程

本专题汇总PHP常见时间戳漏洞,包括2038年溢出、时区漂移、时间戳伪造、重放攻击、时间盲注与日志审计失效等,并给出升级64位环境、使用 DateTimeImmutable、统一UTC、严格超时校验、预处理绑定、hash_equals 比对等修复方案,帮助开发者快速排查和加固时间相关代码。

2026.10.22

0

25

Golangci-lint配置文件与规则设置
Golangci-lint配置文件与规则设置

本专题聚焦Golangci-lint YAML配置文件,讲清运行参数、规则启停、目录排除、问题过滤及严重级别设置,便于团队统一检查标准。

2026.10.22

0

15

热门下载

更多
网站特效
/
网站源码
/
网站素材
/
前端模板

精品课程

更多
热门推荐
/
最新课程
phpStudy极速入门视频教程
phpStudy极速入门视频教程

共6课时 | 54.7万人学习

独孤九贱(4)_PHP视频教程
独孤九贱(4)_PHP视频教程

共89课时 | 134.1万人学习