4

我不明白这个，除非是因为我要释放财产而不是 ivar。有人可以阐明这个问题吗？

    self.dataToBeLoaded = [[NSMutableData alloc] initWithLength:10000];
    [self.dataToBeLoaded release];

警告是Incorrect decrement of the reference count of an object that is not owned by the caller。

该dataToBeLoaded属性具有与其设置器关联的保留属性。

我的理解是 alloc init 增加了保留计数，而属性分配增加了保留计数。由于我只保留一次，所以我在分配后立即释放它。

更新——一些实验结果：

由于我在下面的评论中指出，我收到了关于保留属性对合成设置器的作用的相互矛盾的建议，我想我会使用上面的代码做一个小实验，并通过一些日志记录进行修改：

NSLog(@"retain 1 = %d", [dataToBeLoaded_ retainCount]);
self.dataToBeLoaded = [[NSMutableData alloc] initWithLength:10000];
NSLog(@"retain 2 = %d", [dataToBeLoaded_ retainCount]);
[self.dataToBeLoaded release];
NSLog(@"retain 3 = %d", [dataToBeLoaded_ retainCount]);

每个日志语句的结果是 0、2 和 1。

显然，不可能进入 alloc 或 init 代码来查看保留计数从 0 到 1 到 2。我可以将 NSMutableData 类子类化，但我的时间很短。

我知道很多人说你不能依赖 retainCount 属性的值，但我所拥有的似乎是一致的，我希望在示例中显示的代码的短范围内有合理的行为。所以我倾向于相信之前的建议是正确的——retain 属性是一个在 setter 中包含一个 retain 的承诺。所以在这里我有来自 alloc/init 的保留和来自对 setter 的调用的保留。因此，保留计数设置为 2。

当我运行此代码时：

NSMutableData *theData;
NSLog(@"retain 1 = %d", [theData retainCount]);
theData= [[NSMutableData alloc] initWithLength:10000];
NSLog(@"retain 1a = %d", [theData retainCount]);
self.dataToBeLoaded = theData;
NSLog(@"retain 2 = %d", [theData retainCount]);
[self.dataToBeLoaded release];
NSLog(@"retain 3 = %d", [theData retainCount]);

每个日志语句的保留计数为 0、1、2、1。

所以我有证据表明二传手提供了一个retain. 这似乎更像是一个承诺而不是一个暗示，因为它实际上正在发生。

我愿意接受其他解释。我不想在这件事上傲慢自大。我只是想弄清楚正在发生的事情。我似乎警告（在这个问题的主题中）确实是虚假的，不需要担心。

另一个实验是在@property 语句中使用assign而不是retain作为属性来完成的。使用相同的代码：

NSMutableData *theData;
NSLog(@"retain 1 = %d", [theData retainCount]);
theData= [[NSMutableData alloc] initWithLength:10000];
NSLog(@"retain 1a = %d", [theData retainCount]);
self.dataToBeLoaded = theData;
NSLog(@"retain 2 = %d", [theData retainCount]);
[self.dataToBeLoaded release];
NSLog(@"retain 3 = %d", [theData retainCount]);

每条日志的retain count是0,1,1（setter没有retain），然后报错：message sent to deallocated instance. 上一个版本将保留计数设置为零，这触发了释放。

更新 2

最后的更新——当合成的 setter 被你自己的代码覆盖时，不会再观察到 retain 属性，除非你的 setter 明确包含它。显然（这与我在其他线程中被告知的内容相矛盾）如果这是你想要的，你必须在 setter 中包含你自己的保留。虽然我这里没有测试，但你可能需要先释放旧实例，否则它会被泄露。

此自定义设置器不再具有 @propety 声明的属性：

- (void) setDataToBeLoaded:(NSMutableData *)dataToBeLoaded {
    dataToBeLoaded_ = dataToBeLoaded;
}

这是有道理的。覆盖一个综合的设置器，你会覆盖所有声明的属性。使用综合设置器，并在综合实现中观察声明的属性。

@property 属性表示关于如何实现合成设置器的“承诺”。一旦你编写了一个自定义设置器，你就靠自己了。

