用FauxPas找到潜在Bug
Faux Pas是一个Xcode辅助工具,检查iOS或者Mac项目,找到潜在bug,以及可维护性和风格问题。提供了92条规则可供挑选,支持GUI和CLI。目前发布了beta版,提供30天试用。
使用GUI扫描了项目,发现了许多不少有价值的问题,值得我们学习。
- 编译参数的有益建议,比如-DNDEBUG
- 多处图片缺少和图片重复
- 非retian与retina图尺寸不是2倍
- 没有前缀或者前缀与SDK冲突,建议至少三字符长度
- Retain Delegate
- 重载或者调用一些限制的系统方法
- 缺少获取授权的描述
- Category方法冲突
- 未使用的Error值,建议不关心Error时参数传入NULL
- 设置delegate或者datasource为self时需要在dealloc中置为空
- 对指针和0做比较
- obj ? obj : other obj将会评估2次。而obj ?: other会评估一次
- 界面字符串缺少本地化,我们的项目实际不需要
- 修改传入参数的值
- model类包含的对象的@property建议用copy,防止非setter的修改
- pch中引用过的头文件再次引用多余
- VCS建议,例如建议Xcode workspace data不忽略
- 在init or dealloc不要使用setter。
- 不和习惯的getter。使用something而非getSomething
- 旧语法,例如+[NSNumber numberWithFloat:], -[NSDictionary objectForKey:]。建议转换为现代语法Edit → Refactor → Convert To Modern Objective-C Syntax