我使用JSON获取两种类型的信息,并且我正在添加"操作"使用addObserver到2个不同的操作队列类(forKeyPath:" operations" ...)。
在函数observeValue中我检查是否是operationQueue1.operations.isEmpty,然后我在UI中刷新我的信息。我在使用operationQueue2时使用if else做同样的事情,但是当有时启动2个操作时,应用程序崩溃并显示错误消息:*** Terminating app due to uncaught exception 'NSRangeException', reason: 'Cannot remove an observer <AppName.ViewController 0x102977800> for the key path "operations" from <AppName.OperationQueue1 0x1c4a233c0> because it is not registered as an observer
。&#39;
当只有1个操作开始时,我没有问题。有什么建议吗?
func getInfo1(){//runned in viewDidLoad
operationQueue1.addObserver(forKeyPath:"operations"...)
operationQueue1.dataTask(URL:"..."....){
DispatchQueue.main.async{
NotificationCenter.default.postNotification(NSNotification.Name(rawValue: "NewDataReceived1", userInfo:infoFromTheWebsite)
}
}
}
func NewDataReceived1(){
here I add the information to arrays to be loaded in tableView1
}
HERE IS THE CODE FOR 2ND INFO WHICH IS THE SAME
override func observeValue(forKeyPath keyPath: String?, ....){
if(object as? operationQueue1 == operationQueue1Class && keyPath == "operations" && context == context1){
if(operationQueue1.operations.isEmpty){
DispatchQueue.main.async{
operationQueue1..removeObserver(self, forKeyPath:"operations")
Timer.scheduled("refreshingTableInformation1")
}
}
}else if(operationQueue2....){
SAME AS OPERATION 1, BUT USING DIFFERENT FUNC TO REFRESH TABLE INFORMATION AND THE TABLES ARE DIFFERENT
}else{
super.observeValue(forKeyPath: keyPath, of: object, change: change, context: context)
}
}
func refreshingTableInformation1(){
tableView1.reloadData()
Timer.scheduled("getInfo1", repeat:false)
}
func refreshingTableInformation2(){
tableView2.reloadData()
Timer.scheduled("getInfo2", repeat:false)
}
有时它可以工作10秒并且崩溃,有时会工作超过60秒然后崩溃......
答案 0 :(得分:4)
此处的代码容易受到竞争条件的影响。请考虑以下情形:
getInfo1()
,将操作添加到operationQueue1
。
操作完成,这意味着您的KVO观察被调用。队列现在为空,因此您的观察计划会在主调度队列中删除您的观察者。
现在,在您提交到主队列的操作之前,能够运行,其他调用getInfo1()
,这会向operationQueue1
添加一个新操作,该操作在操作之前完成你在第2步排队的机会有机会运行(嘿,也许主队列忙于某事;这很容易发生,因为它是一个串行队列)。
当队列为空时,您对[{1}} 第一次调用的观察再次被称为 ,导致另一个取消注册要提交到主队列的块。
最终两个注销区块在主队列上执行。第二个程序崩溃了程序,因为你已经取消了观察者的注册。
您可以通过使用Swift 4的基于块的观察者来修复此问题(假设代码没有此类问题的更多问题),并将观察者设置为getInfo1()
而不是显式取消注册。但是,我建议KVO是你想要做的wrong tool。正如旧的“水晶探索”游戏的说明所说,这有点像使用高射炮杀死蚊子。
从上面的代码中我可以看到,看起来你正在使用KVO来安排一个通知,告知你提交给队列的操作或一组操作何时完成。取决于您的nil
方法实际执行的操作,以下是我要做的事情:
如果只提交一个操作:将操作的dataTask
属性设置为一个用于刷新表信息的闭包。
如果您提交了多个操作:创建一个新的completionBlock
来刷新您的表信息,并使用您提交到队列的其他每个操作调用该操作的BlockOperation
。然后,提交该操作。
这将为您提供更清洁,更无故障的方法来监控您的任务完成情况。而且由于您不再需要队列完全清空,您甚至可能不再需要使用两个单独的队列,这取决于您使用它们做了什么。