在Objective-C中保留计数和工厂方法

Question

我之前曾在这个论坛上找到过创建工厂函数以从笔尖构造自定义视图的最佳方法（ 这是我以前的文章）

我现在正在使用以下代码：

+ (LoadingV *)loadingViewCopyFromNib 
{   
    LoadingV *view = nil;   
    NSArray* nibViews = [[NSBundle mainBundle] loadNibNamed:@"LoadingV" owner:self options:nil];
    view = (LoadingV*)[nibViews objectAtIndex: 0];

    // Setting up properties
    CGRect frm = view.progress.frame;
    frm.size.width *=1.5;
    frm.size.height *=1.5;
    view.progress.frame = frm;
    view.waitLbl.text = NSLocalizedString(@"Please wait", @"");     
    return view;    <------- warning is here
}

// In .h file
...
LoadingView* loadV;
@property (nonatomic, retain) LoadingView* loadV;

// in .m file
@synthesize loadV;
...
self.loadV = [LoadingV loadingViewCopyFromNib];

当我进行构建和分析时，我收到有关工厂功能的以下警告：

/LPAPP/Classes/LoadingV.m:34:5具有+0保留计数的对象返回给调用方，其中预期+1（拥有）保留计数

为什么会这样呢？ 我知道，在函数内分配的局部变量除非保留并自动释放，否则不会超出其范围。 但就我而言，我不是在创建新对象，而只是返回对现有对象的引用。 那么，为什么我会收到警告？ 这样安全吗:)

干杯AF

Answer 1

尽管迈克是正确的，但警告的原因却完全不同。 您的方法名称包括“ copy”，它被解释为返回+1保留计数（类似于alloc，init）。 请记住，一旦过渡到ARC，这可能会引起问题！

Answer 2

当涉及到诸如NSArray类的集合类时，由objectAtIndex:和其他类似访问器返回的引用不能保证在释放父容器时保持有效。 换句话说， objectAtIndex:不返回自动发布的对象。

这意味着一旦释放了来自其的数组，返回的指针可能最终将变得无效。

要解决此问题，请在return语句中使用retain + autorelease ：

return [[view retain] autorelease];

更新：

我无法在我的Xcode版本中重现此警告。 但是Martin可能是正确的，因为您使用的GCC / clang版本对“副本”的解释不正确。 最新的Xcode和gcc / clang编译器不会出现此警告，并且规则是仅将“ copy”或“ mutableCopy”的前缀解释为返回+1保留的对象。