从未调用过完成处理程序

Question

我有这样的功能。

class func getAccountBalances(completionHandler:@escaping ((_ balances:Any) -> Void)){
//Alamofire request and we get the result. But sometimes the result fails.
    switch response.result {
    case .success(let value):
        completionHandler(value)
    case .failure(let error):
        print ("error is: \(error)")
}

如果失败，我不会使用代码来处理结果。那是一件坏事？如果调用失败，我是否需要一个完成处理程序，以便该函数不会留在内存中等待调用该完成处理程序？最佳做法是什么？

Answer 1

一般来说，对每个案例都调用完成是一种很好的做法。这样做的原因是您通常希望让上层（业务逻辑层）决定是否应该将某些余额（例如）标记为已保存，或者在发生错误时显示一个对话框。这是一个很好的实践，认为一切都应该是一个模块。话虽如此，如果另一个模块想要在某个时候调用相同的函数，那么让该模块处理结果可能是一个好主意。这可以通过多种方式实现，我不会进入这里，这是你的决定。

但是，这不是必须的。如果一个块不会被调用，它应该被释放，然后一切都是好的内存。因此，在您的示例中，如果您不将块保留在其他地方（例如将其保存在进行 getAccountBalances 调用的类中的变量中），您应该没问题。

另一个重要的部分是当你调用函数时要小心，不要在块内保留self 的地方造成内存泄漏：

getAccountBalances() { _ in 
   self.updateUI()
}

这个块将创建一个保留给自己，如果调用一切正常，但用户离开了屏幕，你最终可能会使用被释放的变量并使应用程序崩溃。这里的一个好的做法是不要在回调中保留 self，而是在此之前使其变弱：

getAccountBalances() { [weak self] _ in 
   // This is not necessarily needed, and you could use self?.updateUI() instead. 
   // However, this is usually another thing i like to do and consider it a good practice
   guard let `self` = self else { return }
   self.updateUI()
}

Answer 2

你的标题：Completion handler never called

您的代码：如果出现错误，则不会向完成处理程序返回任何值。

如果出现故障，您希望如何看到竞争处理程序的结果？你想让应用程序崩溃吗？这种方式更好，因为它可以处理两种情况：

class func getAccountBalances(completionHandler:@escaping ((_ balances:Any?) -> Void)){ //make Any an optional
//Alamofire request and we get the result. But sometimes the result fails.
    switch response.result {
    case .success(let value):
        completionHandler(value)
    case .failure(let error):
        print ("error is: \(error)")
        completionHandler(nil)
}

您对该函数的新调用：

getAccountBalances() { value in 
guard let _value = value else { // anticipate on the error return }
// use _value, the balances are inside.
}

另一种方法不是让它为零，而是向下转换它内部的值。看起来像这样：

class func getAccountBalances(completionHandler:@escaping ((_ balances:Any) -> Void)){
//Alamofire request and we get the result. But sometimes the result fails.
    switch response.result {
    case .success(let value):
        completionHandler(value)
    case .failure(let error):
        print ("error is: \(error)")
        completionHandler(error) //notice this change
}

那么你的函数应该是这样的：

getAccountBalances() { value in 
if let error = value as? Error { //or whatever type your error is in the function
  //there is an error
}
}