OBJ-C：如何释放从方法返回的对象？

Question

我对从方法返回的对象的内存警告感到困惑。 这是我的代码。

-(void)returnHeaderView
{
  self.headerView=[[UIView alloc]init];
  headerView.frame=CGRectMake(0, 0, 955, 45);


  UILabel *fromLabel=[self returnLabel];
  fromLabel.frame=CGRectMake(400, 5, 200, 44);
  fromLabel.text=@"Open Time";
  [headerView addSubview:fromLabel];
  [fromLabel release];(in correct decrement of the reference count of an object that is not owned at this point by the caller)

 [self.headerView addSubview:fromLabel];
 [self.view addSubview:self.headerView];
 [self.headerView release];

}

-(UILabel *)returnLabel
{
UILabel *label= [[UILabel alloc] init] ;
label.textColor = [UIColor blackColor];
label.font = FONT_TITLE;
label.numberOfLines=1;
label.textAlignment=UITextAlignmentLeft;
label.lineBreakMode=UILineBreakModeWordWrap;
label.backgroundColor=[UIColor clearColor];

return label;

}

这是我的两种方法。 1.-(void)returnHeaderView。 2.-(UILabel *)returnLabel.

-returnLabel 是返回标签，方法返回的 UILabel 对象的引用传递给 returnHeaderView 方法的 fromLabel UILabel 对象。 然后我释放 fromLabel 对象。

但它给出了内存警告（ 正确减少调用者此时不拥有的对象的引用计数）。

所以有人建议我这段代码有什么问题。 以及如何释放方法返回的对象。

谢谢。

Answer 1

在您的returnHeaderView 中，您实际上并没有保留fromLabel，因此您不拥有它，也不应该在那里释放它（警告“不正确地减少不属于的对象的引用计数调用者的这一点”）。

最好的选择是自动释放返回的标签：return [label autorelease];，让它一直存在，直到周围的自动释放池被耗尽。请注意，这也意味着从您的 returnHeaderView 方法中删除 [fromLabel release]; 调用。

如果这听起来很复杂，请按照实际推荐的方式使用 ARC。

Answer 2

请进行小改动并删除泄漏并自动释放 self.headerView 以删除其他内存泄漏警告：

-(void)returnHeaderView
{
  self.headerView=[[[UIView alloc]init] autorelease];
  headerView.frame=CGRectMake(0, 0, 955, 45);

  UILabel *fromLabel = [[self returnLabel] retain];
  fromLabel.frame    = CGRectMake(400, 5, 200, 44);
  fromLabel.text     = @"Open Time";
  [headerView addSubview:fromLabel];
  [self.headerView addSubview:fromLabel];
  [self.view addSubview:self.headerView];
  //[self.headerView release];
}

-(UILabel *)returnLabel
{
  UILabel *label= [[UILabel alloc] init] ;
  label.textColor = [UIColor blackColor];
  label.font = FONT_TITLE;
  label.numberOfLines=1;
  label.textAlignment=UITextAlignmentLeft;
  label.lineBreakMode=UILineBreakModeWordWrap;
  label.backgroundColor=[UIColor clearColor];
  return [label autorelease];
}

Answer 3

在我看来，您的 returnLabel 方法返回了一个保留计数为 1 的对象，而不是编译器建议的零。我认为您可能只需要更改方法名称即可向编译器提示返回对象的所有权。尝试将returnLabel 更改为createReturnLabel。那么该方法将符合copy/create naming convention。

Answer 4

您可以通过更改代码来避免泄漏：

-(void)returnHeaderView
{
  self.headerView=[[UIView alloc]init];
  headerView.frame=CGRectMake(0, 0, 955, 45);


  UILabel *fromLabel = [[self returnLabel] retain];
  fromLabel.frame    = CGRectMake(400, 5, 200, 44);
  fromLabel.text     = @"Open Time";
  [headerView addSubview:fromLabel];
  [self.headerView addSubview:fromLabel];
  [self.view addSubview:self.headerView];

  [fromLabel release];
  [self.headerView release];

}

-(UILabel *)returnLabel
{
UILabel *label= [[UILabel alloc] init] ;
label.textColor = [UIColor blackColor];
label.font = FONT_TITLE;
label.numberOfLines=1;
label.textAlignment=UITextAlignmentLeft;
label.lineBreakMode=UILineBreakModeWordWrap;
label.backgroundColor=[UIColor clearColor];

return [label autorelease];

}

建议：避免此类泄漏的最佳方法是使用 ARC。