コピー後にNSMutableArrayのすべての要素が消えます。

Question

NSMutableArrayを使用してビューコントローラを初期化しています。しかしinitでは、NSMutableArrayで初期化する必要がありますが、それ以降はすべての要素が消えてしまいます。

UIViewController* vc = [[[PhotoAlbumViewController alloc] initWithImages:imageArrayBig] autorelease]; 

以下は、上記の機能の定義です。

// This is the definition of the init function 
- (id)initWithImages:(NSMutableArray *)bigImages { 

    if ((self = [self initWithNibName:nil bundle:nil])) { 
     _imageArrayBig = [[NSMutableArray alloc] init]; 
     for (ArrayObject* tmp in bigImages) { 
      [_imageArrayBig addObject: tmp]; 
     } 
    } 
    // After the copying, all the elements in bigImages and _imageArrayBig disappear :(
    return self; 
} 

メモリリークはありますか？

Answer 1

imageArrayBigと_imageArrayBigの両方を使用しているため、プロパティ名とインスタンス名が混在しているようです。

あなたがまた

NSMutableArray* tmp = [[NSMutableArray alloc] init]; 
[tmp addObjectsFromArray:bigImages]; 
self.imageArrayBig = tmp; 
[tmp release]; 

よう addObjectsFromArray

何か

Answer 2

あなたが投稿したコードを使用することにより、より効果的に配列をコピーすることができますプロパティ

である好ましくself.imageArrayBig、1に固執する方が良いです@endersが示唆するように最適化することができます。あなたは財産を必要としません。 initに渡すときにbigImagesにオブジェクトがあることは確かですか？ NSLogを追加して確認してください。

- (id)initWithImages:(NSMutableArray *)bigImages { 
    self = [self initWithNibName:nil bundle:nil]; 
    if (self) { 
     NSLog(@"MyController:initWithImages: bigImages count = %d", [bigImages count]); 
     _imageArrayBig = [[NSMutableArray alloc] initWithArray:bigImages]; 
    } 
    return self; 
} 

なぜ可変配列を渡す必要があるのか​​明確ではありません。新しい可変配列を作成する場合は、NSArrayを渡すことができます。