Отображение сообщения об ошибке сети пользователю в Swift

Вопрос в том, как я могу сделать этот код многоразовым, особенно проверку ошибок в сетевом методе и условие в обработчике завершения, чтобы у меня не было повторяющегося кода?

Я создал метод, который выполняет сетевой запрос с помощью URLSession и вызывает обработчик завершения с кодом состояния в качестве аргумента. При обработке завершения я создал условие, которое показывает сообщение об ошибке или выполняет переход на основе кода состояния. Весь этот код работает, но я хочу сделать его многоразовым, чтобы у меня не было повторяющегося кода.

Метод подключения к сети:

func saveMessage(data: String, day: String, completion: @escaping (Int)->()) {
    let url = URL(string: "\(Constants.baseURL)/daily_mindset/today_message")
    guard let requestUrl = url else { fatalError() }
    var request = URLRequest(url: requestUrl)
    request.httpMethod = "POST"
    
    // Set HTTP Request Header
    request.setValue("application/json", forHTTPHeaderField: "Accept")
    request.setValue("application/json", forHTTPHeaderField: "Content-Type")
    
    let jsonData = encodeJSON(with: data, day: day)
    request.httpBody = jsonData
    
    let task = URLSession.shared.dataTask(with: request) { (data, response, error) in
        if error != nil {
            completion(700)
            return
        }
        
        guard let response = response as? HTTPURLResponse else {
            completion(701)
            return
        }

        guard (200...299).contains(response.statusCode) else {
            completion(response.statusCode)
            return
        }
        
        guard let mime = response.mimeType, mime == "application/json" else {
            completion(702)
            return
        }
        
        guard let data = data else {
            completion(703)
            return
        }
        
        do {
            let todoItemModel = try JSONDecoder().decode(MessageData.self, from: data)
            Constants.currentMindsetId = todoItemModel._id!
            print("Response data:\n \(todoItemModel)")
        } catch let jsonErr{
            print(jsonErr)
        }
        completion(response.statusCode)
    }
    task.resume()
}

Вызов сетевого метода с обработчиком завершения:

messageManager.saveMessage(data: textView.text, day: day, completion: {(statusCode: Int) -> Void in
                    if (200...299).contains(statusCode) {
                        DispatchQueue.main.async {
                            self.performSegue(withIdentifier: "ToDailyMindsetScreen", sender: sender)
                        }
                    } else if (400...499).contains(statusCode) {
                        DispatchQueue.main.async {
                            self.errorLabel.text = "Please make sure you filled in the all the required fields."
                        }
                    } else if (500...599).contains(statusCode) {
                        DispatchQueue.main.async {
                            self.errorLabel.text = "Sorry, couldn't reach our server."
                        }
                    } else if (700...).contains(statusCode) {
                        DispatchQueue.main.async {
                            self.errorLabel.text = "Sorry, something went wrong. Try again later."
                        }
                    }
            })

Код в сетевом методе, который я хочу использовать повторно:

if error != nil {
            completion(700)
            return
        }
        
        guard let response = response as? HTTPURLResponse else {
            completion(701)
            return
        }

        guard (200...299).contains(response.statusCode) else {
            completion(response.statusCode)
            return
        }
        
        guard let mime = response.mimeType, mime == "application/json" else {
            completion(702)
            return
        }
        
        guard let data = data else {
            completion(703)
            return
        }

Код в обработчике завершения, который я хочу использовать повторно:

if (200...299).contains(statusCode) {
                        DispatchQueue.main.async {
                            self.performSegue(withIdentifier: "ToDailyMindsetScreen", sender: sender)
                        }
                    } else if (400...499).contains(statusCode) {
                        DispatchQueue.main.async {
                            self.errorLabel.text = "Please make sure you filled in the all the required fields."
                        }
                    } else if (500...599).contains(statusCode) {
                        DispatchQueue.main.async {
                            self.errorLabel.text = "Sorry, couldn't reach our server."
                        }
                    } else if (700...).contains(statusCode) {
                        DispatchQueue.main.async {
                            self.errorLabel.text = "Sorry, something went wrong. Try again later."
                        }
                    }

person Fabio    schedule 18.08.2020    source источник
comment
Возможно, вам следует опубликовать это на codereview.stackexchange.com вместо этого.   -  person Joakim Danielson    schedule 18.08.2020


Ответы (1)


Если сообщения об ошибках относятся к ViewController, вы можете начать с создания функции, которая возвращает сообщение на основе кода состояния, например:

private func getErrorMessageFor(statusCode: Int) -> String? {
    if (200...299).contains(statusCode) {
        //If no error message is returned assume that the request was a success
        return nil
    } else if (400...499).contains(statusCode) {
        return "Please make sure you filled in the all the required fields."
    } else if (500...599).contains(statusCode) {
        return "Sorry, couldn't reach our server."
    } else if (700...).contains(statusCode) {
        return "Sorry, something went wrong. Try again later."
    } else {
        return "Message for other errors?"
    }
}

Вы всегда можете переместить этот код в подкласс ViewController, чтобы предоставить более общие сообщения об ошибках, и переопределить его позже, чтобы предоставить более подробные ошибки для конкретного контроллера представления.

class BaseViewController: UIViewController {
    func getErrorMessageFor(statusCode: Int) -> String? {
        //base implementation here
    }
}

class OtherViewController: BaseViewController {
    override func getErrorMessageFor(statusCode: Int) -> String? {
        //create a new error message only for statusCode 404
        if statusCode == 404 {
            return "The requested resource was not found on the server. Please contact the support team"
        } else {
            return super.getErrorMessageFor(statusCode: statusCode)
        }
    } 
}

Имейте в виду, что по мере роста вашего приложения вы можете захотеть создать APIClient, который будет обрабатывать сеть и обработку ошибок за вас. Взгляните на https://bustoutsolutions.github.io/siesta/, это очень удобно дружелюбный

person mgapinski    schedule 18.08.2020