前端代码审查清单

是什么？为什么？

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

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

前端

前端安全

• 所有的用户可以在页面中输入信息的地方，是否做了防 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 ？是否可以删掉或者完善功能？
• 可能产生意外情况的地方是否留下说明？

测试

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

故障处理

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

贡献

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