Objective-C Speicherverwaltung - ziemlich sicher mache ich das alles falsch

Question

Nach 3 Stunden oder so, habe ich es endlich geschafft, ein Speicherleck in einem View-Controller zu beheben. Das Leck wurde von einem UIPickerView verursacht, dessen Eigenschaft in der Headerdatei auf "beibehalten" gesetzt ist.

Der folgende Code verwaltet, es zu beheben:

- (void)viewDidLoad { 
    [super viewDidLoad];  
    myPicker = [[[UIPickerView alloc] initWithFrame:CGRectZero]autorelease]; 
} 

- (void)dealloc { 
    [super dealloc]; 
    [myPicker release]; 
    myPicker = nil; 
} 

Bitte sagen Sie mir nicht, wie schockierend dieser Code ist ... Ich weiß, dass es schlecht ist. Ich habe eine Freigabe, und eine Autorelease. Problem ist, wenn ich einen Teil des Obigen ändere oder entferne, kommt das Gedächtnisleck zurück.

ich aber wusste ich, wie Objective-C-Speicherverwaltung funktioniert, natürlich nicht ...

Warum der obige Code den Speicherverlust zu beheben, und was könnte eine korrekte Version des Codes aussehen?

-

EDIT:

Wenn jemand das gleiche Problem hat, oder interessiert - das Problem war, dass eines der anderen Objekte in meiner Klasse wurde auf ‚behalten‘ und nicht ‚zuweisen ". (Wenn Sie kein Objekt besitzen, sollte die Eigenschaft zugewiesen, nicht beibehalten werden).

Wie Cannondale sagte, behebt das Entfernen der zusätzlichen Behalten alles, und nur eine Version ist notwendig.

Answer 1

Sie müssen einen myPicker an anderer Stelle in Ihrem Code aufbewahren. Ihre myPicker-Zuweisungszeile wird diesen Speicher freigeben, sobald sich der Stapel für den viewDidLoad-Aufruf aufrollt (das ist es, was die Autorelease ihm sagt).

Sie müssen irgendwo nach diesem Punkt ein Retain machen, wenn nicht, dann wird Ihre [myPicker-Version] versuchen, nicht zugewiesenen Speicher mit unvorhersehbaren Ergebnissen freizugeben.

Was Sie tun sollten, ist die Zuweisung des Speichers in viewDidLoad (so entfernen Sie die Autorelease). Stellen Sie sicher, dass Sie das Objekt nirgendwo anders behalten und myPicker vom Dealloc freigeben.

Auch ... was bbum re der dealloc sagte;)

Answer 2

Was sagte Cannonade. Dies sollte funktionieren:

myPicker = [[UIPickerView alloc] initWithFrame:CGRectZero]; 

Ihre dealloc gesprengt wird, auch. Call to super muss immer der letzte sein (darüber nachdenken) und das kann zu undefiniertem Verhalten führen.

- (void)dealloc { 
    [myPicker release]; 
    myPicker = nil; 
    [super dealloc]; 
}