苹果示例代码中的错误代码？

我已经看过了苹果“的PageControl”样本项目（See here），我看到这个代码功能：苹果示例代码中的错误代码？

- (void)loadScrollViewWithPage:(int)page 
{ 
    if (page < 0) 
     return; 
    if (page >= kNumberOfPages) 
     return; 

    // replace the placeholder if necessary 
    MyViewController *controller = [viewControllers objectAtIndex:page]; 
    if ((NSNull *)controller == [NSNull null]) 
    { 
     controller = [[MyViewController alloc] initWithPageNumber:page]; 
     [viewControllers replaceObjectAtIndex:page withObject:controller]; 
     [controller release]; 
    } 

    // add the controller's view to the scroll view 
    if (controller.view.superview == nil) 
    { 
     CGRect frame = scrollView.frame; 
     frame.origin.x = frame.size.width * page; 
     frame.origin.y = 0; 
     controller.view.frame = frame; 
     [scrollView addSubview:controller.view]; 

     NSDictionary *numberItem = [self.contentList objectAtIndex:page]; 
     controller.numberImage.image = [UIImage imageNamed:[numberItem valueForKey:ImageKey]]; 
     controller.numberTitle.text = [numberItem valueForKey:NameKey]; 
    } 
}

也有一些是我不明白。
如果测试部分if ((NSNull *)controller == [NSNull null])是真的，那么我们有

controller = [[MyViewController alloc] initWithPageNumber:page]; 
[viewControllers replaceObjectAtIndex:page withObject:controller]; 
[controller release];

然后刚过：

if (controller.view.superview == nil)

但是，控制器刚刚被释放。我认为它可能会工作，因为控制器仍然在记忆中，但这不是编码这些东西的错误方式吗？

来源

2011-03-18 Oliver

当将controller对象添加到保留该对象的viewControllers中时。因此，它在释放后仍然是一个活的对象。

它错了吗？也许，它需要什么NSArray的知识，但这是相当普遍的。

我会做它像这样：

if ((NSNull *)controller == [NSNull null]) 
{ 
    controller = [[[MyViewController alloc] initWithPageNumber:page] autorelease]; 
    [viewControllers replaceObjectAtIndex:page withObject:controller]; 
    controller = [viewControllers objectAtIndex:page]; 
}

来源

2011-03-18 23:26:28 MarkPowell

+1使用autorelease的想法。就目前而言，代码不仅依赖于NSArray保留添加到其中的对象的事实（正如您所说，这很常见），而且还取决于其他线程是否从NSArray中删除对象。（不可否认，因为这些是UI对象，这不太可能发生，但继续使用不保留的对象仍然是业障。） – 2011-03-19 01:33:08

我认为你是对的。如果我正在编码，我会添加：

if ((NSNull *)controller == [NSNull null]) 
{ 
    controller = [[MyViewController alloc] initWithPageNumber:page]; 
    [viewControllers replaceObjectAtIndex:page withObject:controller]; 
    [controller release]; 
    controller = [viewControllers objectAtIndex:page]; 
}

要在发布后获取参考。

或者我想另一种方式是使用autorelease而不是release。

来源

2011-03-18 23:21:34 NWCoder

我能为downvote解释，好吗？ – 2011-03-18 23:56:22

@fluchtpunkt：我不明白这个downvote的原因...这个答案似乎是有效的...... – Oliver 2011-03-19 00:17:05

苹果示例代码中的错误代码？

回答

相关问题