iOS算法篇-Code Review

1.架构/设计

  • 单一职责原则

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

  • 行为是否统一

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

  • 代码污染

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

  • 重复代码
  • 开闭原则
  • 面向接口编程
  • 健壮性

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

  • 错误处理

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

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

2.代码风格式

  • 可读性
  • 命名
  • 函数长度/类长度
  • 注释
  • 参数个数:最好不要超过3个

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

最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
平台声明:文章内容(如有图片或视频亦包括在内)由作者上传并发布,文章内容仅代表作者本人观点,简书系信息发布平台,仅提供信息存储服务。

推荐阅读更多精彩内容

  • Android 自定义View的各种姿势1 Activity的显示之ViewRootImpl详解 Activity...
    passiontim阅读 175,734评论 25 709
  • 发现 关注 消息 iOS 第三方库、插件、知名博客总结 作者大灰狼的小绵羊哥哥关注 2017.06.26 09:4...
    肇东周阅读 14,230评论 4 61
  • 我的标签是:可爱活泼、善于交流沟通?just so? 不,绝对不是~可以活力满满,但并不是精力过剩~ 我是刚毕业的...
    篱笆的嫩阅读 877评论 0 0
  • 接触敏捷方法到现在已经有5个月的时间左右,期间我们也遵循敏捷开发模式进行项目的开发。另外就是因为在面试过程中被提问...
    _LG_阅读 3,816评论 2 3
  • 麦达数字为何震荡不前,数万股民为何愁苦不语,冀东装备走势为何如此强劲,大盘何时才能触底反弹,千股齐跌究竟是何人所为...
    北极雪路阅读 1,339评论 0 0