项目中的走查
项目中的CodeReview
之前开发项目中的CodeView为人工走查,还有一种自动化走查方式,常用的有OLCInt
,其中已经自动包含一些规则了,结合xcode就可以自动化走查。一些较大的项目中,走查会结合情境分析逻辑等,或者一些规格要求无法打入规则中,这时候一般采用人工走查
常见的走查:
- 代码走查
- 一些if嵌套是否必要,是否重复;
- 是否有多余未使用的常量变量,注释,废弃代码,大段注释代码等;
- 属性类型是否错误,例如代理用weak、内存是否有泄漏,例如循环引用,闭包内未采用__weak等;
- 一些代码优化,例如是否可以减少变量,用系统自身的值(例如cell数据源里面的判断,可以用inexpath判断,就不必再单独取出这个cell判断);
- 一些数据越界判断处理,例如cell数据源方法最好坐下判断,当前所取的值是否在当前数据源方法内;
- 一些逻辑上/需求上必须优化的点(例如集合竞价每3秒请求一次改为只在竞价期请求);
- 一些代码位置的存放,例如资源文件(图片,html等)放到自己的
Resource
文件夹中,该写的代码放到各自的模块儿中(例如网络请求放到XXXX模块儿中,router跳转的模块儿字符串放到XXXX模块儿中,部分竞价绘制放到XXXX模块儿中等等); - 有没有代码污染,强耦合性;
- UI走查
代码是否匹配原型图,像素点,颜色等是否符合预期;