在方法中创建自动释放的对象并将其返回到实例变量会导致崩溃

Question

我在方法中创建一个对象并将其返回给一个变量，这让我变得一团糟。正如这篇文章中那样，我知道我应该在这种情况，但是当我这样做时，它崩溃了。

我编写了一个方法来创建图像数组并返回该数组。它看起来像这样：

- (NSMutableArray *)createImagesFor:(NSString *)animName withFrames:(int)numberFrames {
    NSMutableArray *imageArray = [[NSMutableArray alloc] initWithCapacity:numberFrames];
    for (int i = 1; i <= numberFrames; ++i) {
        NSString *imageName = [[NSString alloc]initWithFormat:@"%@%i.png", animName, i];
        [imageArray addObject:[UIImage imageNamed:imageName]];
        [imageName release];
    }
    return imageArray;
}

我这样称呼它：

NSMutableArray *imageArray;
imageArray = [self createImagesFor:@"jumping" withFrames:2];
self.animationImages = imageArray;
[imageArray release];

但是，当我运行构建分析器时，它会编译，但有以下抱怨：

第 109 行分配的对象存在潜在泄漏
1. 方法返回一个 Objective-C 对象，其保留计数+1（拥有引用）
2. 对象作为拥有引用返回给调用者（单个保留计数转移给调用者）
3. 第 109 行分配的对象是从名称 ('createImagesFor:withFrames:') 不包含 'copy' 或以 'new' 或 'alloc' 开头的方法返回的。这违反了 Cocoa 内存管理指南中给出的命名约定规则（对象泄漏）

我查看了 内存管理文档，但除了自动释放变量（这会使其崩溃）之外，我不确定哪里出错了。这就是我自动释放它的方式：

NSMutableArray *imageArray = [[[NSMutableArray alloc] initWithCapacity:numberFrames]autorelease];

我尝试将 *imageArray 保留为 这里建议像这样：

NSMutableArray *imageArray;
[imageArray retain];
    imageArray = [self createImagesFor:@"jumping" withFrames:2];
    self.animationImages = imageArray;
    [imageArray release];

但这也会崩溃。

分析器建议我将该方法的名称更改为“newCreateImagesFor:withFrames:”之类的名称，但我不明白这是如何解决问题的？

感谢您的帮助。

迈克尔

Answer 1

您应该将第一块代码的最后一行更改为 return [imageArray autorelease] 并摆脱第二部分中的释放（通常不需要释放方法调用返回的对象） 。这就是分析仪抱怨的内容。但是，我不明白为什么它会导致崩溃。

animationImages 属性是如何定义的？这可能是你问题的根源。

Answer 2

    - (NSMutableArray *)createImagesFor:(NSString *)animName withFrames:(int)numberFrames {
    NSMutableArray *imageArray = [[NSMutableArray alloc] initWithCapacity:numberFrames];
    for (int i = 1; i <= numberFrames; ++i) {
        NSString *imageName = [[NSString alloc]initWithFormat:@"%@%i.png", animName, i];
        [imageArray addObject:[UIImage imageNamed:imageName]];
        [imageName release];
    }
    return imageArray;
}

返回一个 imageArray 对象，该对象的保留计数为 +1 但不会自动释放，clang 静态分析器会警告您这一点，这与命名约定有关，因为您的方法未命名类似于 newImagesFor... 或 allocImagesFor... 或 copyImagesFor...。

---

NSMutableArray *imageArray;

[imageArray retain];
// sending message to nil, does nothing

imageArray = [self createImagesFor:@"jumping" withFrames:2];
// imageArray has a retain count of 1
// and it is an autoreleased object

self.animationImages = imageArray; 

[imageArray release]; 
// retain count = 0, will be dealloc'd, 
// however it is already in the autorelease pool, 
// it will be over-released at the end of current event run loop

当您声明 imageArray 时，它是一个指向类 NSMutableArray 的空指针，在您的情况下发送一条消息 retain 到一个空指针Objective-C 是可以的并且不会抛出异常。

---

如果您使用属性访问器来缓存 imageArray 对象，则应声明您的属性访问器以保留您分配给的对象。

@property (retain) NSMutableArray *imageArray;

---

既然您的方法正确返回了自动释放的 imageArray，并且你有一个正确的属性访问器，所需要的只是

NSMutableArray *imageArray;
imageArray = [self createImagesFor:@"jumping" withFrames:2];
self.animationImages = imageArray;