多线程-NSOperation中使用ASIHttpRequest注意事项

时间:2022-09-03 21:26:08

最近做的iPhone项目中有一如下功能:

app在用户许可后将本地Photos的照片上传到服务器,期间用户可以做其他任何操作,等上传成功后弹出一个toast通知用户。

原先的代码结构是:

  1. 获取照片的操作放在NSOperation的子类A中
  2. 获取完照片后,逐个生成一个上传类B(此上传类是ASIFormDataRequest的子类),并把它添加到NSOperationQueue中。

其中operationqueue设置了最大运行数是1,但是实际测试下来发现所有的上传都是并发的,一查代码,发现上传类B居然没有实现main,就一个init函数。初始化完之后直接startAsynchronous了,然后返回self。真是奇葩~~

于是将上传类B修改,添加了main函数,但是运行的时候出错:

- (void)reportFinished
{
if (delegate && [delegate respondsToSelector:didFinishSelector]) {
[delegate performSelector:didFinishSelector withObject:self];
}---------------------->提示bad_access的错误 #if NS_BLOCKS_AVAILABLE
if(completionBlock){
completionBlock();
}
#endif if (queue && [queue respondsToSelector:@selector(requestFinished:)]) {
[queue performSelector:@selector(requestFinished:) withObject:self];
}
}

查看delegate的值,发现已经overrelease了。B在设置的时候,将delegate设置为A的实例了,A的实例怎么会不等B的返回就结束了呢?

原来A本身是一个operation,假设运行在次线程 M中。B因为是继承ASIFormDataRequest,其实也是一个NSOperation,也就是说B运行的时候也是运行在次线程N中的。因为B使用的是异步运行,N必然不同于M。而A在将上传操作结束完以后,就结束了,系统就会回收A的内存。这个时候在N中运行的B尚未收到响应。等到response返回的时候,A早就已经释放了,所以就会有如上的错误。

怎么解决呢?有同事是把A设置为property。这样可以解决,但是当需要调用A的类很多的时候,就会比较麻烦。

其实解决的办法很简单,就是在A中重载isFinished方法,当确定所有的照片上传上去后返回YES否则返回NO,这样我们就可以控制A,避免系统“过早”的释放。

与此同时我们发现,ASIHttpRequest的delegate响应都会路由到主线程:

- (void)requestFinished
{
#if DEBUG_REQUEST_STATUS || DEBUG_THROTTLING
NSLog(@"[STATUS] Request finished: %@",self);
#endif
if ([self error] || [self mainRequest]) {
return;
}
if ([self isPACFileRequest]) {
[self reportFinished];
} else {
[self performSelectorOnMainThread:@selector(reportFinished) withObject:nil waitUntilDone:[NSThread isMainThread]];
}
}

*上有关于这样做的讨论,这里要说明的是因为B是运行在后台,delegate是A,不需要在主线程响应。我们可以在B中重载上述函数,将performSelectorOnMainThread:函数去掉,直接调用reportFinished。

进一步考虑,iOS上获取本地照片现在一般用ALAssetsLibrary,这个库一般是用block去枚举,换言之获得照片内容的操作已经是在次线程中操作的了。

这样一来A也就可以不需要是NSOperation,是个一般的NSObject即可。

最近项目添加了很多“奇葩”的功能,可是参与的决定权不在自己这边,虽然我列出了很多不应该这样做的理由和依据。但是需求人员都以本国的用户需求为借口——看来公司越来越成为外资公司在华的外包公司了。这个职位也变得越来越乏味,虽然不见得能马上跳槽,但是也学会了在“逆境”中坚强:学习产品的设计,和非开发人员的沟通,重构代码。觉得有句话说的真好:要想做自己想做的事,就得先做自己不想做的事。

与诸君共勉!