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
  • 国网淘宝天猫解析

国网淘宝天猫解析 · Changes

Page history
update: 同花顺财经-基础信息 authored Nov 18, 2021 by 袁波's avatar 袁波
Hide whitespace changes
Inline Side-by-side
Showing with 27 additions and 3 deletions
+27 -3
  • code-review/国网淘宝天猫解析.md code-review/国网淘宝天猫解析.md +27 -3
  • No files found.
code-review/国网淘宝天猫解析.md
View page @ d10b947b
## 评审目标 ## 评审目标
```
1. 实现方案的正确性 1. 实现方案的正确性
2. 代码的会味道 2. 代码的会味道
3. 规范性 3. 规范性
```
### 流程 ### 流程
```
1. 被评审人需先整体描述需要解决的问题、解决流程 (被评审人讲解过程中,评审人可以记录问题,不要打断被评审者的思路) 1. 被评审人需先整体描述需要解决的问题、解决流程 (被评审人讲解过程中,评审人可以记录问题,不要打断被评审者的思路)
2. 被评审人讲完,评审人和与会人员可以提问题 2. 被评审人讲完,评审人和与会人员可以提问题
3. 评审人进行评审 (被评审者或者与会人员记录评审待改进的内容,有时并不是只针对被评审者,而是所有编码者) 3. 评审人进行评审 (被评审者或者与会人员记录评审待改进的内容,有时并不是只针对被评审者,而是所有编码者)
4. 评审完成之后,落实待修改项,主要是缺陷和规范性 4. 评审完成之后,落实待修改项,主要是缺陷和规范性
```
### 代码的坏味道 ### 代码的坏味道
```
1. 无效代码 1. 无效代码
a、321行remain_cnt不会小于0,最小为0,当remain_cnt为0时就没有任务了,此时程序break,337行if语句冗余 a、321行remain_cnt不会小于0,最小为0,当remain_cnt为0时就没有任务了,此时程序break,337行if语句冗余
...@@ -47,19 +54,36 @@ ...@@ -47,19 +54,36 @@
b、处理并发不够完美,不同进程会拿到相同任务,解决办法使用队列,一个进程写任务,多个进程消费任务 b、处理并发不够完美,不同进程会拿到相同任务,解决办法使用队列,一个进程写任务,多个进程消费任务
```
### 规范性 ### 规范性
```
1. 缺少项目文档 (app_environ_protection_grade/readme.md) 1. 缺少项目文档 (app_environ_protection_grade/readme.md)
2. data_pump配置文件,缺少流程必要注释说明, 缺少profile (方便随时测试、复现) 2. data_pump配置文件,缺少流程必要注释说明, 缺少profile (方便随时测试、复现)
```
--- ---
### 改进落实 ### 改进落实
```
缺陷: 缺陷:
规范性: 规范性:
时间: 时间:
负责人: 负责人:
\ No newline at end of file ```
### 已改进问题
``````
缺陷:
规范性:
时间:
负责人:
```
```
\ No newline at end of file
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