分类
dev

Code Review (ObjC for iOS project)

Review时, 找寻的目标..

假定项目/公司既有代码规范, coder遵从or不遵从规范都是显而易见的.

故不评审代码样式/风格/规范.

找寻的目标, 应该是潜在bug和有性能问题的代码.

一些ObjC审查问题

代码层面的基础问题:

  1. 内存泄漏; 野指针问题.
  2. 线程安全.
  3. 循环依赖.
  4. 集合类型“越界”可能.
  5. 条件判断等的边界考虑.

代码组织设计上的问题:

  1. 类的单一职责原则;
  2. 方法设计, 与其他的耦合关系;
  3. 模块的设计问题;
  4. 算法类方法的瓶颈审查.