Возвращение объектов с autorelease, но я до сих пор утечка памяти

Question

Я протечки память об этом:

мой пользовательский класс:

+ (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]; } 

в приложение делегата I инициируют его:

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

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

Как это утечки? Я выпускаю или автоматически проверяю их. Приложение немедленно сработает, если я не сохраню их, из-за раннего выпуска, я думаю. Он также падает, если я добавлю еще один релиз.

Answer 1

Почему вы думаете, что у вас утечка памяти? Начните с этого. И какой объект вы сталкиваетесь при доступе? Это, скорее всего, скажет вам, к какому объекту относится ваше недодержанное. Если бы я должен был предположить, я бы заподозрил initVector, просто потому, что это очень странное имя для метода. Что оно делает? Почему он не называется просто «init»?

Это этот код с резьбой? Вы делаете намного больше удержания/авторекламы, чем это обычно уместно. Вам не нужно сохранять объект, чтобы удерживать его через текущий цикл событий. Вообще говоря, вы сохраняете только ivars, потому что это то, что вам нужно, через следующий цикл событий. Если у вас много вызовов, чтобы сохранить за пределами доступа, то вы почти наверняка неправильно управляете памятью. Выше должно быть:

+ (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]; 

Лично я бы обрабатывать v1/v2 с autorelease, потому что я думаю, что это делает код более ремонтопригодны и понятно, но есть и другие школы мысли:

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