Problème Ajout à NSMutableArray en boucle

Question

Je ne suis pas sûr du problème avec cette boucle, mais je continue à recevoir SIGABRT chaque fois que je l'exécute. Selon le journal, le problème est lorsque j'essaie d'ajouter le NSNumber au tableau NSMutable près de la fin de la boucle. Évidemment j'ai fait une erreur élémentaire mais je ne suis pas sûr du problème.

NSArray *toArray = [ourDictionary objectForKey:toString]; 
NSMutableArray *allValuesMArray = [[NSMutableArray alloc] init]; 
while (done == NO) 
{ 
    if (i == 10) 
     done = YES; 
      /* 
      *The job here is to multiply these three numbers together and store the 
      *product in the mutable array. It tells me NSObject did not recognize selector 
      *and then crashes. 
      *original and multiplyFrom are always the same value, and multiplyTo is updated 
      *from an array I made above from a p-list. 
      *I'm hoping I didn't make a ton of rookie mistakes here, but I'm new to dealing with 
      *NSMutableArray and such. 
      */ 

    NSNumber *original = [NSNumber numberWithDouble:convertThis]; 
    NSNumber *multiplyFrom = [NSNumber numberWithDouble:multiply]; 
    NSNumber *multiplyTo = [NSNumber numberWithDouble:[[toArray objectAtIndex:i] doubleValue]]; 
    NSNumber *product = [[NSNumber alloc] init]; 

    product = [NSNumber numberWithDouble:([original doubleValue] * 
              [multiplyFrom doubleValue] * 
              [multiplyTo doubleValue])]; 

    [allValuesMArray addObject:product]; 
      //This line ^^^ causes crash 
    i++; 
} 
NSArray *returnThisArray = allValuesMArray; 
[allValuesMArray autorelease]; 
return returnThisArray; 

Answer 1

Vous libérez le allValuesMArray: [allValuesMArray release];

Étant donné que le nombre de rétention est 0 à ce stade, le tableau sera immédiatement libéré.

Essayez d'utiliser [allValuesMArray autorelease]. Cela libérera le tableau dans le futur, donnant la possibilité à la méthode appelante d'utiliser le tableau non libéré ou de conserver le tableau pour une utilisation ultérieure.

Vous pouvez également vérifier si le produit n'est pas nul avant de l'ajouter à la matrice.

Et vous avez une fuite de mémoire.

product = [[NSNumber alloc] init];

Ensuite, vous lui assignerez un objet différent plus tard.

Answer 2

Je suis sûr que c'est une faute de frappe, mais vous n'allouez pas un pointeur et même pas un type de NSMutableArray, mais de type NSArray. Vérifiez la ligne 2:

NSMutableArray allValuesMArray = [[NSArray alloc] init]; 

Il devrait être

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

Answer 3

Créez-vous un NSArray et l'assigner à un NSMutableArray? Peut-être que vous vouliez dire ce qui suit:

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

Answer 4

Voici votre code dans un nettoyé-, sous forme non-fuite:

NSMutableArray *convertedValues = [NSMutableArray array]; 
// moved outside of the loop and descriptively named: 
double normalizedValue = convertThis * multiply; 
// make use of NSFastEnumeration -- better readability _and_ reliability 
for (NSNumber *scaleFactor in [ourDictionary objectForKey:toString]) 
{ 
    // just in case you want to add a breakpoint to see if everything works as expected: 
    double convertedValue = normalizedValue * [scaleFactor doubleValue]; 
    [convertedValues addObject:[NSNumber numberWithDouble:convertedValue]]; 
} 
return convertedValues; 

Si quelque chose va mal avec ce code, je parie que le tableau retourné par [ourDictionary objectForKey:toString] contient au moins une instance qui n'est pas un NSNumber - que vous reconnaîtrez par un NSException levé à [scaleFactor doubleValue].