2

我创建了 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
4

5 回答 5

2

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

也就是说，我相信您的大部分问题都来自您的对象模型错误这一事实。既不作为类也没有意义GetBooks。DownloadAndParseBook您可能的意思是BookManager（保存所有书籍的BookDownloadController东西）和（管理一本书的下载的东西）。BookManager应该跟踪所有当前（BookDownloadControllers在 anNSSet或NSArrayivar 中）。每个人都BookDownloadController应该跟踪它NSURLConnection（在 ivar 中）。您不应该只是创建连接并让它们“自力更生”（即自我保留）。这感觉很方便，但它使代码以后很难处理。您无法控制要建立的连接数。您无法取消连接。它很快就变得一团糟。

于 2012-04-10T14:27:38.953 回答
1

不，这不是最佳实践。保留/释放您的对象应该由您的对象的“所有者”完成。对于您的特定示例，您的 DownloadAndParseBook 对象的所有者是执行分配/初始化的对象。那应该是保留/释放您的 DownloadAndParseBook 实例的原因。这里的最佳实践是对 DownloadAndParseBook 进行分配/初始化，保留由所有者完成的所有下载/解析逻辑，然后向所有者发送回调以告知所有操作已完成（例如通过委托），此时， ow 向您的对象发送释放消息。

于 2012-04-10T13:39:44.537 回答
0

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

于 2012-04-10T13:02:03.923 回答
0

与其他响应者不同，我会说您的模式可能有效。另请参阅是否允许调用 [self release] 来控制对象生命周期？

但是，您的代码中还有一些其他问题：

  • 在-(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它也不能自动释放自己，因为它必须在请求被服务之前一直处于活动状态。所以它可以工作的唯一方法是上面描述的模式

于 2012-04-10T13:53:18.197 回答
0

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

于 2012-04-10T14:24:24.957 回答