内存泄漏，MutableArray 中的对象没有释放？

Question

Xcode 告诉我，下面的代码在内存泄漏方面存在一些问题。

@property (nonatomic, retain) NSMutableArray *naw_rows;

-(void) loadTableRows:(BOOL)shouldReload
{
    [naw_rows removeAllObjects];

  [self.naw_rows addObject: [[CellModel alloc] initialize:@"name"     title:@"Name"          value: self.currentProfile.name]];
  [self.naw_rows addObject: [[CellModel alloc] initialize:@"company"  title:@"Company name"  value: self.currentProfile.company]];
  [self.naw_rows addObject: [[CellModel alloc] initialize:@"address"  title:@"Address"         value: self.currentProfile.address]];
  [self.naw_rows addObject: [[CellModel alloc] initialize:@"zipcode"  title:@"Zipcode"      value: self.currentProfile.zipcode]];
  [self.naw_rows addObject: [[CellModel alloc] initialize:@"city"     title:@"City"        value: self.currentProfile.city]];
}

// here is my cellModel object:
@implementation CellModel

-(id) initialize:(NSString *)newName title:(NSString *)newTitle value:(NSString *)newValue;
{
    if (self == [super init])
    {
        name = newName;
        title = newTitle;
        value = newValue;
    }
    return self;
}

- (NSString *) getName
{
   return name;
}

- (NSString *) getTitle
{
    return title;
}

- (NSString *) getValue
{
    return value;
}

-(void)dealloc
{
    [super dealloc];
}

@end;

所有 addObject 行都给出以下错误：

在线分配的对象的潜在泄漏 -- 方法返回一个 具有 +1 保留计数的 Objective-C 对象（拥有引用）对象 在线分配 -- 稍后在此执行路径中未引用 并且保留计数为 +1（对象泄露）

在有关内存泄漏的其他主题中，我发现这是正确的方法：

CellModel *model = [[CellModel alloc] initialize:@"name" title:@"Name" value: self.currentProfile.name];
[self.naw_rows addObject: model];
[model release];

但这给了我以下错误：

不正确的对象的引用计数递减不正确 此时由调用者拥有

那么我做错了什么？在我的第一段代码中，保留计数应为 1。由数组拥有。我假设当我使用 [array remodeAllObjects] 时对象被释放

提前致谢，

尼哥

Answer 1

这泄露了：

[self.naw_rows addObject: [[CellModel alloc] initialize:@"name"     title:@"Name"          value: self.currentProfile.name]];

您拥有 alloc-init 返回的对象，并且您有责任通过向其发送 release 或 autorelease 消息放弃对它的所有权，但您没有这样做。

按照您的建议，使用临时变量确实可以解决问题：

CellModel *model = [[CellModel alloc] initialize:@"name" title:@"Name" value: self.currentProfile.name];
[self.naw_rows addObject: model];
[model release];

那么，分析仪为什么会抱怨呢？因为您的初始化程序的名称。将其重命名为 initWithName:title:value: 之类的名称，您将看到“调用者此时不拥有的对象的引用计数的错误递减”消失了。

约定是初始化方法的名称应该以缩写init开头。

此外，您的类的实现不会将 self 分配给调用 super 的初始化程序的结果。这个：

if (self == [super init])

应该是：

if ((self = [super init]))

或者，如果您愿意：

self = [super init];
if (self)

---

另外，CellModel类的实例变量的内存管理是错误的。您应该保留或复制作为参数传递给您的 init 方法的对象，并在 dealloc 中释放它们。

访问器方法也打破了naming conventions，它们应该简单命名为name、title等。前缀“get”仅用于间接返回对象的方法。

Answer 2

我相信这是因为 CellModel 拥有该对象。虽然你已经释放了它，但对象还没有。要修复此添加代码以释放 dealloc 方法中的数组。

-(void)dealloc
{
    [self.naw_rows release];
    [super dealloc];
}

另外，作为旁注，您通常不应依赖 retainCount，因为它很容易产生误导。