iOS Code Review

CodeReview

iOS App 稳定性指标及监测(转载)

代码规范及CodeReview要点

iOS-程序员必知->代码评审(Code Review)

平安金融科技移动技术周报(专题版のCode Review)

我是如何进行code review的

评审内容

[1]架构/设计/常规
  1. 单一职责原则

这是经常被违背的原则。一个类只能干一个事情,一个方法最好也只干一件事情。比较常见的违背是一个类既干UI的事情,又干逻辑的事情,这个在低质量的客户端代码里很常见

  1. 行为是否统一,例如:
    1 . 缓存是否统一
    2 . 错误处理是否统一
    3 . 错误提示是否统一
    4 . 弹出框是否统一

  2. 代码污染
    代码有没有对其他模块强耦合

  3. 重复代码-->应该抽取

  4. 开闭原则

  5. 面向接口编程

  6. 健壮性

是否考虑线程安全
数据访问是否一致性
边界处理是否完整
逻辑是否健壮
是否有内存泄漏
有没有循环依赖
有没有野指针
是否检查了数组的“越界“错误

  1. 错误处理

  2. 改动是不是对代码的提升
    新的改动是打补丁,让代码质量继续恶化,还是对代码质量做了修复

  3. 效率/性能
    1)关键算法的时间复杂度多少?有没有可能有潜在的性能瓶颈
    2)客户端程序对频繁消息和较大数据等耗时操作是否处理得当

[2]代码风格
  1. 可读性
    衡量可读性的可以有很好实践的标准,就是 Reviewer 能否非常容易的理解这个代码。如果不是,那意味着代码的可读性要进行改进

  2. 命名
    命名对可读性非常重要
    是否跟系统属性命名造成冲突
    英语用词尽量准确一点,必要时可以查字典

  3. 函数长度/类长度
    函数太长的不好阅读
    类太长了,检查是否违反的 单一职责 原则

  4. 注释
    恰到好处的注释,不是注释越多越好

  5. 参数个数
    不要太多,一般不要超过 3 个

Review Your Own Code First

每次提交前整体把自己的代码过一遍非常有帮助,尤其是看看有没有犯低级错误

作者:船长_
链接:http://www.jianshu.com/p/289ecf08efd1
來源:

你可能感兴趣的:(iOS Code Review)