在循环中添加到 NSMutableArray 时出现问题

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

我确信这是一个拼写错误，但您没有分配指针，甚至不是 NSMutableArray 类型，而是分配 NSArray 类型。检查行号2：

NSMutableArray allValuesMArray = [[NSArray alloc] init];

应该是

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

Answer 2

您是否正在创建 NSArray 并将其分配给 NSMutableArray？也许您的意思如下：

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

Answer 3

您正在释放 allValuesMArray: [ allValuesMArray release];

由于此时保留计数为 0，因此这将立即释放数组。

尝试使用 [ allValuesMArray autorelease ]。这将在将来释放该数组，使调用方法有机会使用未释放的数组或保留该数组以供以后使用。

您还可以在将产品添加到数组之前检查它是否不为零。

而且你有内存泄漏。

产品 = [[NSNumber 分配] 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] 返回的数组至少包含一个不存在的实例t 一个 NSNumber — 您可以通过在 [scaleFactor doubleValue] 处抛出 NSException 来识别它。