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 .-（无效）returnHeaderView。 2 .-（UILabel *）returnLabel。

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

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

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

谢谢。

Answer 1

在您的returnHeaderView您实际上并不保留fromLabel ，因此您不拥有它并且不应该在那里释放它（警告“调用者此时不拥有的对象的引用计数的错误减少”） 。

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

如果这听起来很复杂，请按照实际建议使用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 。 然后该方法将符合复制/创建命名约定 。

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。