Проблема Добавление в NSMutableArray в Loop

Question

Я не уверен, в чем проблема с этим циклом, но я продолжаю получать SIGABRT всякий раз, когда я его запускаю. Согласно журналу, проблема заключается в том, что я пытаюсь добавить NSNumber в массив NSMutable в конце цикла. Очевидно, я сделал элементарную ошибку, но я не уверен, в чем проблема.

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

Вы выпускаете allValuesMArray: [allValuesMArray release];

Поскольку в этот момент количество удержаний равно 0, это немедленно освободит массив.

Попробуйте использовать [allValuesMArray autorelease]. Это позволит освободить массив в будущем, предоставив возможность вызывающему методу использовать неизданный массив или сохранить массив для последующего использования.

Также вы можете проверить, нет ли продукта, кроме того, чтобы добавить его в массив.

И у вас есть утечка памяти.

продукт = [[NSNumber alloc] init];

Затем вы назначаете ему другой объект позже.

Answer 2

Я уверен, что это опечатка, но вы не выделяете указатель и даже тип NSMutableArray, но типа NSArray. Проверить номер строки. 2:

NSMutableArray allValuesMArray = [[NSArray alloc] init]; 

Это должно быть

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

Answer 3

Вы создавая NSArray и назначая его на NSMutableArray? Возможно, вы имели в виду следующее:

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

Answer 4

Вот ваш код в Начищенное, герметичном вид:

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; 

Если что-то пойдет не так с этим кодом, я уверен, что массив, возвращаемый [ourDictionary objectForKey:toString] содержит, по крайней мере, один экземпляр, который не является NSNumber, который вы узнаете по NSException, который был сброшен на [scaleFactor doubleValue].