Skip to content

code review #4

Open
Open
@youngwind

Description

@youngwind

code review很难

code review很难,主要基于以下原因。

  1. 太忙了,连新需求都没完成,哪有时间review。
  2. 惯性,培养新的良好习惯是需要一番挣扎的。

但是,code review很重要

除了一般的HTML、CSS、JS规范,在进行 code review的时候还需要注意以下几点。

  1. 代码是否简单易懂?
  2. 是否存在多余或者重复的代码?
  3. 代码是否尽可能地模块化(组件化)?
  4. 是否有被注释掉的代码?
  5. 是否有可以被库函数代替的代码?
  6. 是否有可以删除的日志或调试代码?
  7. 无效的参数值是否能够处理?
    ……

Metadata

Metadata

Assignees

No one assigned

    Labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions