预测试
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自己的代码