ios - 如何优雅地处理使用带有闭包的 NSURLSessionTask 的方法中的错误？

Question

我经常有这样的代码，当我有一个处理与 REST API 通信的类时，里面有很多方法使用 NSURLSession 任务，闭包作为回调。我还提供了其中一些方法的回调闭包，这样我就可以一个接一个地链接不同的 API 调用。我经常从 View Controller 中调用这些方法，例如作为按下按钮后的 Action 或在 viewDidLoad 中调用。

让我们以一个示例类为例，该类使用一种发送 POST 请求以向 API 注册新用户的方法与虚构的 API 对话:

class ApiConnection {
    var session: NSURLSession!

    init() {
        let config = NSURLSessionConfiguration.defaultSessionConfiguration()
        config.URLCache = NSURLCache.sharedURLCache()
        self.session = NSURLSession(configuration: config)
    }


    func registerNewUser(user user: String, password: String, callback: (() -> Void)? ) {
        let data = ["email": user, "password": password]
        let url = NSURL(string: "https://backend.myapp.com/auth/register/")
        let request = NSMutableURLRequest(URL: url!)
        request.setValue("application/json; charset=utf-8", forHTTPHeaderField: "Content-Type")
        try! request.HTTPBody = NSJSONSerialization.dataWithJSONObject(data, options: NSJSONWritingOptions())

        let task = session.dataTaskWithRequest(request) {
            data, response, error in

            if (error != nil) {
                print("TODO do the right thing here for error: \(error!.localizedDescription)")
                return
            }

            if let httpResponse = response as? NSHTTPURLResponse {

                do {
                    let result = try NSJSONSerialization.JSONObjectWithData(data!, options: NSJSONReadingOptions()) as? NSDictionary

                    if httpResponse.statusCode == 201 {

                        print("registration went successfully")
                        callback?()

                    } else {

                        print("registration went wrong, not registered new user: \(result)")

                    }



                }
                catch {
                    print("TODO something went wrong decoding the JSON response")
                }
            }


        }
        task.resume()
    }

}

我现在可以，例如在 View Controller 中，用这样的东西调用这个方法:

let myAPI = ApiConnection()
myAPI.registerNewUser(user: "email@foo.bar", password: "testing") {
    // continue here, probably doing the next call
    print("user is registered")
}

到目前为止一切顺利，但这个例子完全缺乏适当的错误处理(上面代码中的“TODO”，以便显示警告对话框或其他任何东西。我想知道如何改进我的代码以优雅的方式做到这一点.

当然，我可以在我创建的完成闭包中返回一个错误对象，并在 View Controller 中执行一些 if error != nil 魔术，但这会很丑陋并且会变得连续调用很麻烦。

我认为新的 do-try-catch 习语可能是完美的:有一个只在成功案例中执行的完成闭包，除此之外，当出现任何错误时，即使有一个异常我可以在 APIConnection 类中编写格式良好的错误消息。但是经过一些阅读后我发现要抛出闭包并不容易，因为这个闭包可以随时调用，即使在方法本身已经执行之后，对我来说很有意义但我不知道如何很好地处理这种情况。

你们有经验的程序员如何处理这个问题？我的 APIConnection 类的整个方法和架构是否愚蠢，整个事情可以做得更优雅？ Plaese，欢迎任何评论。

Answer 1

最适合解决此类问题的是 promises 概念。您在代码中执行的操作很容易被视为管道操作:get_server_respose |解析_json |执行回调。在同步方法中，这将非常容易编写，并将提供干净/可读的代码。在异步世界中，事情变得复杂，您很容易迷失在 callback hell 中。 .

有了 promises，您可以以非常类似于管道同步版本的方式链接异步操作，并且您只能在链的末端有一个失败点。

下面是您的问题的 promise-d 版本:

let task =  session.dataTaskWithRequest(request)
task.promisify.then({response, data in        
    guard let httpResponse = response as? NSHTTPURLResponse else {
       return failedPromise(Error(...)}
    }
    guard response.statusCode == 201 else {
       return failedPromise(Error(...)}
    }
    return jsonParsePromise(data);
}).then({ parsedDict in
    print("Registration went successfully")
    callback?()
}).then(nil, { error in
    print("An error occured: \(error)")
})
task.resume()

更加简洁，像 json 解析这样的任务在一个单独的(可测试的)单元中进行，并且只有一个失败点。

附言您可以更进一步并 promise registerNewUser 函数本身:

func registerNewUser(user user: String, password: String) -> Promise {
    //... task setup
    return task.promisify.then({response, data in        
        guard let httpResponse = response as? NSHTTPURLResponse else {
           return failedPromise(Error(...)}
        }
        guard response.statusCode == 201 else {
           return failedPromise(Error(...)}
        }
        return jsonParsePromise(data);
    })
    task.resume()
}

，方便函数的客户端使用:

registerNewUser(usernameTextfield.text, passwordTextfied.text).then({user in
    print("Registration succeeded with user \(user)")
}, {error in
    print("Registration failed with error \(error)")
})