前端代码审查清单

是什么?为什么?

前端代码审查清单是一个保证前端代码质量的审查清单。当我们在开发写代码的时候,总会各种各样的问题,自测的时候由于太熟悉自己的代码逻辑往往测试不够充分,无法发现问题。

前端代码审查清单就是为了解决这个问题!清单存放了一些常见的问题,当我们开发完成之后,对照清单思考一下这些问题在代码中是否遇到或者妥善处理,从而提高代码质量。

前端

前端安全

  • 所有的用户可以在页面中输入信息的地方,是否做了防 XSS 以及特殊字符的过滤处理?
  • 与后端接口交互,获取信息使用 GET 方式,传送信息使用 POST 方式。后端接口应对各项参数做校验。前端也要判断接口是否返回合法、正确。
  • 开发与 DOM 操作有关的代码时,是否对 DOM 不存在或者被人为修改的情况做处理?
  • 获取数据和信息时,是否对类型做过处理和转换并设置为空时的默认值?比如:var num = parseInt(Str);
  • 在所有会发生错误的地方,是否编写了错误处理逻辑?比如:阻止继续执行、显示错误信息、记录错误日志和信息等。
  • 代码里获取 window.location 相关属性的地方,是否对里面的 XSS 字符做了过滤处理?

前端性能

  • JS 代码是否尽量放在底部?CSS 代码是否尽量放在了顶部?
  • 是否部署 CDN 或者开启了缓存功能?
  • 上线或者发布前,是否对静态资源进行打包、压缩处理?
  • 正确使用预加载、懒加载等技术手段提高性能。
  • 是否对图片等资源进行压缩以及 CSS Sprite 处理?

代码质量

  • 你的代码是否遵循团队要求的代码规范?
  • 是否有冗余代码没有注释掉或者删掉?例如:删除或者注释 console.log 避免低端 IE 报错等。
  • 关键功能是否还有优化的空间?
  • 代码是否简单易懂,逻辑清晰,模块化?
  • 变量名是否简单易懂?是否拼写正确?
  • CSS 属性是否有拼写错误?
  • HTML 标签是否书写正确,是否嵌套正确?
  • JS 代码是否经过 JSLint 或者 ESLint 等工具校验?
  • 是否将核心功能尽可能独立,从而避免其他功能出现问题影响到核心功能?

注释

  • 是否在重要功能附近添加合适的注释?
  • 注释是否包含了开发人员信息、开发时间、开发者联系方式以及相关功能说明?
  • 换位思考,你能根据你的注释推断出下面代码的功能吗?
  • 代码里是否还存在 TODO ?是否可以删掉或者完善功能?
  • 可能产生意外情况的地方是否留下说明?

测试

  • 代码逻辑是否正确、可用、符合需求?
  • 在进行各种操作的时候,是否有报错出现?
  • 是否有资源加载出错或者失败?
  • 是否按照项目要求,使用相关设备以及浏览器进行测试和体验?
  • 是否对边界条件以及看起来比较极端的情况做过测试?

故障处理

  • 是否考虑过如果发生线上故障,如何做回滚处理?处理什么文件?
  • 功能与功能之间是否足够独立?是否设置开关?当某个功能发生故障是否可以通过开关关掉?

贡献

由于本人才疏学浅,部分场景没有覆盖,欢迎大家补充更多审查点,提高前端代码质量!

欢迎留言

28 条评论

  1. 赞!我来打卡了!谢谢分享⌇●﹏●⌇

    Windows 10 x64 Firefox 79.0