De retour des objets avec autorelease mais je fuir encore la mémoire

Question

Je suis une fuite de mémoire sur ce point:

ma classe personnalisée:

+ (id)vectorWithX:(float)dimx Y:(float)dimy{ 
return [[[Vector alloc] initVectorWithX:dimx Y:dimy] autorelease]; } 


- (Vector*)add:(Vector*)q { 
return [[[Vector vectorWithX:x+q.x Y:y+q.y] retain] autorelease]; } 

déléguer app je le lancer:

Vector *v1 = [[Vector alloc] initVector]; 
Vector *v2 = [[Vector alloc] initVector];  
Vector *vtotal = [[v1 add:v2] retain]; 

[v1 release]; 
[v2 release]; 
[vtotal release]; 

Comment cette des fuites? Je les libère ou les autoelease correctement. L'application se bloque immédiatement si je ne les conserve pas, à cause d'une libération anticipée je suppose. Il se bloque également si j'ajoute une autre version.

Answer 1

Pourquoi pensez-vous que vous fuyez la mémoire? Commence avec ça. Et quel objet plantez-vous en accédant? Cela vous indiquera très probablement quel est l'objet de votre sous-conservation. Si je devais faire une supposition, je soupçonnerais initVector, juste parce que c'est un nom très étrange pour une méthode. Qu'est ce que ça fait? Pourquoi n'est-il pas appelé simplement "init"?

Est-ce que ce code est enfilé? Vous faites beaucoup plus de conservation/autorelease que ce qui est généralement approprié. Vous n'avez pas besoin de conserver un objet pour le conserver dans la boucle d'événement en cours. D'une manière générale, vous ne retenez que des ivars, car ceux-ci sont ce dont vous avez besoin dans la prochaine boucle d'événements. Si vous avez beaucoup d'appels à conserver en dehors des accesseurs, alors vous êtes presque certainement en train de gérer la mémoire. Ci-dessus doivent être:

+ (id)vectorWithX:(float)dimx y:(float)dimy 
{ 
    return [[[Vector alloc] initVectorWithX:dimx y:dimy] autorelease]; 
} 

- (Vector*)add:(Vector*)q 
{ 
    return [Vector vectorWithX:(self.x + q.x) y:(self.y + q.y)]; 
} 

... 

Vector *v1 = [[Vector alloc] initVector]; 
Vector *v2 = [[Vector alloc] initVector]; 
Vector *vtotal = [v1 add:v2]; 
... 
[v1 release]; 
[v2 release]; 

Personnellement, je gérer v1/v2 avec autorelease parce que je pense que cela rend le code plus maintenable et compréhensible, mais il y a d'autres écoles de pensée:

Vector *v1 = [[[Vector alloc] initVector] autorelease]; 
Vector *v2 = [[[Vector alloc] initVector] autorelease]; 
Vector *vtotal = [v1 add:v2];