释放使用 initFromFile 方法创建的对象时崩溃

Question

我创建了一个名为“DateTracker”的对象，它符合 NSCoding，因此它包含 encodeWithCoder 和 initWithCoder 方法。当我初始化它时，我调用以下内容：

DateTracker *currentTracker = [[DateTracker alloc] initFromFile];

initFromFile 方法如下所示：

- (id)initFromFile { 
    NSString *filePath = [self dataFilePath];
    if ([[NSFileManager defaultManager] attributesOfFileSystemForPath:filePath error:NULL]) {
        NSData *data = [[NSMutableData alloc] initWithContentsOfFile:filePath];
        NSKeyedUnarchiver *unarchiver = [[NSKeyedUnarchiver alloc] initForReadingWithData:data];
        self = [unarchiver decodeObjectForKey:kDateDataKey];
        [unarchiver finishDecoding];
        [unarchiver release];
        [data release];
    }
    return self;
}

但是当我尝试打电话时

[currentTracker release];

我的应用程序崩溃了。

当我使用性能工具运行应用程序以检查内存泄漏时，它会抱怨我没有释放此对象。

任何想法我做错了什么？

Answer 1

这一行：

self = [unarchiver decodeObjectForKey:kDateDataKey];

会给你带来麻烦。

您正在做的是分配一个 DateTracker 对象 ([DateTracker alloc])，然后创建一个新的 DateTracker 对象（由-decodeObjectForKey:）并让“self”指针指向新对象。这样做有两个问题：

• 你不再有对旧对象的引用，所以它被泄露了
• 新对象未保留，因此它会消失（如果您尝试释放它会导致崩溃）

我会说让对象替换自身的方法有点可疑。也许您最好将filePath 变量移到DateTracker 对象之外，然后通过以下方式取消归档：

DateTracker *currentTracker = [[DateTracker unarchiveFromFile:filePath] retain];

其中unarchiveFromFile: 是一个类方法，它基本上完成了initFromFile 所做的工作，而不会与self 混淆：

+ (DateTracker*)unarchiveFromFile:(NSString *)filePath { 
    DateTracker *result = nil;
    if ([[NSFileManager defaultManager] attributesOfFileSystemForPath:filePath error:NULL]) {
        NSData *data = [[NSMutableData alloc] initWithContentsOfFile:filePath];
        NSKeyedUnarchiver *unarchiver = [[NSKeyedUnarchiver alloc] initForReadingWithData:data];
        result = [unarchiver decodeObjectForKey:kDateDataKey];
        [unarchiver finishDecoding];
        [unarchiver release];
        [data release];
    }
    return result;
}