Speicherverlust beim Hinzufügen von Objekten zu Nsarray

Question

In dem folgenden Code ist PersonListArray ein NSMutableArray und ich bekomme die Liste der Personen aus der SQLite-DB und fügen sie zu meinem Array.

Person* tmpPerson = [[Person alloc] init]; 
tmpPerson.personName = @"Mike"; 
tmpPerson.personEmail = @"mike@mike.com"; 

[PersonListArray addObject:tmpPerson]; 
[tmpPerson release]; 

Auch wenn ich hier das Objekt Person bin Loslassen, es ist ein Speicherleck zu geben, die ich vermute, ist aufgrund der Anordnung einen Referenzzähler, um es zu halten. Ich verwende das Array an anderer Stelle im Programm und lasse es dann sicher los.

Wie empfiehlt es sich, neue Objekte für ein Array zu erstellen und dieses Problem nicht zu lösen?

Im dealloc Methode, wo ich das Array freigeben

-(void) dealloc{ 
    [PersonListArray release]; // this contains the numerous Person objects 
    [super dealloc]; 
} 

sollte ich sie manuell wie dieser Stelle veröffentlichen?

-(void) dealloc{ 

    for (int i = 0; i<PersonListArray.count;i++) 
    { 
    Person * tmpPerson = [PersonListArray objectAtIndex:i]; 
    [tmpPerson release]; 
    } 

    [PersonListArray release]; 
    [super dealloc]; 
} 

Answer 1

Ihr Code, wie ursprünglich implementiert, ist korrekt. Ein Array behält Onjects bei, die hinzugefügt werden, und gibt sie entweder wieder frei, wenn sie aus dem Array entfernt werden oder wenn das Array freigegeben wird. Sie müssen nicht selbst durch das Array gehen.

Welche Mittel verwenden Sie, um das Leck zu erkennen? Wenn es Instrumente sind, dann magst du es falsch verstehen, was es dir sagt. Wenn es ein Leck erkennt, kann es Ihnen anzeigen, wo der Speicher zuerst zugewiesen wurde. Es kann Ihnen nicht zeigen, welches Objekt für das Leck verantwortlich ist. Ich würde daher raten, dass die angegebene dealloc-Methode nie aufgerufen wird (weil das Objekt geleakt wird) oder dass jemand anderes das Array behält und es nicht freigibt. Versuchen Sie, ein NSLog in dealloc zu setzen, um sicherzustellen, dass es auftritt; Als einmaliger Test könnten Sie versuchen, PersonListArray nach der Freigabe zu protokollieren - wenn das nicht zu einer Speicherausnahme führt, dann hat es höchstwahrscheinlich jemand anderes gespeichert.

[ENTFERNT: mein ursprünglicher Text "Versuchen Sie, einen NSLog von [PersonListArray retenrCount] zu Ihrem Dealloc hinzuzufügen, um herauszufinden, was der Fall ist."; Siehe Kommentar von bbum unten]

Die häufigste Ursache für versehentliche zusätzliche behält ist @ Eigenschaft/@ sythesize Eigenschaften, die festgelegt sind, beibehalten, aber für eine Zuordnung eine Freigabe nicht zu Dealloc hinzugefügt.

Answer 2

Der Code, den Sie uns zeigen, ist korrekt und enthält keine Lecks. Der letzte Abschnitt ist jedoch falsch und würde Ihr Programm zum Absturz bringen, da Sie Person Objekte freigeben, die Sie nicht mehr besitzen.

Answer 3

An anderer Stelle in Ihrer App rufen Sie wahrscheinlich [PersonListArray objectAtIndex:n] an und leiten sie an verschiedene andere Teile Ihrer App weiter. Einer der anderen Teile Ihrer App wird wahrscheinlich verloren gehen.

Wenn Sie Lecks verwenden, klicken Sie auf den jeweiligen "Art von Leck", dann klicken Sie auf die Speicheradresse, und es zeigt Ihnen die Allok/Frei/behalten/Freigabe/Autorelease Verlauf dieser Speicheradresse. Wenn Sie die Detailansicht aktivieren (Cmd-E denke ich), werden Sie auch Stack-Traces für all diese sehen. Suchen Sie nach etwas, das eine Zurückbehaltung, aber keine entsprechende Freigabe vornimmt. (Es ist ein bisschen schwierig, wenn die Dinge von mehreren Autoreleased-Arrays beibehalten werden ...)