SubstringToIndex内存泄露
问题描述:
我使用这个功能来抽取子,它的工作原理,但有两个泄漏:SubstringToIndex内存泄露
-(NSString*)EstraiP:(NSString*)str ini:(NSString*)ini fin:(NSString*)fin occ:(int)occ{
NSRange rstr1;
for(int i=0; i < occ; i++){
rstr1=[str rangeOfString:fin];
str=[str substringFromIndex:rstr1.location+rstr1.length];
}
NSString* FinalStr;
rstr1=[str rangeOfString:ini];
if(occ==0){
if(rstr1.length==0)
return @"Non Trovato inizio";
FinalStr=[str substringFromIndex:(rstr1.location + rstr1.length)] ;
}else{
if(rstr1.length==0)
return @"Non Trovato inizio";
FinalStr=[str substringFromIndex:rstr1.location+rstr1.length] ;
}
NSRange rstr2=[FinalStr rangeOfString:fin];
if(rstr2.length==0)
return @"Non Trovata fine";
FinalStr=[FinalStr substringToIndex:rstr2.location];
return FinalStr;
}
这对泄密行一些内存:
str=[str substringFromIndex:rstr1.location+rstr1.length];
FinalStr=[FinalStr substringToIndex:rstr2.location];
我环顾四周,但没有发现任何东西.... 没有分配或保留,所以我不应该释放他们......可能是什么问题? 我希望我自己解释一下
谢谢!
答
您的初始for
循环很可能会引发异常。如果找不到匹配项,则-[NSString rangeOfString:]
返回位置字段设置为NSNotFound的NSRange对象。在这种情况下,当您将该值插入-substringFromIndex:
时,最终会触发异常。
您不应该用大写的第一个字母命名变量。这样的命名通常是为类名保留的,并且会让你的代码非常混乱。您可能需要使用NSMaxRange(aRange)
而不是使用aRange.location + aRange.length
。它编译成完全相同的东西,但它更具可读性。
最后,你没有内存泄漏。你为什么认为有一个?
什么告诉你这里有泄漏? – 2010-07-02 17:08:07
如果您想确保子字符串不存在,请使用rstr1.location == NSNotFound'。 – kennytm 2010-07-02 18:59:45