Why do I have a memory leak here?

tags:

views:

81

answers:

1

+2 Q:

Why do I have a memory leak here?

Hi, I do not understand why Instruments/Leaks tells me that there is a memory leak in my viewDidLoad. I do release both objects in dealoc and set them to nil in viewDidUnload. Can someone give me a clue here?

Link to screen-dump from Instruments: http://i26.tinypic.com/28227iw.png

BR //Christoffer

+1 A:

I do release both objects in dealoc and set them to nil in viewDidUnload

I might be misunderstanding the grammer of the sentence, but you should release and set to nil in both dealloc and viewDidUnload.

As a general rule, you should always release and set to nil as a pair unless you have a specific reason not to.

Akusete 2010-07-20 10:08:24

Not to mention that just releasing an object without setting references to it to nil is plain dangerous.

Costique 2010-07-20 10:17:30

Thanks for your replies. I see the error/flaws in the code that you point out and corrected them. But the result is the same. So how should this be implemented if you want to initialize class objects each time the view loads?

Christofffer 2010-07-20 11:40:44

You should release them in viewUnload and create them in viewLoad

Akusete 2010-07-20 11:53:41

Ok I got it to work by initializing the NSMutableArray in another way:This Work: self.listOfItems = [[NSMutableArray alloc]init];self.paymentmethods = [[NSMutableArray alloc]init];This does not work:self.listOfItems = [[NSMutableArray array]retain];self.paymentmethods = [[NSMutableArray array]retain];Do you guys know why the other causes a leak?

Christofffer 2010-07-20 12:02:56

Both of those leak, assuming the properties are retained.

tc. 2010-07-20 12:32:20

ansaurus

tags:

views:

answers:

Why do I have a memory leak here?

related questions