SCRM 2020-07

【Code Review】SCRM前端2020-07-11

本期 Review 发现及总结以下事项:

  1. 多条 style 应写成 class,优先使用已定义变量

  2. 类型定义完整,尽量不使用 any

  3. 重复计算操作应保存变量, 重复代码应考虑封装复用

  4. 遵循 eslint、ts 规则,而不是屏蔽报错

  5. 需要在指定组件才执行的方法,应该放到组件内执行

  6. 正确理解和规范使用 api,避免多余繁琐写法、留下 bug 隐患、降低可读性

  7. 页面内容较多时应考虑组件拆分

  8. 数据请求减少非必要阻塞,优化页面响应速度

  9. 复杂 if/else 逻辑判断可考虑策略模式优化

  10. 命名统一约定使用,通用组件要提高复用率

  11. 废弃无用代码要删除,必要注释要写上

  12. 避免不必要闭包

  13. 正确设置 useEffect 依赖

最后更新于