NSMutableArray：调用方法时的内存管理

Question

在我的 iOS 应用程序中，我使用了一个名为 imageMArray 的 NSMutableArray。我已经设置了它的 getter 和 setter 属性并实例化了它。

在 viewDidLoad 中：

imageMArray=[[NSMutableArray alloc] initWithArray:CategoryImages];
imageMArray=[self shuffleOnlyArray:imageMArray];

在 ShuffleOnlyArray 方法中：

 NSMutableArray *destArray1 = [[NSMutableArray alloc] initWithCapacity: [sourceArray count]] ;
return destArray1;

随机播放方法：

imageMArray=[[self shuffleOnlyArray:imageMArray] retain];

Shuffle 方法中似乎存在内存泄漏。

我应该释放 imageMArray 还是将其设置为 nil？如果应该释放，是否应该自动释放？

Answer 1

imageMArray=[[NSMutableArray alloc] initWithArray:CategoryImages];

在上面的语句中，你有一个内存泄漏。 相反，您可以如下所示。

imageMArray = [NSMutableArray arrayWithArray:CategoryImages];

在 ShuffleOnlyArray 方法中，返回自动释放的对象。

NSMutableArray *destArray1 = [[NSMutableArray alloc] initWithCapacity: [sourceArray count]] ;
return [destArray1 autorelease];

但是在你得到它之后，保留（取得所有权）这个数组对象。

imageMArray=[[self shuffleOnlyArray:imageMArray] retain];

编辑

在shuffle方法中，执行如下操作：

NSMutableArray *imageMArray1 = [imageMArray mutableCopy];
if( imageMArray )
{
   [imageMArray release];
}
imageMArray=[[self shuffleOnlyArray:imageMArray1] retain];
[imageMArray1 release];

编辑 2： 另一种解决方案：

使用SO link中提到的类别随机播放

无需新建和释放数组。

Answer 2

1 您在以下几行中已经存在内存泄漏。

imageMArray = [[NSMutableArray alloc] initWithArray:CategoryImages];

imageMArray = [self shuffleOnlyArray:imageMArray];

在第一行中，您创建了一个保留计数为 1 的对象。 然后你说你的 imageMArray 指针指向其他对象。你应该释放第一个对象，因为你对第一个对象的引用很糟糕，并且在更改引用后无法释放它！

2 你不应该使用retain，因为你的ShuffleOnlyArray 方法返回一个retained 对象。 你的工厂方法应该返回一个自动释放对象，工厂的调用者应该决定是否保留它。

希望我足够清楚