NSString fuite même avec la libération au bon endroit (je suppose)?

Question

Lorsque j'analyse le code suivant avec des instruments, il fait état d'une fuite sur la variable imageName:

//loadImagesFromPotatoesIndexesArray 
-(void) loadImagesFromPotatoesIndexesArray{ 

    //Load Textures from Disk 
    textures = [[NSMutableArray alloc] init]; 
    //NSArray *masks = [[NSArray alloc] initWithArray:mainDelegate.masksArray]; 

    for (int i = 0;i<[potatoesIndexesArray count];i++){ 

     int imageNumber = [[potatoesIndexesArray objectAtIndex:i]intValue]; 

     NSString *imageName = [[NSString alloc] initWithFormat:@"texture%d",imageNumber]; 

     UIImage *image = [[UIImage alloc] initWithContentsOfFile:[[NSBundle mainBundle] pathForResource:imageName ofType:@"png"]]; 

     NSArray *pics = [[NSArray alloc] initWithObjects: 
         [self maskImage:image withMask:[mainDelegate.masksArray objectAtIndex:i]], 
         [self maskImage:image withMask:[mainDelegate.masksArray objectAtIndex:i]], 
         imageName, 
         nil]; 

     [textures addObject:pics]; 

     [image release]; 
     [imageName release]; 
     [pics release]; 
    } 
} 

[potatoesIndexesArray count] = 16, donc j'ai 16 fois que NSCFString ... Mais fuite moi le code respecte la gestion de la mémoire ... évidemment pas !!! Qu'est-ce que j'ai fait de mal?

Answer 1

Si pense que lorsque vous ajoutez imageName dans votre tableau les photos il le conserver ;-) (je pense que la réponse à votre question)

Mais, pourquoi fais-tu un alloc ici? Pourquoi ne pas faire quelque chose comme

[ NSString stringWithFormat:@"" ]

?

Bonne chance!

Answer 2

Ceci est un problème compliqué. Vous allouez l'imageName, de sorte que retainCount est 1, puis vous l'ajoutez dans un tableau, le nombre de retain est 2, lorsque vous relâchez imageName, le nombre de retenue est 1 à nouveau. Alors si vous libérez aussi le tableau de pics, tout ira bien. Mais votre tableau de photos est ajouté dans les textures, puis le pics est publié, donc vos photos retainCount est toujours 1. Et votre imageName est une fuite. Mais, si vous relâchez le tableau textures, tout ira bien

NSString *imageName = [[NSString alloc] initWithFormat:@"texture%d",imageNumber]; 

    NSArray *pics = [[NSArray alloc] initWithObjects: 
        [self maskImage:image withMask:[mainDelegate.masksArray objectAtIndex:i]], 
        [self maskImage:image withMask:[mainDelegate.masksArray objectAtIndex:i]], 
        imageName, 
        nil]; 

    [imageName release]; 

Answer 3

Vous ne diffusons jamais le tableau « textures ». Il tient toujours tout.

Answer 4

À quelle fréquence loadImagesFromPotatoesIndexesArray est-il appelé dans votre code? Si elle est appelée plusieurs fois, toutes les valeurs du tableau d'origine seront divulguées, car vous ne publiez pas correctement textures avant de le remplacer par un nouveau tableau.

Si elle est appelé plus d'une fois, cela devrait faire l'affaire:

// load textures from disk 
[textures removeAllObjects]; 
//NSArray *masks = [[NSArray ... 

for (int i=0; ...