4

4 回答 4

4

关键是想清楚下面的代码在做什么。为了清楚起见，我会完整地写出来：

[self setDataToBeLoaded:[[NSMutableData alloc] initWithLength:10000]];

这将创建一个具有 +1 保留计数的对象并将其传递给setDataToBeLoaded:. (*) 然后它丢弃对该对象的引用，将其泄漏。

[[self dataToBeLoaded] release];

这会调用dataToBeLoaded并释放返回的对象。没有任何保证返回dataToBeLoaded的对象与传递给的对象相同setDataToBeLoaded:。您可能认为它们是相同的，并且查看您的代码您可能会说服自己它总是会以这种方式工作，但这不是 API 承诺。

Antwan发布的代码是正确的：

NSMutableData *data = [[NSMutableData alloc] initWithLength:1000];
self.dataToBeLoaded = data;
[data release];

这将创建一个具有 +1 保留计数的对象。然后将它传递给一个方法，然后释放它。

或者，如果您愿意使用自动释放池，您可以将其简化为：

self.dataToBeLoaded = [NSMutableData dataWithLength:1000];

(*) 从技术上讲，这将消息传递给self可能会或可能不会导致调用此方法的消息，但这会使问题变得混乱。在大多数情况下，假设它是一个方法调用。但不要假装它只是设置属性。它真的会调用一些方法。

编辑：

也许这段代码会使问题更清楚一些。它表示常见的缓存解决方案：

.h
@interface MYObject : NSObject 
@property (nonatomic, readwrite, strong) NSString *stuff;
@end

.m
@interface MYObject ()
@property (nonatomic, readwrite, weak) MYStuffManager *manager;

@implementation MYObject

... Initialize manager ...

- (NSString*)stuff {
  return [self.manager stuffForObject:self];
}

- (void)setStuff:(NSString *)stuff {
   [self.manager setStuff:stuff forObject:self];
}

现在也许manager在后台做了一些愚蠢的事情。也许它缓存了stuff. 也许它会复制它们。也许它将它们包装到其他对象中。重要的是你不能-stuff总是依赖于返回你传递给的同一个对象-setStuff:。所以你当然不应该释放它。

请注意，标题中没有任何内容表明这一点，也没有任何内容应该表明这一点。这不是来电者的事。但是如果调用者释放了 的结果-stuff，那么你将遇到难以调试的崩溃。

@synthesize只是编写一些乏味代码的简写（实现stuff并setStuff:作为读写 ivar 的代码）。但没有什么说你必须@synthesize为你的财产使用。

于 2012-05-13T00:24:27.300 回答
2

我的猜测是该方法

- (NSMutableData *)dataToBeLoaded;

不包含任何内存管理关键字，因此假定您不拥有返回的数据，因此不应释放它。

要么使用

NSMutableData *data = [[NSMutableData alloc] initWithLength:1000];
self.dataToBeLoaded = data;
[data release]; data = nil;

或者如果您可以在实际需要它时为什么不延迟加载它？

- (NSMutableData *)dataToBeLoaded;
{
    if (!_dataToBeLoaded) {
        _dataToBeLoaded = [[NSMutableData alloc] initWithLength:1000];
    }
    return _dataToBeLoaded;
}
于 2012-05-12T23:37:45.623 回答
1

这只是意味着您正在释放一个您不拥有的对象。

我会说直接使用实例 var 调用它，而不是使用 getter，但不确定这是否会修复您的分析警告。还有为什么不使用 [NSMutableData dataWithLength:1000]; 它是自动发布的，因此不需要额外的发布调用（并且可能也会摆脱那个警告！）

其他可以解决的方法：

NSMutableData *data = [[NSMutableData alloc] initWithLength:1000];
self.databToBeLoaded = data;
[data release];
于 2012-05-12T23:29:51.763 回答
0

我提供了一些更新，我认为这些更新可以回答这里发生的事情。通过一些测试结果，我的结论是这个警告是虚假的，这意味着它并没有真正识别出不正确的代码。更新应该不言自明。它们在上面给出。

于 2012-05-17T06:22:14.677 回答