ios - 看到此代码中的任何风险/潜在崩溃吗？想改进一下

Question

在发布我的应用程序并安装 Crashlytics 后，我注意到我每天在特定代码区域遇到几次崩溃。我想看看社区是否可以帮助我辨别它，因为收集的数据不足以帮助确定正在发生的事情。

我在测试期间和 25 名 Beta 测试人员无法看到这一点，但现在我有几千名用户，这每天都是一个问题。

基本上，我们的想法是发送请求以获取一些内容(作为 JSON 数组)，然后将其拉入手机。

我采用了更通用的方法，所以希望我犯了一些其他人可以发现的错误？非常感谢!

这样发出一个请求，得到一个数组，然后保存。我知道我可以在哪里放置 try/catch，但除此之外我不确定它为什么会崩溃。也许我所在的服务器偶尔会超时或发生其他事情？Crashlytics 似乎暗示它在这条线上:

NSJSONSerialization.JSONObjectWithData(data!, options: .MutableContainers)

0x00000001001117c4 _TFZFC118MyProxy8GetAsyncFMS0_U_S_14Deserializable__FTGCS_10RequestQ__8callbackFGCS_11ResponseGSaQ0___T__T_U_FTGSQCSo6NSData_GSQCSo13NSURLResponse_GSQCSo7NSError__T_ (MyProxy.swift:141)

好的，这里是代理函数:

class func GetAsync<R, T: Deserializable>(request: Request<R>, callback: (Response<Array<T>>) -> ())
{
    var list = Array<T>()
    var response = Response<Array<T>>()
    let serverRequest = NSMutableURLRequest(URL: NSURL(string: API_URL + request.Url)!)
    serverRequest.HTTPMethod = "GET"

    let task = NSURLSession.sharedSession().dataTaskWithRequest(serverRequest,
        completionHandler: {
            data, r, error in

            if error != nil {
                response.Status = .ERROR
            } else {

                 let responseArray = try! NSJSONSerialization.JSONObjectWithData(data!, options: .MutableContainers) as! NSArray
                //Append each item from the array to a list
                for item: AnyObject in responseArray {
                    let row = T(dict: item as! NSDictionary)
                    list.append(row)
                }

                    response = Response<Array<T>>(status: ResponseCode.OK, value: list)
                }

            }

            callback(response)
    })


    task.resume()
}

对于我从服务器检索单个 JSON 项的情况，我编写了一个类似的(回调是针对单个对象，而不是数组:

class func GetAsync<R, T: Deserializable>(request: Request<R>, callback: (Response<T>) -> ())
    {
        var response = Response<T>()
        let serverRequest = NSMutableURLRequest(URL: NSURL(string: API_URL + request.Url)!)
        serverRequest.HTTPMethod = "GET"

        let task = NSURLSession.sharedSession().dataTaskWithRequest(serverRequest,
            completionHandler: {
                data, r, error in

                if error != nil {
                    response.Status = .ERROR
                } else {

                    var responseStatus: ResponseCode = .NO_CODE
                    let responseObject: AnyObject?
                    do {
                        responseObject = try NSJSONSerialization.JSONObjectWithData(data!, options: .MutableContainers)
                    } catch _ {
                        responseObject = nil
                    }

                    if (responseObject != nil) {
                        let responseDictionary = responseObject as! NSDictionary
                        let code = responseDictionary["code"] as! String?

                        if (code != nil) {
                            let c = ResponseCode(rawValue: code!)
                            if (c != nil) {
                                responseStatus = c!
                            }
                        }

                        response = Response<T>(status: responseStatus, value: T(dict: responseDictionary))
                    }
                }

            callback(response)
        })

        task.resume()
    }

看到任何不好的编码实践或我可以改进的地方吗？谢谢!

Answer 1

至少这一行:

let responseArray = try! NSJSONSerialization.JSONObjectWithData(data!, options: .MutableContainers) as! NSArray

应该改为:

do {
    if let data = data, let responseArray = try NSJSONSerialization.JSONObjectWithData(data, options: []) as? NSArray {
        // use responseArray here
    } else {
        // the data was nil or the JSON wasn't an array
    }
} catch {
    print(error)
}

避免因 nil 数据或错误的 JSON 而崩溃。