問題ループ内の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]; 

それはあなたがNSArrayのを

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

Answer 3

を作成していると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ではないインスタンスが少なくとも1つ含まれています。NSExceptionが[scaleFactor doubleValue]にスローされると認識します。