Java后端代码审查规范

Java后端代码审查规范一、总则（一）目的明确。为规范Java后端代码审查工作，提升代码质量与系统稳定性，特制定本规范。（二）适用范围。本规范适用于公司所有Java后端项目，包括但不限于新项目开发、现有系统维护及重构等场景。二、审查原则（一）权责划定。各单位主要负责人是第一责任人，技术负责人承担直接管理责任，开发人员需严格执行审查流程。（二）标准统一。所有代码审查必须遵循公司统一编码规范，确保技术标准的一致性。（三）流程规范。代码审查需通过指定工具平台完成，严禁线下审查或口头确认。三、审查流程（一）提交规范。开发人员完成单元测试后，需在平台提交代码审查申请，附测试用例及功能说明。（二）分配机制。技术负责人根据代码复杂度分配审查人，核心模块需至少两人交叉审查。（三）审查时效。代码提交后48小时内必须完成审查，特殊情况需提前报备。四、审查内容（一）代码结构。1.类与方法命名需符合驼峰命名法，类名首字母大写，方法名首字母小写。2.单类文件行数不超过500行，方法行数不超过100行。3.接口与实现类需保持同步修改。（二）性能要求。1.核心业务接口响应时间不得超过200ms，慢查询SQL需标注优化方案。2.缓存命中率应达到85%以上，热点数据必须配置二级缓存。3.高并发场景下，线程池核心数需根据CPU核数设置。（三）安全规范。1.所有用户输入必须进行参数校验，禁止直接拼接SQL。2.敏感数据（如密码、卡号）需进行加密存储，传输时使用HTTPS协议。3.定期进行安全漏洞扫描，高危问题需72小时内修复。五、审查标准（一）异常处理。1.业务异常需自定义异常类，系统异常使用RuntimeException。2.异常信息需包含错误码与详细描述，禁止抛出空异常。3.关键操作需添加全局异常捕获，防止程序崩溃。（二）日志规范。1.核心业务操作必须记录操作日志，包括用户ID、操作时间、参数值等。2.错误日志需包含堆栈信息与错误级别，禁止手动删除日志。3.日志文件需定期清理，保留周期不少于90天。（三）版本控制。1.提交代码必须添加详细注释，说明修改原因与影响范围。2.分支命名需符合规范，如feature/模块名/问题描述。3.合并前必须执行分支检测，确保无冲突。六、审查工具（一）平台使用。1.审查流程必须通过GitLabCodeReview完成，禁止使用邮件或聊天工具。2.代码冲突需在平台解决，历史记录需保留至少6个月。（二）插件配置。1.ESLint插件必须配置标准规则，禁止手动关闭校验。2.SonarQube需开启全部安全规则，低风险问题除外。（三）自动化检查。1.提交代码前必须执行静态检查，问题未解决不得提交。2.自动化覆盖率不得低于80%，核心模块需达到90%。七、责任追究（一）处罚机制。1.未通过审查直接上线，对项目负责人罚款500元，技术负责人罚款1000元。2.审查人未按期完成审查，取消当月评优资格。（二）改进措施。1.连续三个月审查问题率超过5%，需参加技术培训。2.重大问题（如导致系统崩溃）需提交书面检讨，并通报全公司。（三）申诉流程。1.对审查结果有异议，可在平台提交申诉，由技术委员会复核。2.复核结果为最终决定，不得再次申诉。八、持续改进（一）定期评估。每月组织代码质量分析会，通报审查问题趋势，调整审查标准。（二）知识沉淀。1.典型问题需整理成案例库，供新人学习。2.每季度更新审查规范，发布新版指南。（三）技术分享。1.开发团队每月组织代码分享会，交流最佳实践。2.邀请外部专家进行技术培训，提升审查能力。九、附则（