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
  • 210909

Last edited by 王鹏举 Sep 09, 2021
Page history
This is an old version of this page. You can view the most recent version or browse the history.

210909

wpj

我从这些角度来做代码审核。

设计

  • 项目地址
  • 文件夹目录结构清晰
  • 但是两个项目混在一起
  • pyreverse对类的组合支持不好

复杂性

  • field_map.py
    • 对IDE不友好
    • class
    • namedtuple
    • dataclass 推荐
  • util.py
    • ConditionDynamicSingleLink
    • Compress
  • wenshu.py
    • cookie_manager.py
  • 异步
    • 最大并发请求数限制?
    • CookieManager线程安全?

测试

有测试

命名

  • WenshuSpider
  • CookieManager
    • update_cookie_by_cookie_header
  • Condition
  • Dynamic
  • DataDumpIO
  • Decrypt

注释

  • 主干WenshuSpider中 每个函数都有注释

文档

  • 缺少README
  • kb Wiki和project之间的关系太弱
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