文章测试2
2025年6月9日
2 分钟
分析步骤
- 获取所有变更文件列表:
git diff [源分支] [变更分支] --name-only | cat
- 对每个变更文件查看具体改动:
git diff [源分支] [变更分支] -- [文件路径] | cat
- 分析代码变更,重点关注以下方面并按优先级评估:
-
核心业务逻辑与业务流程
-
架构设计与工程实践
-
异常处理与容错机制
-
性能与安全隐患
-
接口设计与兼容性
-
代码质量与可维护性
-
测试覆盖与可靠性
-
配置管理与环境适配
报告生成
- 创建标准化代码审查报告文件夹和文件:
mkdir -p "cr report"
- 生成报告文件,采用以下命名格式:
[YYYYMMDD] [变更分支名去除后缀] CR 报告.markdown
- 报告内容结构:
-
标题:[变更分支名去除后缀] CR 报告
-
改动点总结:简明列表
-
问题与不合理之处(按优先级分类):
P0-严重问题(必须修复)
-
逻辑错误与功能缺陷:可能导致程序不可用或结果错误的问题
-
安全漏洞:包括数据安全、权限校验等方面的缺陷
-
高风险并发问题:可能导致数据不一致、死锁的并发控制缺陷
-
严重性能瓶颈:明显会影响系统运行效率的设计或实现问题
P1-重要改进(建议修复)
-
代码健壮性问题:异常处理不足、边界情况未考虑等
-
设计缺陷:不符合最佳实践、架构不合理等问题
-
潜在性能隐患:未来可能导致性能问题的设计或实现
-
可测试性不足:难以进行单元测试或集成测试的代码结构
P2-优化建议(酌情考虑)
-
代码质量优化:重复代码、过长方法、过度复杂的实现等
-
命名与文档:不清晰的变量命名、缺少必要注释等问题
-
配置管理:硬编码配置、配置分散等问题
-
工程实践建议:项目结构、构建流程等方面的优化建议
-
每个问题分类下,使用简洁的要点列表(3-5 点),突出关键问题,并尽可能提供具体文件和行号
-
对于某一优先级没有发现问题的情况,在该优先级标题下标注"无",保持报告结构完整
输出要求
-
最终报告保存在 "cr report" 文件夹中
-
文件格式必须为 Markdown
-
内容应简明扼要,重点突出,便于开发者理解和修复
-
仅关注问题和不合理之处,无需包含优点和肯定内容
-
中文和英文字母、数字之间保留一个空格,提高可读性
-
对于每个问题,尽可能提供修复建议或解决方向
-
确保三级优先级分类始终保留,如无问题则标注"无"
评论 (0)
暂无评论,来发表第一条评论吧!