- （void）dealloc質問

Question

次のコードが100％正しいかどうか教えてください。 Expecially 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を指し示していても指していなくてもかまいません。これにより、いくつかの非常に奇妙なバグ、通常ランダムなオブジェクトがリリースされる可能性があります。

あなたのクラスは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 = ...を使用して）SwitchViewalloc/autoreleaseで

シーケンスは、あなたのルーチンはルーチン（と少し超えた）の長さのために所有権を保持していることになります。 self.SecondController =部分は、(nonatomic,retain)と宣言したので、クラスにSecondControllerオブジェクトが保持されていることを確認します。

Answer 2

プロパティセッターを使用してSecondControllerを割り当てる必要があります。

私はSwitchViewでそれを示し、その後、唯一alloc/initあなたを示唆して一度コントローラを表示するには、その：

#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 

この方法でそれを作成する-SwitchViewが呼び出されるたびに対立するものとして、あなただけ実際に、一度そのSecondControllerビューコントローラを作成。