Review时, 找寻的目标..
假定项目/公司既有代码规范, coder遵从or不遵从规范都是显而易见的.
故不评审代码样式/风格/规范.
找寻的目标, 应该是潜在bug和有性能问题的代码.
一些ObjC审查问题
代码层面的基础问题:
- 内存泄漏; 野指针问题.
- 线程安全.
- 循环依赖.
- 集合类型“越界”可能.
- 条件判断等的边界考虑.
代码组织设计上的问题:
- 类的单一职责原则;
- 方法设计, 与其他的耦合关系;
- 模块的设计问题;
- 算法类方法的瓶颈审查.
假定项目/公司既有代码规范, coder遵从or不遵从规范都是显而易见的.
故不评审代码样式/风格/规范.
找寻的目标, 应该是潜在bug和有性能问题的代码.
代码层面的基础问题:
代码组织设计上的问题: