Apple示例代码上的代码错误？

Question

我查看了Apple"PageControl"示例项目(见这里),我在代码中看到了这个函数:

- (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];
}