【算法】Code Review

1.架构/设计

• 单一职责原则

一个类只干一件事情,一个方法只干一件事情,常见的违背:类即干UI,又干逻辑

• 行为是否统一

缓存/错误处理/错误提示/弹出框是否统一......解决为:kit下新建文件夹,所有缓存全放里面

• 代码污染

学习解耦和则去参考第三方框架,如果想使用AFN,最好建个单例来使用,若有十个模块需要网络请求,在十个模块里面写了AFN,若有一天AFN不更新了,需要改十个模块的代码,应该在十个类里面调自己的网络工具类,而自己的网络工具类封装了AFN,若以后有改动,只需要在自己的网络工具类里面改一个地方就行
代码有没有对其他模块强耦合,比如我的模块依赖于其他模块,如果其他模块不见了,我的就不行了

• 重复代码

• 开闭原则

• 面向接口编程

• 健壮性

是否考虑线程安全
数据访问是否一致性
边界处理是否完整
逻辑是否健壮
是否有内存泄露
有没有循环依赖
有没有野指针

• 错误处理

改动是不是对代码的提升(你的改动是只是加了if来打补丁,让代码质量继续恶化,还是对代码质量做了修复)

• 效率/性能

  关键算法的事件复杂度是多少,有没有可能有潜在的性能瓶颈(上传图片的时候,从相册选择的过程会有问题,因为图片很大,当打开图片的时候,我们的内存会增加10M的图片,会有内存问题)
  客户端程序对频繁消息和较大数据等耗时操作是否处理得当

2.代码风格式

• 可读性

• 命名

• 函数长度/类长度

• 注释

• 参数个数:最好不要超过3个

每天晚上对自己的代码过一遍,补充注释,整理思路,避免低级错误