最近在使用Analyze分析一个老项目中的代码时,发现了很多问题。感觉其中有的问题本不该出现,当然有些问题也需要在今后的开发中多注意。现将遇到的相关错误信息总结如下。
一、Core Foundation/Objective-C(CF/OC语法)
1、The 'viewWillAppear:' instance method in UIViewController subclass 'xxx' is missing a [super viewWillAppear:] call
解释:UIViewController子类'xxx'中的'viewWillAppear:'实例方法缺少[super viewWillAppear:]调用。
分析:在viewWillAppear
中加上[super viewWillAppear:]
即可。
2、Instance variable used while 'self' is not set to the result of '[(super or self) init…]'
解释:'self'时使用的实例变量未设置为'[(super或self)init ...]的结果。
分析:改成如下即可:
self = [super initWithFrame:frame];
if (self) {
}
二、Dead store(未使用的变量)
1、Value stored to 'xxx' is never read
解释:永远不会读取存储到'xxx'的值。
分析:变量没有被使用,删除或注释掉即可。
2、Value stored to 'xxx' during its initialization is never read
解释:永远不会读取在初始化期间存储到'xxx'的值。
分析:也就是说这个model
你申请了两块内存,导致其中一块内存没有被使用。即去掉初始化方法,直接赋值即可,如下:
ACMediaModel *model = _mediaArray[indexPath.row];
三、Memory error(内存错误)
1、nil passed to a callee that requires a non-null 1st parameter
解释:nil传递给需要第一个参数非null的被调用者。
分析:代码中这里的plistPath
是这样初始化的:NSString *plistPath;
且下面只有if
判断,没有else
,这会导致参数可能为空,最好给一个初始值或者使用if
、else if
、else
判断。我们再来看一下Xcode里显示的分析过程:
false
,就会出现上述的错误,所以说这是有潜在的隐患的。
2、nil returned from a method that is expected to return a non-null value
解释:方法需要返回一个非空值,即在不改返回空的地方返回空。
分析:这里是在cellForRowAtIndexPath
方面里,只有else if
判断没有 else
判断,这可能会导致cell为空,最好把else if
改成 else
。
四、Memory(Core Foundation/Objective-C)(CF/OC内存)
1、Potential leak of an object stored into ‘xx’
我们可以具体看一下Xcode的提示信息:解释:存在xx中的对象有潜在的泄漏。
分析:这里的String
类型转CFString
类型有问题,应当在最后对string1
进行一次CFRelease
,或者使用如下方法转换:
CFStringRef string1 = (__bridge CFStringRef)str;
CFMutableStringRef string = CFStringCreateMutableCopy(NULL, 0, string1);
2、Incorrect decrement of the reference count of an object that is not owned at this point by the caller
解释:调用者此时不拥有的对象的引用计数的不正确的减少;引用计数问题。
分析:例如在上面的场景中,使用桥接之后,如果还在最后对string1
调用CFRelease
方法就会出现上述错误,由于__bridge
用于Objective-C
和Core Foundation
指针之间的转换,这种转换不会更换对象的所有权,所以无需使用调用CFRelease
方法。
3、Potential leak of an object of type xx
解释:xx类型的对象存在潜在的泄漏。
五、Logic error(逻辑错误)
1、Property of mutable type 'xxx' has 'copy' attribute; an immutable object will be stored instead
解释:对可变类型使用copy修饰,将存储不可变对象。
分析:对可变对象使用copy修饰的时候,可变对象将变成不可变对象。虽然编译不会报错,但是如果对其进行可变的相关操作就会造成crash。
2、Converting a pointer value of type 'NSNumber *' to a primitive boolean value; instead, either compare the pointer to nil or call -boolValue
解释:将类型为“NSNumber *”的指针值转换为原始布尔值;相反,要么将指针与nil进行比较,要么调用-boolValue。
分析:使用NSNumber作为判断条件的时候没有指定类型,编译器不知道怎么判断;将这个值与nil比较。
3、function call argument is an uninitialized value
解释:函数调用参数是未初始化的值。
分析:给textSize
一个初始值,如下:
CGSize textSize = CGSizeZero;
4、The right operand of '-' is a garbage value
解释: '-'的右操作数是垃圾值。
分析:同上,给textSize
一个初始值。
六、API Misuse(Apple)(API调用错误)
1、Argument to 'NSMutableArray' method 'addObject:' cannot be nil
解释:'NSMutableArray'方法'addObject:'的参数不能为空。
分析:最好给参数一个初始值或者默认值,同时在设置的时候做一个防空的判断。
2、Dictionary value cannot be nil
解释:字典里的值不能为空。
分析:同上。