2012-07-24 107 views
0

后分析我的cocos2d游戏我有一个警告“上线525分配的对象的潜在泄漏并存入‘的valueString’”在此代码对象的潜在泄漏分配

525 NSString * valueString=[[[NSString alloc] initWithString:[NSString stringWithFormat:@"%@: %@",kGameTimeeng,[allFunctions getTimeFormat:(int) _timeLimit]]] retain]; 

    if([_language isEqualToString:@"rus"]){ 
     [valueString release]; 
     valueString=[[[NSString alloc] initWithString:[NSString stringWithFormat:@"%@: %@",kGameTimerus,[allFunctions getTimeFormat:(int) _timeLimit]]] retain]; 
    }  

    id sequence=[CCSequence actions: 
       [CCCallFuncND actionWithTarget: allFunctions selector: @selector(setLabelColor:withIndex:) data:(void*)color], 
       [CCCallFuncND actionWithTarget: allFunctions selector: @selector(setLabelValue:withValue:) data:(NSString*)valueString], 
       // [CCCallFuncND actionWithTarget: self selector: @selector(setLabelStroke:withTag:) data:(void*)TagCurentPointsLabelStroke], 
       [CCBlink actionWithDuration:0.5f blinks:2], 
       [CCShow action], 
       [CCCallFuncND actionWithTarget: allFunctions selector: @selector(setLabelColor:withIndex:) data:(void*)colorAfter], 

       nil]; 

    [_timeLimitLabel runAction:sequence]; 
    [valueString release]; 

allFunctions .m

-(void) setLabelValue:(id) sender withValue:(NSString*) value 
{ 
    CCLabelTTF *label=(CCLabelTTF *)sender; 
    NSString * valueString=[[[NSString alloc] initWithString:[NSString stringWithFormat:@"%@",value]] autorelease]; 
    [label setString:[NSString stringWithFormat:@"%@",valueString]]; 
    //[valueString release]; 
} 

你能解释一下为什么吗?

+0

在allFunctions.m中,您应该重新排列字符串。但是,在这些情况下,我会放弃字符串。使用autorelease而不是release,不要释放它。此外(但shoudl与警告不相关)我会将535行移入if ... rus语句的else分支。 – 2012-07-24 09:31:23

+0

对不起,在allFunctions.m你已经自动发布它。我忽略了这一点。这没关系。 – 2012-07-24 09:37:39

回答

1
valueString = [[[NSString alloc] initWithString:[NSString stringWithFormat:@"%@: %@",kGameTimerus,[allFunctions getTimeFormat:(int) _timeLimit]]] retain]; 

您在这里保留了两次:alloc &保留。然后你只发布一次:

[valueString release]; 

这就是为什么有潜在的泄漏(实际上,这是一个泄漏)。

而对于

NSString * valueString = [[[NSString alloc] initWithString:[NSString stringWithFormat:@"%@",value]] autorelease]; 

你留住一次(ALLOC),并释放(autorelease)时,不再需要valueString。这没关系。

+0

非常感谢!有帮助 – 2012-07-24 09:51:22

+0

@AlexanderSharunov欢迎你!顺便说一句,如果你不熟悉内存管理,你可以尝试使用[ARC](http://developer.apple.com/library/mac/#releasenotes/ObjectiveC/RN-TransitioningToARC/Introduction/Introduction.html)。 :) – Kjuly 2012-07-24 09:55:21

+0

好的,我会试试,对不起我的英文 – 2012-07-24 16:24:18

1

当您将init分配给一个对象时,该对象已将保留计数设置为1,因此通常不需要保留它。当您在第一个代码示例([valueString release];)结束时释放它时,它将保留计数为1,因为您在init alloc之后保留了它。

我不确定CCSequenceCCCallFuncND如何处理有关内存管理的参数,但是如果从指定行中删除了保留,则应该是安全的。

希望这会有所帮助。

+0

非常感谢!有帮助 – 2012-07-24 09:51:55

2
525 if([_language isEqualToString:@"rus"]){ 
     [valueString release]; 
     valueString=[[[NSString alloc] initWithString:[NSString stringWithFormat:@"%@: %@",kGameTimerus,[allFunctions getTimeFormat:(int) _timeLimit]]] autorelease]; 
    } else {  
     NSString * valueString=[[[NSString alloc] initWithString:[NSString stringWithFormat:@"%@: %@",kGameTimeeng,[allFunctions getTimeFormat:(int) _timeLimit]]] autorelease]; 
    } 


    id sequence=[CCSequence actions: 
       [CCCallFuncND actionWithTarget: allFunctions selector: @selector(setLabelColor:withIndex:) data:(void*)color], 
       [CCCallFuncND actionWithTarget: allFunctions selector: @selector(setLabelValue:withValue:) data:(NSString*)valueString], 
       // [CCCallFuncND actionWithTarget: self selector: @selector(setLabelStroke:withTag:) data:(void*)TagCurentPointsLabelStroke], 
       [CCBlink actionWithDuration:0.5f blinks:2], 
       [CCShow action], 
       [CCCallFuncND actionWithTarget: allFunctions selector: @selector(setLabelColor:withIndex:) data:(void*)colorAfter], 

       nil]; 

    [_timeLimitLabel runAction:sequence];