- (void)dealloc 问题

Question

您能告诉我以下代码是否 100% 正确吗？特别是dealloc 部分

FirstViewController.h

#import <UIKit/UIKit.h>
#import "SecondViewController.h"

@class SecondViewController

@interface FirstViewController : UIViewController
{
    SecondViewController   *SecondController;
}

- (IBAction)SwitchView;

@property (nonatomic, retain) IBOutlet SecondViewController *SecondController;

@end

FirstViewController.m

#import "FirstViewController.h"

@implementation FirstViewController

@synthesize SecondController;

- (IBAction)SwitchView
{    
    SecondController = [[SecondViewController alloc] initWithNibName:@"SecondViewController" bundle:nil];
    SecondController.modalTransitionStyle = UIModalPresentationFullScreen;
    [self presentModalViewController:SecondController animated:YES];
    [SecondController release];
}

/// OTHER CODE HERE ///

- (void)dealloc
{
    [SecondController release];
    [super dealloc];
}

@end

谢谢！

Answer 1

不，这是不正确的。您将release 消息发送到dealloc 中的指针，但该指针可能不再指向SecondController。这可能会导致一些非常奇怪的错误，通常是随机对象被释放。

在 Objective-c 术语中，您的类不保留（认为“拥有”）SecondController，因此它不应该首先尝试在dealloc 上释放它。

要以正确的方式声明和释放所有权，请这样做：

- (IBAction)SwitchView
{    
    self.SecondController = [[[SecondViewController alloc] 
                  initWithNibName:@"SecondViewController" bundle:nil] autorelease];
    self.SecondController.modalTransitionStyle = UIModalPresentationFullScreen;
    [self presentModalViewController:self.SecondController animated:YES];
}

/// OTHER CODE HERE ///

- (void)dealloc
{
    self.SecondController = nil;
    [super dealloc];
}

这也将保护您免受SwitchView 和dealloc 之间发生的任何其他事情的影响。 （只要那东西遵守规则并使用self.SecondController = ...更改属性）

在SwitchView 中，alloc/autorelease 序列使您的例程在例程的长度（以及稍长一点）内保持所有权。 self.SecondController = 部分确保您的类保留 SecondController 对象，因为您声明了 (nonatomic,retain)。

Answer 2

您应该使用属性设置器来分配SecondController。

我建议你只 alloc/init 那个视图控制器一次，然后在 SwitchView 显示它：

#import "FirstViewController.h"

@implementation FirstViewController

- (id)initWithNibName:(NSString *)nibName bundle:(NSBundle *)nibBundle {
   if((self = [super initWithNibName:nibName bundle:nibBundle])) {
      self.SecondController = [[[SecondViewController alloc] initWithNibName:@"SecondViewController" bundle:nil] autorelease];
      SecondController.modalTransitionStyle = UIModalPresentationFullScreen;
   }
   return self;
}

- (IBAction)SwitchView
{    
    [self presentModalViewController:SecondController animated:YES];
}

/// OTHER CODE HERE ///

- (void)dealloc
{
    [SecondController release];
    [super dealloc];
}

@end

这样，您实际上只创建了一次 SecondController 视图控制器，而不是每次调用 -SwitchView 时创建它。