Possible Duplicate:
Most memory efficient way to split an NSString in to substrings
I’m trying to split a 20Mb string. I’ve tried using componentsSeparatedByString but it consumes too much RAM. I think that this is down to the fact that it splits the string but also leaves the original string intact. This means that the string is effectivly stored in memory twice (even if I release the original string right after the split it is still an issue.)
I’m very new to Objective C. I’ve tried to write some code that removes the substring from the original string as it adds it to the array of found strings. The idea is that as the mutable array of found strings gets larger the original string gets smaller. The only problem is that it leaks memory and crashes. If someone could tell me what I’m doing wrong then that yould be great!
NSRange range = [mainHtml rangeOfString:@"<p class=NumberedParagraph>"];
int counter = 1;
// locations will == int max if it can't find any more occurances
while (range.location < [mainHtml length]) {
NSString *curStr;
NSRange curStrRange;
NSRange rangeToSearchIn = NSMakeRange(range.location+1, [mainHtml length] - range.location - 1);
NSRange nextRange = [mainHtml rangeOfString:@"<p class=NumberedParagraph>" options:NSCaseInsensitiveSearch range:rangeToSearchIn];
if (nextRange.location > [mainHtml length])
{
// This is the last string - get everything up to the end of the file
curStrRange = NSMakeRange(0, [mainHtml length]);
curStr = [mainHtml substringFromIndex:range.location];
} else {
curStrRange = NSMakeRange(range.location, nextRange.location - range.location);
curStr = [mainHtml substringWithRange:curStrRange];
}
// Remove the substring just processed from the orignal string
// * it crashes here, normally on the 3rd itteration
mainHtml = [mainHtml substringFromIndex:curStrRange.location + curStrRange.length];
range = [mainHtml rangeOfString:@"<p class=NumberedParagraph>"];
[self.parts addObject:curStr];
}
I think that @babbidi had the correct idea. mainHtml is large and you have many autoreleased copies of it around (one copy for each iteration) that are not being released. Try adding the following @autorelease in your code to release all the autoreleased objects at the end of each loop. If you are not using Mac OS X 10.7 then you need only create the autorelease pool manually outside the main loop and drain it once per iteration.