NSArray dans NSArray ne pas retourner l'image que je veux

Question

J'ai un extrait de code ici que je ne peux pas faire fonctionner.

NSUInteger i; 
//NSMutableArray *textures = [[NSMutableArray alloc] initWithCapacity:kNumTextures]; 
//NSMutableArray *texturesHighlighted = [[NSMutableArray alloc] initWithCapacity:kNumTextures]; 

NSMutableArray *textures= [[NSMutableArray alloc] init]; 


for (i = 1; i <= kNumTextures; i++) 
{ 
    NSString *imageName = [NSString stringWithFormat:@"texture%d.png", i]; 
    NSString *imageNameHighlighted = [NSString stringWithFormat:@"texture%d_select.png", i]; 
    UIImage *image = [UIImage imageNamed:imageName]; 
    UIImage *imageHighlighted = [UIImage imageNamed:imageNameHighlighted]; 
    //NSArray *pics = [[NSArray alloc] initWithObjects:(UIImage)image,(UIImage)imageHighlighted,nil]; 
    NSArray *pics = [NSArray arrayWithObjects:image,imageHighlighted,nil]; 
    [textures addObject:pics]; 
    [pics release]; 
} 

//select randomly the position of the picture that will be represented twice on the board 
NSInteger randomTexture = arc4random()%([textures count]+1); 

//extract image corresponding to the randomly selected index 
//remove corresponding pictures from textures array 
NSArray *coupleTexture = [textures objectAtIndex:randomTexture]; 
[textures removeObjectAtIndex:randomTexture]; 

//create the image array containing 1 couple + all other pictures 
NSMutableArray *texturesBoard = [[NSMutableArray alloc] initWithCapacity:kNumPotatoes]; 

[texturesBoard addObject:coupleTexture]; 
[texturesBoard addObject:coupleTexture]; 
[coupleTexture release]; 

NSArray *pics = [[NSArray alloc] init]; 
for (pics in textures) { 
    [texturesBoard addObject:pics]; 
} 
[pics release]; 

//shuffle the textures 
//[texturesBoard shuffledMutableArray]; 

//Array with masks 
NSMutableArray *masks= [[NSMutableArray alloc] init]; 

for (i = 1; i <= kNumMasks; i++) 
{ 
    NSString *maskName = [NSString stringWithFormat:@"mask%d.png", i]; 
    UIImage *mask = [UIImage imageNamed:maskName]; 
    //NSArray *pics = [[NSArray alloc] initWithObjects:mask,nil]; 
    [masks addObject:mask]; 

    //[pics release]; 
    [maskName release]; 
    [mask release]; 
} 

//Now mask all images in texturesBoard 
NSMutableArray *list = [[NSMutableArray alloc] init]; 
for (i = 0; i <= kNumMasks-1; i++) 
{ 
    //take on image couple from textures 
    NSArray *imgArray = [texturesBoard objectAtIndex:i]; 
    UIImage *mask = [masks objectAtIndex:i]; 

    //mask it with the mask un the array at corresponding index 
    UIImage *img1 =(UIImage *) [imgArray objectAtIndex:0]; 
    UIImage *img2 =(UIImage *) [imgArray objectAtIndex:1]; 
    UIImage *picsMasked = [self maskImage:(UIImage *)img1 withMask:(UIImage *)mask]; 
    UIImage *picsHighlightedMasked = [self maskImage:(UIImage *)img2 withMask:(UIImage *)mask]; 
    //Init image with highlighted status 
    TapDetectingImageView *imageView = [[TapDetectingImageView alloc] initWithImage:picsMasked imageHighlighted:picsHighlightedMasked]; 
    [list addObject:imageView]; 
} 

Le problème ici est que: img1 et img2, ne sont pas des images mais plutôt NSArray avec plusieurs entrées. Je ne peux pas comprendre pourquoi ... n'importe quel esprit frais ici pourrait me donner un indice.

merci beaucoup.

Answer 1

Commençons par un problème de mémoire dans la première boucle. Il y a une libération excessive:

NSArray *pics = [NSArray arrayWithObjects:image,imageHighlighted,nil]; 
[textures addObject:pics]; 
[pics release]; // <-- pics is overreleased since it is created autoreleased 

Deuxième problème de mémoire. coupleTexture est surchargé.

NSArray *coupleTexture = [textures objectAtIndex:randomTexture]; 
[textures removeObjectAtIndex:randomTexture]; 

//create the image array containing 1 couple + all other pictures 
NSMutableArray *texturesBoard = [[NSMutableArray alloc] initWithCapacity:kNumPotatoes]; 

[texturesBoard addObject:coupleTexture]; 
[texturesBoard addObject:coupleTexture]; 
[coupleTexture release];  // <--- coupleTexture is autoreleased since it is returned by objectAtIndex: and not retained 

Vous devriez apprendre à faire des énumérations rapides. Il devrait ressembler à ceci:

for (NSArray *pics in textures) { 
    [texturesBoard addObject:pics]; 
} 

Plus question de mémoire dans les masques pour la boucle:

NSString *maskName = [NSString stringWithFormat:@"mask%d.png", i]; 
UIImage *mask = [UIImage imageNamed:maskName]; 
//NSArray *pics = [[NSArray alloc] initWithObjects:mask,nil]; 
[masks addObject:mask]; 

//[pics release]; 
[maskName release];  // <-- overreleased.. was created by stringWithFormat 
[mask release];   // <-- overreleased.. same same 

Par conséquent, le tableau des masques n'est pas sorti ... ainsi que le tableau de la liste créée juste après la boucle ...

Oh boy. Ici, je dois arrêter depuis que je commence aussi obtenir des maux de tête;) Ce que vous devez savoir:

• Seulement release ce que vous avez créé par init ou ce que vous retained explicitement
• Utilisez énumération rapide de la bonne manière
• Structure votre code ... Peut-être extraire certaines des boucles for pour séparer les méthodes et déboguer leurs résultats séparément.

Ne vous découragez pas. Continuez à apprendre :)

Answer 2

Ce code m'a donné mal à la tête. Vous devez vraiment nettoyer cela et encapsuler dans certaines méthodes ou fonctions, surtout si vous insistez pour réaffecter les mêmes noms de variables encore et encore.

Je pense que votre problème est ici:

NSArray *pics = [[NSArray alloc] init]; 
for (pics in textures) { 
    [texturesBoard addObject:pics]; 
} 
[pics release]; 

D'abord, vous assigner une initialisé par un tableau vide, alors vous lui assigner les tableaux dans des textures puis vous relâchez le dernier tableau que vous avez attribué. Il pense que cela brouille le tableau texturesBoard. Il tue très probablement l'un des tableaux rendant la taille du tableau erronée.

Il devrait juste être:

NSArray *pics; 
for (pics in textures) { 
    [texturesBoard addObject:pics]; 
}