Instruments dit que j'ai une fuite de mémoire, mais je ne le vois pas

Question

J'ai une méthode pour générer un objet Deck (sous-classe NSObject avec une propriété NSMutableArray) et il remplit le tableau avec des objets Card (UIView sous-classe avec un certain nombre et une propriété NSString). Quand je demande un Deck, je vérifie s'il existe déjà (je pense) et si c'est le cas, relâchez-le avant d'en avoir un nouveau.

Le code de mon viewcontroller ...

#import "FlashTestViewController.h" 

@implementation FlashTestViewController 

- (IBAction)generateDeck { 

    if (aDeck != nil) { 
     [aDeck release]; 
    } 

    aDeck = [[Deck alloc] initDeckWithOperator:@"+"]; 
} 


- (IBAction)generateCard { 

    if (aCard != nil) { 
     [aCard fadeAway]; 
    } 

    aCard = [aDeck newCardFromDeck]; 
    [self.view addSubview:aCard]; 
} 

- (void)fadeAway { 
    [aCard removeFromSuperview]; 
    [aCard release]; 
    } 

    @end 

---

La classe de plate-forme est la suivante ...

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

@class Deck; 

@interface Deck : NSObject { 

    NSMutableArray* cards; 
} 

@property(nonatomic, retain) NSMutableArray* cards; 

- (id)initDeckWithOperator: (NSString*)mathOper; 
- (id)newCardFromDeck; 

@end 

---

- (id)initDeckWithOperator: (NSString*)mathOper { 

    if (cards != nil) { 
     [cards release]; 
    } 
    cards = [[NSMutableArray alloc] init]; 
    for (int i=0; i<11; i++) { 
     for (int j=0; j<11; j++) { 
      int xPos = (random() % 220) + 10; 
      int yPos = (random() % 360) + 10; 
      Card* aCard = [[Card alloc] initWithFrame:CGRectMake(xPos, yPos, 60, 80)]; 
      aCard.upperOperand = i; 
      aCard.lowerOperand = j; 
      aCard.theOperator = mathOper; 
      aCard.theResult = i + j; 

      UITextView* upperTextView = [[UITextView alloc] initWithFrame:CGRectMake(5, 5, 50, 20)]; 
     NSString* upperOper = [[NSString alloc] initWithFormat:@"  %d", i]; 
     upperTextView.text = upperOper; 
     [aCard addSubview:upperTextView]; 
     [upperTextView release]; 
     [upperOper release]; 

     UITextView* middleTextView = [[UITextView alloc] initWithFrame:CGRectMake(5, 30, 50, 20)]; 
     NSString* middleOper = [[NSString alloc] initWithFormat:@"%@ %d", mathOper, j]; 
     middleTextView.text = middleOper; 
     [aCard addSubview:middleTextView]; 
     [middleTextView release]; 
     [middleOper release]; 

     UITextView* lowerTextView = [[UITextView alloc] initWithFrame:CGRectMake(5, 55, 50, 20)]; 
     NSString* lowerOper = [[NSString alloc] initWithFormat:@"  %d", j+i]; 
      lowerTextView.text = lowerOper; 
      [aCard addSubview:lowerTextView]; 
      [lowerTextView release]; 
      [lowerOper release]; 

      [cards addObject: aCard]; 
      [aCard release]; 
     } 
    } 
    return self; 
} 

- (id)newCardFromDeck { 
    int index = random() % [cards count]; 
    Card* selectedCard = [[cards objectAtIndex:index] retain]; 
    [cards removeObjectAtIndex:index]; 
    return selectedCard; 
} 

@end 

---

Je fais la même chose quand je demande une nouvelle carte de la méthode newCardFromDeck et cela fonctionne. Aucune suggestion?

Merci!

Answer 1

Ajouter ce code à votre fichier Deck.m:

- (void)dealloc 
{ 
    [cards release]; 
    [super dealloc]; 
} 

Answer 2

En regardant cette ligne de code:

cards = [[NSMutableArray alloc] init]; 

Avez-vous relâchiez cards dans votre méthode dealloc? Il semble que cela pourrait être une fuite de mémoire.

Answer 3

aDeck dans generateDeck est aussi être une fuite si vous ne relâchez pas dans la dealloc de la vue.

Answer 4

En newCardFromDeck:

Card* selectedCard = [[cards objectAtIndex:index] retain]; 

ressemble à vous conservez la carte et le retourner quelque part. Où cette valeur de retour finit-elle? S'il finit dans une autre variable avec une propriété 'retain', il peut être conservé une seconde fois (lors de l'affectation à la variable).