iPhone - Wird dieses Leck?

Question

Ich habe einen Setter wie folgt aus:

- (UIImagePickerController *) foto { 

    if (_foto == nil) { 
     _foto = [[UIImagePickerController alloc] init]; 
     _foto.delegate = self; 
    } 
    return _foto; 
} 

es wie

@property (nonatomic, retain) UIImagePickerController *foto; 

mit

@synthesize foto = _foto; 

auf meinem dealloc

Ich habe erklärt

[_foto release]; 

An einem gewissen Punkt in meinem Code Ich mag diesen

self.foto = nil; 

aber etwas in meiner Seele tun, sagt das zu self.foto zugewiesen Objekt vorher undicht wird, weil es alloc auf dem Setter war ... wie mache ich es richtig?

danke.

Answer 1

Edit: Nein, das sollte in Ordnung sein. Solange Sie _foto vor der Veröffentlichung nichts anderes zuweisen, sollte es funktionieren.

Yup. Sie erstellen ein Objekt und verlieren dann den Zeiger darauf. Wenn Sie eine Autorelease in die Init-Zeile werfen, wird das Problem behoben. Sie könnten auch ARC verwenden.

Die Init-Zeile tut eigentlich nichts ... Sie weisen den Zeiger einem von Ihnen erstellten Objekt zu und weisen es dann einem anderen Objekt zu.

Answer 2

Ich glaube nicht, dass dort ein Leck ist. Wenn Sie self.foto wie folgt zuweisen: self.foto = nil;, wird die vorherige automatisch freigegeben. Wenn Sie es folgendermaßen zuweisen: _foto = nil;, müssen Sie es vor der Zuweisung manuell freigeben.

Answer 3

Ja, das funktioniert, und wird nicht lecken. Wenn Sie den Wert _foto festlegen, ist die Anzahl seiner Retouren 1 (weil Sie alloc aufgerufen haben). Solange Sie es freigeben (was Sie gesagt haben) in dealloc, sollten Sie in Ordnung sein, wie die Retain-Anzahl wird auf 0 zurückgehen. NUR, wenn Ihr Setter ist auch von Ihnen geschrieben und falsch geschrieben. Es muss den alten Wert explizit freigeben, wenn es nicht nil ist. Etwas wie dieses:

- (void)setFoto:(UIImagePickerController*)foto { 
    if (_foto) { 
      [_foto release]; 
      _foto = nil; 
    } 
    if (foto) 
      _foto = [foto retain]; 
}