Question

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

让我们看一个示例类，它使用一个方法与假想的API进行对话，该方法发送POST请求以使用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()
    }

}

我现在可以，例如在视图控制器中，用这样的方法调用此方法：

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

到目前为止一直很好，但这个例子完全没有正确的错误处理（上面代码中的“TODO”，为了显示警告标签或者其他什么。我想知道如何改进我的代码以优雅的方式做到这一点

当然，我可以在我创建的完成闭包中给出一个错误对象，并在视图控制器中执行一些if error != nil魔术，但这对于连续调用来说会很难看并且变得很麻烦。

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

你是如何经验丰富的程序员处理这个的？我的整个方法和我的APIConnection类的架构是愚蠢的，整个事情可以做得更优雅吗？ Plaese，任何评论都表示赞赏。

Answer 1

最适合此类问题的是promises概念。您在代码中执行的操作可以轻松地视为管道操作：get_server_respose | parse_json | execute_callback。在同步方法中，这将非常简单，并且将提供干净/可读的代码。在异步世界中，事情变得复杂，你很容易迷失在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在单独的（可测试的）单元中解析，只有一个故障点。

P.S。您可以更进一步宣传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)")
})

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

1 个答案: