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

Question

我已经看过了苹果“的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) 

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

Answer 1

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

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

我会做它像这样：

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

Answer 2

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

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

要在发布后获取参考。

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