我创建了 DownloadAndParseBook 类。在收到任何数据或网络错误之前,它不会自动发布。
我使用了[ self 释放],[ self 保留]。使用[ self 释放],[ self 保留]是一种好方法吗? DownloadAndParseBook 是否包含任何潜在的错误?
@implementation GetBooks
-(void) books
{
for(int i =0; i<10; i++)
{
DownloadAndParseBook *downloadAndParseBook =
[[[DownloadAndParseBook alloc] init]autorelease];
[downloadAndParseBook startLoadingBook];
}
}
@end
@implementation DownloadAndParseBook
- (id)initWithAbookint)bookID
{
if(self = [super init])
{
[self retain];
}
return self;
}
- (void)startLoadingBook
{
[NSURLConnection connectionWithRequest:request delegate:self];
}
- (void)connectionNSURLConnection *)connection didFailWithErrorNSError *)error
{
[self release];
}
- (void)connectionDidFinishLoadingNSURLConnection *)connection
{
[self saveResultToDatabase];
[self release];
}
@end
Best Answer-推荐答案 strong>
self 保留是一种非常合适的模式。这种情况很少见,但有时在某些类型的多线程代码中,确保您不会在处理某些东西的过程中消失很重要。也就是说,这不是那些时代之一。我无法想象您当前的方法会有所帮助的情况。如果有人创建了您的对象,然后从不调用 startLoadingBook ,那么它就会泄漏。如果有人调用 startLoadingBook ,那么无论如何都会保留您的对象,因为 NSURLConnection 会保留其委托(delegate),直到完成。
也就是说,我相信您的大部分问题都来自您的对象模型错误这一事实。 GetBooks 和 DownloadAndParseBook 作为类都没有意义。您可能的意思是 BookManager (保存所有书籍的东西)和 BookDownloadController (管理单本书下载的东西)。 BookManager 应该跟踪所有当前的 BookDownloadControllers (在 NSSet 或 NSArray ivar 中)。每个 BookDownloadController 都应该跟踪它的 NSURLConnection (在 ivar 中)。您不应该只是创建连接并让它们“自食其力”(即 self 保留)。这感觉很方便,但它使代码以后很难处理。您无法控制要建立的连接数。您无法取消连接。很快就会变得一团糟。
关于ios - 使用[ self 释放],[ self 保留]是一种好方法吗?,我们在Stack Overflow上找到一个类似的问题:
https://stackoverflow.com/questions/10089211/
|