Leak Problem with [NSMutableArray removeAllObjects];

[surgen12]

Hey Guys

I have leak problem here.
The Instrument that checks leak
says there is leak with code below.


[self.artRecords_list removeAllObjects];

[self.artRecords_list release];


self.artRecords_list = [[NSMutableArray alloc] init];

while(sqlite3_step(statement) == SQLITE_ROW)
{

self.ArtInfo = [[ArtRecord alloc] init]; // <- This is leaking point.

char * str_empty = (char * )sqlite3_column_text(statement,0);
if(str_empty != NULL)
{
self.ArtInfo.art_title = [NSString stringWithUTF8Stringchar * )sqlite3_column_text(statement,0)];
self.ArtInfo.part_title = [NSString stringWithUTF8Stringchar * )sqlite3_column_text(statement,1)];

[self.artRecords_list addObject:self.ArtInfo];
//[self.ArtInfo autorelease];
//[self.ArtInfo release];


}


Yes I need to make a list of Object that contains some information.
So I decided make NSMutableArray and I fill it with this Object.

and it says there is Leak.
I added some code like:
//[self.ArtInfo autorelease];
//[self.ArtInfo release];
these.

and It crashes.
Wherever point I put "release" code It crashes.

But I can't find any other way to make list of objects dynamically.

please help.

Thanks in advance.

[JDave]

Without looking at all the code (AND/OR knowing the specific error you are getting), I would guess that ArtInfo is a global variable (ie, you declared it in your .h file, and synthesize it in your .m, right?). If so, then you set it to nil and release it in viewDidUnload: (iOS4 methods aside...) and dealloc:, respectively... Releasing it inside the run loop will cause uncertain results, depending on the retain count... which is likely 0 when you release it. To check if it is a retain count issue, do this:

[ArtInfo retain]; // retain count now +1
[ArtInfo release]; // without the above code, this line will cause a 'crash'

Try 'build and analyze' (in BUILD menu of XCODE).... it sometimes gives more info than leaks, IMHO.

ALSO, look at how you create your ArtRecord class (the .h/.m), it may be leaky, too.

Good Luck.

[flamingliquid]

Is you artInfo property set to retain? If so, then that is why it is leaking. Check out this link:

http://www.iphonedevsdk.com/forum/ip...roperties.html

[surgen12]

I will try .

[surgen12]

I tried below.

[ArtInfo retain]; // retain count now +1
[ArtInfo release]; // without the above code, this line will cause a 'crash'

I put this code upper part of the while loop like below.

while(sqlite3_step(statement) == SQLITE_ROW)
{
if (ArtInfo != nil) {
//ArtInfo = nil;
[ArtInfo retain];
[ArtInfo release];

}


ArtInfo = [[ArtRecord alloc] init];

char * str_empty = (char * )sqlite3_column_text(statement,0);
if(str_empty != NULL)
{

ArtInfo.art_title = [NSString stringWithUTF8Stringchar * )sqlite3_column_text(statement,0)];
ArtInfo.part_title = [NSString stringWithUTF8Stringchar * )sqlite3_column_text(statement,1)];

[self.artRecords_list addObject:ArtInfo];
}
else
{
self.str_cnt = @"0";
}
}


There was no crash and it appeared working well.

but the instrument pointed the same leak again.
so according to this code it release every ArtInfo that created
and It is not retain variable it is global variable declared in .h file.

I think this 'Leak instrument' isn't so perfect.

[surgen12]

Recently I purchased a book, named "Advanced iPhone Develop..." something like that.

and I found that they use NSMutableDictionary for this kind job.

So I stopped using my own class "ArtRecord" for this job and I used NSMutableDictionary.

And It worked so well.

No leak for this job.

My Custom class "ArtRecord" was the problem.
I still have no idea why it didn't work for this job it did work with NSMutableDictionary.

May be some special skill for this sdk utility class.

Anyway Nice experience.

Thank you all of you.