堵塞类之间的sortedArrayUsingSelector和/或initWithArray内存泄漏

Question

我已经努力解决这个内存泄漏有一段时间了，所以我希望社区能够提供一些帮助。内存管理仍然是我正在努力理解的一个问题（是的，我有内存管理指南）。

根据 Instruments Leak 工具，一旦我向后导航（带导航控制器的后退按钮）离开相关屏幕，我就会泄漏 NSArray。我在下面展示了我能想到的所有相关代码，如果需要的话可以分享更多。

我知道我正在有序数组函数中分配/初始化一个数组。这是因为，据我所知，sortedArrayUsingSelector 仅返回指向旧数组的指针，而不是真正的副本，因此如果我想保留数组，我需要复制值。

问题是如何将这个排序数组传递给不同的类，同时仍然正确管理我对它的所有权？我在 dealloc 中释放它，如果函数要分配一个新值等，我会释放它。但是说实话，我不知道我是否正确执行了此操作。

就像我说的，我仍然在努力正确理解如何正确处理所有内存管理内容，因此任何帮助将不胜感激。

相关模型类的.h 文件

@interface InstalledDataTracker : NSObject {

    ...other code...

    NSArray *orderedZonesArray;

    ...other code...
}

@property (nonatomic, retain) NSArray *orderedZonesArray;

相关模型类的.m 文件

@synthesize orderedZonesArray;

...other code...

- (NSArray *)orderedZonesArray {
    if (!orderedZonesArray || installedDataChangedSinceLastRead) {
        if (orderedZonesArray) {
            [orderedZonesArray release];
        }
        NSArray *unorderedZones = [NSArray arrayWithArray:[self.installedAreas allKeys]];
        orderedZonesArray = [[NSArray alloc] initWithArray:[unorderedZones sortedArrayUsingSelector:@selector(localizedCompare:)]];
    }
    return orderedZonesArray;
}

- (void) dealloc {

    ...other code...
    [orderedZonesArray release], orderedZonesArray = nil;

    [super dealloc];
}

View Controller 中的.h

#import <UIKit/UIKit.h>

@class InstalledDataTracker;

@interface SBVC_LSC01_ZoneSelect : UIViewController <UITableViewDataSource, UITableViewDelegate> {

    ... other stuff...

    InstalledDataTracker *_dataTracker;
}

@property (nonatomic, retain) InstalledDataTracker *dataTracker;

View 中的 init控制器

@synthesize dataTracker = _dataTracker;

- (id)initWithPerson:(NSString *)person {
    if (self = [super init]) {
        ...other stuff...

        self.dataTracker = [[InstalledDataTracker alloc] init];
    }
    return self;
}

- (void)dealloc
{
    ...other stuff...
    [self.dataTracker release];
    [super dealloc];
}

视图控制器中的泄漏方法

- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath {

    static NSString *CellIdentifier = @"Cell";

    UITableViewCell *cell = [tableView dequeueReusableCellWithIdentifier:CellIdentifier];
    if (cell == nil) {
        cell = [[[UITableViewCell alloc] initWithStyle:UITableViewCellStyleDefault reuseIdentifier:CellIdentifier] autorelease];
    }

    AbbreviationLookup *lookup = [[AbbreviationLookup alloc] init];

    NSString *abbreviatedZone = [self.dataTracker.orderedZonesArray objectAtIndex:[indexPath section]];
    cell.textLabel.text = [lookup zoneForAbbreviation:abbreviatedZone];

    [lookup release];

    return cell;

}

仪器泄漏跟踪：

0 libSystem.B.dylib calloc
1 libobjc.A.dylib class_createInstance
2 CoreFoundation __CFAllocateObject2
3 CoreFoundation +[__NSArrayI __new::]
4 CoreFoundation -[NSArray initWithArray:range:copyItems:]
5 CoreFoundation -[NSArray initWithArray:]
6 -[InstalledDataTracker orderedZonesArray]
7 -[SBVC_LSC01_ZoneSelect tableView:cellForRowAtIndexPath:]

