Fuite sur iPad que je ne comprends pas

Question

J'ai une fuite dans mon application et je ne sais pas pourquoi. Peut-être que j'ai toute mauvaise gestion de la mémoire. Dans mon code, j'ai UIViewController objet qui ont Ivar TelephoneValidator * validateur

TelephoneValidator est TelephoneValidator: NSObject

Donc, dans ma fonction d'initialisation de l'objet UIViewController (initWithFieldData) J'ai:

-(id) initWithFieldData: (NSMutableDictionary*) fieldData 
{ 
... 
    validatorOptions  = [fieldData objectForKey:@"fieldValidator"]; 
... 
} 

maintenant, dans mon viewDidLoad J'ai:

- (void)viewDidLoad { 
... 
if (![validatorOptions respondsToSelector:@selector(isEqualToString:)]) { 

      validator = [[TelephoneValidator alloc] initWithOptions: validatorOptions]; 
    } 
    else { 
      validator = nil; 
    } 
... 
} 

Basicly si mes validatorOptions ne NSString vali dator ivar devient l'instance TelephoneValidator.

Dans mon dealloc:

- (void)dealloc { 

    if(validator != nil) 
    { 
      [validator release]; 
      validator = nil; 
    } 
... 
[super dealloc]; 
    } 

J'ai vérifié deux fois si fonctionne dealloc, et il est. Après l'appel de dealloc, le validateur est libéré (l'appel de n'importe quelle méthode sur validator après [release validator] me donne une exception).

Et pourtant, dans Instruments, il me dit que TelephoneValidator est une fuite. Et après avoir double-cliqué dans Instruments la ligne de code qui est highlited est:

validator = [[TelephoneValidator alloc] initWithOptions: validatorOptions]; 

Qu'est-ce que je fais mal?

MISE À JOUR:

Voici mes informations d'en-tête de UIViewController:

@interface GenericViewController : UIViewController <UITextFieldDelegate>{ 

UIImage *backgroundImage; 
NSString *step; // na ktorym kroku jestesmy 
id <GenericControllerDelegate> delegate; //delegata z ktorej bedziemy pobierali dane 
UITextField *textField; 
NSString *fieldName; //nazwa pola (potrzebujemy zeby zapisac do modelu odpowiedni tekst 
UILabel *textLabel; 
UILabel *stepsLabel; 
UILabel *prefixTextLabel; 

NSString *fieldPlaceholder; 
NSString *textLabelText; 
NSString *textLabelTextPl; //w jezyku polskim 
NSString *prefixTextLabelText; //w jezyku eng 
NSString *prefixTextLabelTextPl; //w jezyku polskim prefix 

NSString *fieldRequired; 
NSString *keyboardType; 
NSString *capitalizeType; 

UIButton *button; //forward button 
UIButton *button2; //backward button 

//to bedzie do przerobienia bo bedziemy mieli tablicje walidatorow a nie jeden walidator 
NSString *validatorType; 

//maksymalna dlugosc pola 
int maxLengthOfTextField; 

NSArray* validatorOptions; 

TelephoneValidator *validator; 

//patientModel 
PatientData *patientModel; 

}

tête de TelephoneValidator:

#import <Foundation/Foundation.h> 
#import "MAOTranslate.h" 

@interface TelephoneValidator : NSObject { 


    //opcje walidacyjne 
    NSString *phonePrefix; 
    NSString *phonePostfix; 
    int  phoneLength; 
    NSString *message; 
    NSString *messagePl; 

    UIAlertView *alertView; 
} 

-(id) initWithOptions:(NSArray *) optionsArray; 

-(void) displayMessage; 
-(BOOL) validate: (NSString *) phoneNumber; 


@end 

classe TelephoneValidator:

#import "TelephoneValidator.h" 


@implementation TelephoneValidator 
//@synthesize phoneNumber; 

