侧边栏壁纸
博主头像
江咏之博主等级

江一铭,字咏之,号逍遥居士。襄阳人士也,一介书生耳。喜静少言,不慕荣利,爱好甚广。诗书棋画、赏曲试茶、云游四方皆为嗜之。中闲居无他念,每以品茗自娱。而今已然淡泊名利,致力于生活,虽追理想,但无急功近利,吾辈当先沉稳矣。

  • 累计撰写 174 篇文章
  • 累计创建 3 个标签
  • 累计收到 115 条评论
标签搜索

目 录CONTENT

文章目录

前端代码 review 流程规范

江咏之
2023-02-27 / 0 评论 / 0 点赞 / 239 阅读 / 493 字 / 正在检测是否收录...
温馨提示:
本文最后更新于 2023-02-27,若内容或图片失效,请留言反馈。部分素材来自网络,若不小心影响到您的利益,请联系我们删除。
广告 广告

预测试
0、明确任务需求和实现效果(需要产品和UI协同)

1、单元测试、集成测试通过

2、todo : C2C 测试通过(验证组件)

3、本地开发环境中,eslint 代码规范检查,无报错无警告

本地测试
4、基本功能正常;界面样式正常;移动端兼容良好;谷歌浏览器显示良好(浏览器兼容性和系统兼容性尽量满足)

5、极端情况正常:错误输入(数据类型不对等)、数据量巨大(1000行数据)、请求异常处理(403-404-500)

6、代码兼容性:是否兼容老代码和老数据结构(如果不能兼容,至少界面不报错)

代码查验
7、多余的空行注释删除

8、命名规范:函数名变量名不规范、代码可读性(避免生僻Hook)

9、性能优化:减少时间空间复杂度;减少全局变量;React 中,减少 render 次数(减少不必要的state,生命周期函数优化等)

10、代码安全:代码是否存在 XSS CFRS 攻击

学习点
1、review 其他人的优点:组件化,可拓展,可复用性

2、项目整体的框架,不同组件在文件上和逻辑上的依赖关系;视图、逻辑的分离等;使用了哪些设计模式

3、大量阅读优秀项目的源码,这样才能看到自己的不足之处;让优秀同事review自己的代码

0
广告 广告

评论区