内存泄漏，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] 时对象被释放，

提前致谢，

Nico

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 中释放它们。

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

Answer 2

我相信这是因为 CellModel 拥有该对象。尽管您已经释放了它，但该对象还没有。要修复此问题，请添加代码以在 dealloc 方法中释放数组。

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

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