我尝试过的事情

orderedZonesArray = [[[NSArray alloc] initWithArray:[unorderedZones sortedArrayUsingSelector:@selector(localizedCompare:)]] autorelease];

return [orderedZonesArray autorelease];< /code>

还有一堆其他的东西我不记得了。我为正确“释放”由 alloc/init 创建的所有权所做的许多尝试导致视图控制器中出现某种崩溃/错误访问。这让我对在哪里正确释放数组感到困惑......

非常欢迎详细回复。我还有很多东西要学！

非常感谢。 （此外，为了项目安全，我必须更改一些类和方法名称，因此如果某些内容似乎不匹配，请提及它，我将重新检查是否有拼写错误）

编辑：

@Daniel Hicks，当我删除排序数组的 initWithArray 副本时，如下所示：

orderedZonesArray = [unorderedZones sortedArrayUsingSelector:@selector(localizedCompare:)];

当类尝试从视图控制器内访问数组时，我遇到 EXC_BAD_ACCESS 崩溃didSelectRowAtIndexPath 方法（我相信可能是下次访问数组时）。这是方法。它在第二个 NSLog 行上崩溃，所以我将其保留下来以备不时之需：

- (void) tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath *)indexPath {

    NSLog(@"indexPath = %@", indexPath);

    NSLog(@"self.dataTracker.orderedZonesArray = %@", self.dataTracker.orderedZonesArray);

    NSString *abbreviatedZone = [self.dataTracker.orderedZonesArray objectAtIndex:[indexPath section]];    

    SBVC_LSC02_ZoneSelect *slz2 = [[SBVC_LSC02_ZoneSelect alloc] initWithPerson:self.selectedPerson andZone:abbreviatedZone];
    [self.navigationController pushViewController:slz2 animated:YES];
    [slz2 release];

}

Answer 1

在 viewController 代码中，分配一个 InstalledDataTracker 对象，然后将其传递给保留属性。这会导致保留计数为 2，而不是 1。稍后，当您释放 dataTracker 对象时，只会将保留计数减少 1。

修复此问题的最简单方法是删除 self. 前缀，以便您不会调用由属性执行的自动retain访问器。事实上，我建议在 init 和 dealloc 方法中根本不要使用点语法。关于这个问题存在一些争论，但总的来说，我认为最好避免调用您的属性访问器，除非您有充分的理由这样做。

我会这样写：

- (id)initWithPerson:(NSString *)person {
    if (self = [super init]) {
        ...other stuff...

        dataTracker = [[InstalledDataTracker alloc] init];
    }
    return self;
}

- (void)dealloc {
    ...other stuff...
    [dataTracker release];
    [super dealloc];
}

Answer 2

这是因为，据我所知，sortedArrayUsingSelector 仅返回指向旧数组的指针，而不是真正的副本，因此如果我想保留该数组，我需要复制这些值。

这是一种误解。 sortedArrayUsing... 与其他此类函数类似，返回一个数组，其中包含与原始数组中相同的指针 VALUES。并且，当进行排序的数组复制时，所指向的 OBJECTS 的引用计数会增加（即，在每个复制的指针上进行retain）。因此，排序后的数组和原始数组都是“相等”的，并且两个数组都比另一个数组“拥有”更多的对象。 （事实上​​，检查两个数组的内部结构，您无法判断哪个数组是从哪个数组复制的，除非您注意到一个已排序而另一个未排序。）

因此绝对没有必要使已排序数组的附加副本。

当您制作额外的副本时，您将使用 [[alloc] init...] 操作，该操作返回保留的数组。然后，您将该数组返回给调用者，而不对其执行 autorelease 操作，这意味着如果您的调用者没有显式 release 它，它将泄漏，并且意味着分析器会抱怨它（因为您只能从 copy... 等返回保留的对象）。