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

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