Question

我创建了DownloadAndParseBook类。在发生任何数据或网络错误之前，它不会自动重新调整。

我用[自我释放]，[自我保留]。这是使用[自我释放]，[自我保留]的好方法吗？ DownloadAndParseBook是否包含任何潜在的错误？

@implementation GetBooks

-(void) books
{ 
 for(int i =0; i<10; i++)
 {
   DownloadAndParseBook *downloadAndParseBook =
        [[[DownloadAndParseBook alloc] init]autorelease];
   [downloadAndParseBook startLoadingBook];
 }
}
@end


@implementation DownloadAndParseBook

- (id)initWithAbook:(int)bookID 
{
 if(self = [super init])
 { 
    [self retain];        
 }
 return self;
}

- (void)startLoadingBook
{
 [NSURLConnection connectionWithRequest:request delegate:self];
}

- (void)connection:(NSURLConnection *)connection didFailWithError:(NSError *)error
{
  [self release];    
}

- (void)connectionDidFinishLoading:(NSURLConnection *)connection
{
 [self saveResultToDatabase];
 [self release];
}


@end

Answer 1

自我保留偶尔也是一种合适的模式。这种情况很少见，但有时在某些类型的多线程代码中，确保在处理某些内容时不会消失是非常重要的。那就是说，这不是那个时代之一。我想象一下你当前的方法会有所帮助的情况。如果有人创建了您的对象，然后从不调用startLoadingBook，那么它就会泄漏。如果有人拨打startLoadingBook，则无论如何都会保留您的对象，因为NSURLConnection会保留其委托直到完成。

那就是说，我相信你的问题很大程度上来自于你的对象模型是错误的。 GetBooks和DownloadAndParseBook都不是有意义的。您可能意味着BookManager（可以容纳所有书籍）和BookDownloadController（用于管理单本书下载的内容）。 BookManager应跟踪所有当前BookDownloadControllers（NSSet或NSArray ivar）。每个BookDownloadController都应该跟踪其NSURLConnection（在ivar中）。你不应该只是创建连接并让它们“挂起”（即自我保留）。这感觉很方便，但是这使代码很难在以后处理。您无法控制正在建立的连接数。您无法取消连接。它很快变得一团糟。

Answer 2

不，这不是最佳做法。保留/释放对象应该由对象的“所有者”完成。对于您的特定示例，DownloadAndParseBook对象的所有者是执行alloc / init的对象。那应该是保留/释放你的DownloadAndParseBook实例。这里的最佳实践是DownloadAndParseBook的alloc / init，由所有者保留，所有下载/解析逻辑，然后向所有者发送回调所有操作都已完成（例如通过委托），此时， ower向您的对象发送一条发布消息。

Answer 3

问题是：为什么一个物体需要保留自己？您可能希望像单身一样实现您的类。

Answer 4

与其他响应者不同，我会说你的模式可能有效。另请参阅Is calling [self release] allowed to control object lifetime?

您的代码中还存在其他一些问题：

在-(void) books我想您要将startLoadingBook讯息发送至downloadAndParseBook而不是self
如果您创建initWithAbook方法，则在使用标准init方法初始化图书时，系统不会调用该方法。在上面的当前代码中，[self retain]永远不会被称为
在上面的代码中，bookID将无法保存
我不会在这里使用“init”模式，但是静态函数中的所有内容因此调用者不会错误地使用类的所有权。

代码：

- (id) initWithId:(int)bookId {
  self = [super init];
  if (self) {
    // save bookId here
  }
  return self;
}

+ (void) startLoadingBookWithID:(int)bookId {
  DownloadAndParseBook* book = [[DownloadAndParseBook alloc] initWithId:bookId];
  [NSURLConnection connectionWithRequest:request delegate:book];
}

// release self when it finished the operation 
// and document well that its behaviour

如果您认为合适，NSURLConnection本身应该以完全相同的方式工作：当您完成工作时不释放NSURLConnection时，它会自行完成。但是在connectionWithRequest中它也不能自动释放自己，因为它必须是活着的，直到请求被提供。因此，唯一可行的方法是上述模式

Answer 5

永远不要使用[self release]。唯一可能的例外是单例类/对象。方法release和retain应仅由对象的所有者发送。这通常意味着，无论创建相关对象的对象，也应该是释放它的对象。

这是使用[自我释放]，[自我保留]的好方法吗？

5 个答案: