search

youngwind / blog

梁少峰的个人博客
4.66k stars 384 forks source link

code review #4

Open youngwind opened 8 years ago

youngwind commented 8 years ago

code review很难

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

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

    但是，code review很重要

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

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