문제 루프에서 NSMutableArray에 추가

Question

이 루프가 무슨 문제인지 정확히 모르겠지만 실행할 때마다 SIGABRT가 계속 나타납니다. 로그에 따르면 루프의 끝 부분에있는 NSMutable 배열에 NSNumber를 추가하려고하면 문제가 발생합니다. 분명히 초등 오류를 만들었지 만 문제가 무엇인지 잘 모르겠습니다.

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]를 사용해보세요. 이것은 나중에 배열을 해제 할 것이고, 호출 메소드가 미출력 배열을 사용하거나 나중에 사용할 수 있도록 배열을 유지할 기회를 제공 할 것입니다.

배열에 추가하기 전에 제품이 nil이 아닌지 확인할 수도 있습니다.

그리고 메모리 누수가 발생했습니다.

제품 = [[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]에 던져짐으로써 알 수 있습니다.