Messages "Décrément incorrect" et "Fuite potentielle" de l'analyseur

Question

Lorsque je compile avec l'analyseur, j'obtiens quelques messages. J'ai ces propriétés ont déclaré:

@property (nonatomic, retain) SyncServicePrimary *syncAndCartOne; 
@property (nonatomic, retain) SyncServiceSecondary *syncAndCartTwo; 

Cette méthode est appelée à partir applicationDidBecomeActive et je reçois « fuite potentielle d'un objet alloué ».

-(void)makeTheCartObjectsForCountry:(NSString*)country_key{ 
    self.syncAndCartOne = [[SyncServicePrimary alloc] init]; 
    self.syncAndCartTwo = [[SyncServiceSecondary alloc] init]; 
} 

Ceci est appelé dans applicationWillResignActive; ici je reçois "Décrément incorrect du compte de référence d'un objet".

-(void) removeTheCartObjects{ 
    [self.syncAndCartOne release]; 
    self.syncAndCartOne = Nil;  
    [self.syncAndCartTwo release]; 
    self.syncAndCartTwo = Nil; 
} 

Si je mets les objets à autorelease, l'erreur disparaît, mais je veux que les objets soient libérés lorsque l'application se cache.

Est-ce que c'est quelque chose que je fais bien mais qui est divisé trop loin pour que l'analyseur voit le début et la fin, ou est-ce quelque chose que je peux faire mieux pour ne pas me plaindre?

Il est plus que probable que je manque un concept simple en ce qui concerne release et alloc cycles (je viens de PHP et C#).

Answer 1

Votre problème est ici:

-(void)makeTheCartObjectsForCountry:(NSString*)country_key{ 
    self.syncAndCartOne = [[SyncServicePrimary alloc] init]; 
    self.syncAndCartTwo = [[SyncServiceSecondary alloc] init]; 
} 

Vous créez les objets, puis de les retenir (en raison de la déclaration de la propriété), donc ils ont un compte de référence de 2, quand un seul objet est de leur référencement .

Vous devriez le faire comme ceci:

-(void)makeTheCartObjectsForCountry:(NSString*)country_key{ 
    SyncServicePrimary *primary = [[SyncServicePrimary alloc] init]; 
    self.syncAndCartOne = primary; 
    [primary release]; 

    SyncServiceSecondary *secondary = [[SyncServiceSecondary alloc] init]; 
    self.syncAndCartTwo = secondary; 
    [secondary release]; 
} 

Answer 2

Vous avez défini les propriétés avec attribut retain, de sorte que l'analyseur suppose que la méthode setter pour la propriété ressemble à ceci:

- (void)setSyncAndCartOne:(SyncServicePrimary *)newValue 
{ 
    [newValue retain]; 
    [self->_syncAndCartOne release]; // access the instance variable holding the property value 
    self->_syncAndCartOne = newValue; 
} 

Si vous utilisez @synthesize, la méthode setter va ressembler à ça.

Ainsi, lorsque makeTheCartObjectsForCountry: retours, l'objet syncAndCartOne a un retain nombre de 2, mais doit seulement avoir un nombre de retenir 1. C'est pourquoi l'utilisation autorelease corrige ce problème. Vous ne devriez pas faire [self.syncAndCartOne release] pour la même raison. La méthode setter enverra l'ancien objet release lorsque vous affectez nil à la propriété.