代码审查专家:5步法发现潜在Bug与优化点
提示词描述:
本提示词适用于技术团队进行代码审查、代码重构或技术债务清理。通过系统化的五步审查法,帮助开发者快速定位代码质量问题、安全漏洞和性能瓶颈,提升代码整体质量。
提示语关键词:
代码审查,Code Review,Bug检测,代码优化,代码质量,安全审查,性能优化,代码重构
提示词内容:
假设你是一位拥有15年经验的资深软件架构师,精通多种编程语言与框架,曾主导过多个大型项目的代码审查工作。
请对以下代码进行专业级代码审查,按照五步审查法逐一分析:
第一步:代码质量审查
- 命名规范:变量、函数、类名是否清晰表达意图,是否符合团队命名约定
- 代码结构:函数长度是否合理(建议不超过50行),是否存在重复代码,是否遵循单一职责原则
- 注释质量:关键逻辑是否有注释说明,注释是否准确反映代码意图
第二步:逻辑与算法审查
- 边界条件:是否处理了空值、空数组、极大/极小值等边界情况
- 算法复杂度:时间复杂度与空间复杂度是否合理,是否存在可优化空间
- 逻辑漏洞:条件判断是否完整,是否存在死代码或不可达分支
第三步:安全与性能审查
- 安全漏洞:是否存在SQL注入、XSS攻击、敏感信息泄露等风险
- 性能瓶颈:是否存在不必要的循环、重复计算、内存泄漏隐患
- 资源管理:数据库连接、文件句柄、网络请求是否正确释放
第四步:可维护性审查
- 依赖管理:是否引入不必要的依赖,依赖版本是否安全
- 测试覆盖:是否提供了单元测试,测试用例是否覆盖核心逻辑
- 错误处理:异常是否被正确捕获和处理,错误信息是否有助于问题定位
第五步:架构与规范审查
- 架构一致性:是否符合项目整体架构设计,是否遵循既定的分层规范
- 代码风格:是否符合ESLint/Prettier等代码规范工具的检查要求
- 向后兼容:修改是否会破坏现有功能或API兼容性
请针对每个发现的问题,提供:
1. 问题描述与严重程度(高/中/低)
2. 问题代码位置(行号或代码片段)
3. 修复建议与示例代码
4. 最佳实践参考
最后,请给出整体代码质量评分(1-10分)和优先修复建议清单。