-(id) initWithOptions:(NSArray *) optionsArray; 
{ 
    if(self = [[TelephoneValidator alloc] init]) 
    { 
     phonePrefix = [optionsArray objectAtIndex:0]; 
     phonePostfix = [optionsArray objectAtIndex:1]; 
     phoneLength = [[optionsArray objectAtIndex:2] intValue]; 
     message = [optionsArray objectAtIndex:3]; 
     messagePl = [optionsArray objectAtIndex:4]; 
    } 
    else { 
     self = nil; 
    } 

    return self; 


} 

//wyswietlamy wiadomosc 
-(void) displayMessage 
{ 
    NSString *displayMsg; 
    if ([[MAOTranslate getLanguage] isEqualToString:@"pl"]) { 
     displayMsg = messagePl; 
    } 
    else { 
     displayMsg = message; 
    } 

    alertView = [[UIAlertView alloc] initWithTitle:@"Alert" message:displayMsg delegate:self cancelButtonTitle:@"ok" otherButtonTitles:nil]; 
    [alertView show]; 

} 

-(BOOL) validate: (NSString *) phoneNumber 
{ 


    //dlugosc 
    if ([phoneNumber length] != phoneLength) { 
     NSLog(@"zla dlugosc"); 
     return NO; 
    } 


    NSLog(@"tutaj"); 
    //sprawdzamy prefix  
    if ([phonePrefix length]!= 0) {  
     NSLog(@"w srodku ifa"); 
     if ([phoneNumber compare:phonePrefix options:NSLiteralSearch range:NSMakeRange(0, [phonePrefix length])] != 0) { 
      NSLog(@"zly prefix"); 
      [self displayMessage]; 
      return NO; 
     } 
    } 

    //sprawdzamy postfix 
    if([phonePostfix length] != 0) 
    { 
     if ([phoneNumber compare:phonePostfix options:NSLiteralSearch range:NSMakeRange([phoneNumber length]-[phonePostfix length], [phonePostfix length])] != 0) { 
      NSLog(@"zly postfix"); 
      [self displayMessage]; 
      return NO; 
     } 
    } 


    //sprawdzamy czy string jest numeryczny 

    NSCharacterSet *alphaNums = [NSCharacterSet decimalDigitCharacterSet]; 
    NSCharacterSet *inStringSet = [NSCharacterSet characterSetWithCharactersInString:phoneNumber]; 

    if (![alphaNums isSupersetOfSet:inStringSet]) 
    { 
     NSLog(@"zly format "); 
     [self displayMessage]; 
     return NO; 
    } 

    return YES; //zwalidowany poprawnie 
} 

-(void) dealloc 
{ 

    [alertView release]; 
    alertView = nil; 
    [super dealloc]; 
} 

Answer 1

Voir Ces deux lignes

validator = [[TelephoneValidator alloc] initWithOptions: validatorOptions]; 

et initWithOptions à l'intérieur

if(self = [[TelephoneValidator alloc] init]) 

Vous allocing deux fois le validateur, donc il y a une fuite.

Answer 2

Vous devez appeler [super dealloc] à la fin de la méthode dealloc.

Answer 3

Est-il possible que les instruments pointent vers validatorOptions comme source de la fuite? S'agit-il d'une propriété conservée libérée à dealloc ou non? Je ne peux pas dire avec certitude, le code que vous avez posté n'est pas suffisant pour arriver à une conclusion.

De même, comme le dit willcodejavaforfood, vous devez toujours appeler [super dealloc]; à la fin de votre méthode dealloc. Aucun code ne doit venir après.

Éditer:

Je suis de retour. Mais Bruno Domingues a déjà eu raison, vous allouez deux fois, auquel cas, le premier fuit.Vous devez changer votre code -initWithOptions: à:

-(id) initWithOptions:(NSArray *) optionsArray; 
{ 
    if((self = [super init])){ 
    // ... rest of code is fine 
    } 

    return self; 
}