2010-07-02 98 views
1

我使用这个功能来抽取子,它的工作原理,但有两个泄漏: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]; 

我环顾四周,但没有发现任何东西.... 没有分配或保留,所以我不应该释放他们......可能是什么问题? 我希望我自己解释一下

谢谢!

+2

什么告诉你这里有泄漏? – 2010-07-02 17:08:07

+0

如果您想确保子字符串不存在,请使用rstr1.location == NSNotFound'。 – kennytm 2010-07-02 18:59:45

回答

0

您的初始for循环很可能会引发异常。如果找不到匹配项,则-[NSString rangeOfString:]返回位置字段设置为NSNotFound的NSRange对象。在这种情况下,当您将该值插入-substringFromIndex:时,最终会触发异常。

您不应该用大写的第一个字母命名变量。这样的命名通常是为类名保留的,并且会让你的代码非常混乱。您可能需要使用NSMaxRange(aRange)而不是使用aRange.location + aRange.length。它编译成完全相同的东西,但它更具可读性。

最后,你没有内存泄漏。你为什么认为有一个?