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

Question

我查看了 Apple“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

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

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

释放后获取引用。

或者我想另一种方法是使用自动释放而不是释放。

Answer 2

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

错了吗？也许，它需要知道NSArray 做了什么，但这很常见。

我会这样做的：

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