Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
K
kb
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 2
    • Issues 2
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge requests 0
    • Merge requests 0
  • Operations
    • Operations
    • Incidents
  • Analytics
    • Analytics
    • Repository
    • Value Stream
  • Wiki
    • Wiki
  • Members
    • Members
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • granite
  • kb
  • Wiki
    • Code review
  • procedures

Last edited by 李林坳 Nov 04, 2021
Page history

procedures

评审目标

  1. 实现方案的正确性
  2. 代码的会味道
  3. 规范性

流程

  1. 被评审人需先整体描述需要解决的问题、解决流程 (被评审人讲解过程中,评审人可以记录问题,不要打断被评审者的思路)
  2. 被评审人讲完,评审人和与会人员可以提问题
  3. 评审人进行评审 (被评审者或者与会人员记录评审待改进的内容,有时并不是只针对被评审者,而是所有编码者)
  4. 评审完成之后,落实待修改项,主要是缺陷和规范性

代码的会味道

  1. 无效代码
  2. 重复代码
  3. 异常处理
  4. 必要注释
  5. 。。。

规范性

  1. 缺少项目文档 (app_environ_protection_grade/readme.md)
  2. data_pump配置文件,缺少流程必要注释说明, 缺少profile (方便随时测试、复现)

改进落实

缺陷:

规范性:

时间:

负责人:

Clone repository
  • README
  • basic_guidelines
  • basic_guidelines
    • basic_guidelines
    • dev_guide
    • project_build
    • 开发流程
  • best_practice
  • best_practice
    • AlterTable
    • RDS
    • azkaban
    • create_table
    • design
    • elasticsearch
    • elasticsearch
      • ES运维
    • logstash
View All Pages