内存管理：添加到 NSMutableArray 后释放 viewcontroller 对象

Question

我有以下代码的内存泄漏。如果我使用 [sub release]; 将 sub 添加到 NSmutableArray(subViewController) 后，分析器会显示“调用者此时不拥有的对象的引用计数的不正确递减” ，当我删除 [sub release] 时，它会显示“在 xx 行分配的对象的潜在泄漏”

for (int i=0; i<[self.data count]; i++) {

   SubCategoryViewController *sub =[[SubCategoryViewController alloc] 
           initwithServiceUrl:urlString andHeaderValue: 
          ((PMCategory *)[self.data objectAtIndex:i]).categoryName];

   [[AppDelegate sharedAppDelegate].viewController.subViewControllers addObject:sub];
   [sub release];


}

另外，如果我使用 autorelease，警告将变为 “对象已发送-autorelease 太多次”

SubCategoryViewController *sub =[[[SubCategoryViewController alloc] 
           initwithServiceUrl:urlString andHeaderValue: 
          ((PMCategory *)[self.data objectAtIndex:i]).categoryName]autorelease];

从评论中添加： SubCategoryViewController 初始化方法：

@property (nonatomic, retain) NSMutableArray *data;
@property (nonatomic, copy) NSString *headerText;
@synthesize data = _data;
@synthesize headerText=_headerText;

...

self = [super init];
if (self) {
    self.data = [[NSMutableArray alloc] init] ;
    self.headerText =headerValue;
    self.serviceURL =serviceU;
    self.firstLoad = YES;
}
return self;

Answer 1

这是因为你没有遵循 Objective c 的正确命名约定。每当你编写任何初始化函数时，命名约定就像是

-(id) initFirstWordSecondWord{
}

在 init 大写后面加上第一个字母。

因此，将您的 initwithServiceUrl 更改为 initWithServiceUrl ，您的问题将得到解决。
开心点！！！

Answer 2

如果 data 是具有 retain 属性的属性，则它会被过度保留，一次由：

[[NSMutableArray alloc] init]

，一次由属性设置者。

将其更改为使用返回自动释放的 NSMutableArray 的类便捷方法：

self.data = [NSMutableArray array];

此外，可以通过快速枚举更清晰地编写 for 循环：

for (PMCategory *category in self.data) {
    SubCategoryViewController *sub =[[SubCategoryViewController alloc] 
        initwithServiceUrl:urlString
        andHeaderValue:category.categoryName];