for循环中的NSMutableArray addObject - 内存泄漏

Question

我将字符串（即某个目录中文件的文件名）放入带有 for 循环的 NSMutableArray 中： h 文件：

#import <Three20/Three20.h>

@interface AlbumController : TTThumbsViewController {
    NSMutableArray *images;
}

@property (nonatomic, retain) NSMutableArray *images;

@end

m 文件：

#import "AlbumController.h"
#import "PhotoSource.h"
#import "Photo.h"
@implementation AlbumController
@synthesize images;



-(void)createPhotos {
    NSString *bundleRoot = [[NSBundle mainBundle] bundlePath];
    NSArray *dirContents = [[NSFileManager defaultManager] contentsOfDirectoryAtPath:bundleRoot error:nil];
    NSArray *onlyJPGs = [dirContents filteredArrayUsingPredicate:[NSPredicate predicateWithFormat:@"self ENDSWITH '.jpg'"]];

    NSMutableArray *pics = [[onlyJPGs copy] autorelease];



        if(!self.images) {
 self.images = [[NSMutableArray alloc] init];
}

    for(int i = 0; i < [onlyJPGs count]; i++)
    {
        //NSLog([pics objectAtIndex:i]);


        NSString *ImgURL = [@"bundle://" stringByAppendingString:[pics objectAtIndex:i]];

            Photo *photo = [[Photo alloc] initWithURL:ImgURL smallURL:ImgURL size:CGSizeMake(320, 212)];
[images addObject:photo];
[photo release];

        }



}
-(void)viewDidLoad{

    [self createPhotos]; // method to set up the photos array
    self.photoSource = [[PhotoSource alloc]
                        initWithType:PhotoSourceNormal
                        title:@"Chili Pflanzen"
                        photos:images
                        photos2:nil
                        ];
}

@end

我在模拟器上没有任何问题，但在我的 iPod 上...

错误信息：

数据格式暂时不可用，将在“继续”后重试。 （加载共享库“/Developer/usr/lib/libXcodeDebuggerSupport.dylib”时出现未知错误）

提前致谢

Answer 1

我认为你应该使用 mutableCopy 而不是在你的 pics 数组上复制。

所以而不是： NSMutableArray *pics = [[onlyJPGs copy] autorelease]; 你应该使用： NSMutableArray *pics = [[onlyJPGs mutableCopy] autorelease];

在这个问题中有关复制/可变复制的更多信息：Copy & mutableCopy?

Answer 2

看起来主要问题在于

 [images addObject:[[Photo alloc] initWithURL:ImgURL smallURL:ImgURL size:CGSizeMake(320, 212)]];

在这里，您正在分配照片，但没有释放它。当您将对象添加到数组时，它会增加它的保留计数。

试试改成

Photo *photo = [[Photo alloc] initWithURL:ImgURL smallURL:ImgURL size:CGSizeMake(320, 212)];
[images addObject:photo];
[photo release];

---

另外...

我会改变

 self.images = [[[NSMutableArray alloc] init] autorelease];

到

if(!self.images) {
 self.images = [[NSMutableArray alloc] init];
}

否则，如果它已经初始化，则可能会发生内存泄漏，并且您可能不希望它自动释放；

Answer 3

您的 NSMutableArray 实例已自动释放。您将其分配给images ivar。您已将其声明为保留属性这一事实并不重要，因为您没有将其分配给该属性。我的猜测是您的意思是分配给该属性，而崩溃是由无意的释放引起的。

变化：

images = [[[NSMutableArray alloc] init] autorelease];

...到：

self.images = [[[NSMutableArray alloc] init] autorelease];

...或：

images = [[NSMutableArray alloc] init];

另请注意，当您分配 NSMutableArray 的实例时，您的属性被声明为 NSArray。

另见the Memory Management Programming Guide。