Swift - 使用两个不同的OperationQueue和KVO时应用程序崩溃

时间:2017-12-13 00:31:19

标签: swift crash queue key-value-observing operation

我使用JSON获取两种类型的信息,并且我正在添加&#34;操作&#34;使用addObserver到2个不同的操作队列类(forKeyPath:&#34; operations&#34; ...)。 在函数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秒然后崩溃......

1 个答案:

答案 0 :(得分:4)

此处的代码容易受到竞争条件的影响。请考虑以下情形:

    调用
  1. getInfo1(),将操作添加到operationQueue1

  2. 操作完成,这意味着您的KVO观察被调用。队列现在为空,因此您的观察计划会在主调度队列中删除您的观察者。

  3. 现在,在您提交到主队列的操作之前,能够运行,其他调用getInfo1(),这会向operationQueue1添加一个新操作,该操作在操作之前完成你在第2步排队的机会有机会运行(嘿,也许主队列忙于某事;这很容易发生,因为它是一个串行队列)。

  4. 当队列为空时,您对[{1}} 第一次调用的观察再次被称为 ,导致另一个取消注册要提交到主队列的块。

  5. 最终两个注销区块在主队列上执行。第二个程序崩溃了程序,因为你已经取消了观察者的注册。

  6. 您可以通过使用Swift 4的基于块的观察者来修复此问题(假设代码没有此类问题的更多问题),并将观察者设置为getInfo1()而不是显式取消注册。但是,我建议KVO是你想要做的wrong tool。正如旧的“水晶探索”游戏的说明所说,这有点像使用高射炮杀死蚊子。

    从上面的代码中我可以看到,看起来你正在使用KVO来安排一个通知,告知你提交给队列的操作或一组操作何时完成。取决于您的nil方法实际执行的操作,以下是我要做的事情:

    • 如果只提交一个操作:将操作的dataTask属性设置为一个用于刷新表信息的闭包。

    • 如果您提交了多个操作:创建一个新的completionBlock来刷新您的表信息,并使用您提交到队列的其他每个操作调用该操作的BlockOperation。然后,提交该操作。

    这将为您提供更清洁,更无故障的方法来监控您的任务完成情况。而且由于您不再需要队列完全清空,您甚至可能不再需要使用两个单独的队列,这取决于您使用它们做了什么。