Использование Generics / Codable с API-ответом 204 НЕТ КОНТЕНТА - PullRequest
Новая
       95

Использование Generics / Codable с API-ответом 204 НЕТ КОНТЕНТА

0 голосов
/ 06 января 2019

Я использую дженерики и кодируем с URLSession.

Когда я получаю ответ от API, я проверяю статус в диапазоне 200 - 299 и декодирую данные следующим образом 

        guard let data = data, let value = try? JSONDecoder().decode(T.self, from: data) else {
            return completion(.error("Could not decode JSON response"))
        }
        completion(.success(value))

Затем он передается обработчику завершения, и все в порядке.

У меня есть новая конечная точка, но я должен также выполнить POST, но эта конечная точка возвращает 204 без тела содержимого.

Как таковой, я не могу декодировать ответ, просто как не могу передать тип?

Мой обработчик завершения ожидает 

enum Either<T> {
    case success(T)
    case error(String?)
}

и включение моего ответа statusCode, например, так: 

   case 204:
        let value = String(stringLiteral: "no content")
        return completion(.success(value))

выдает ошибку

Член 'success' в 'Either <<em>>' производит результат типа 'Either', но контекст ожидает, что 'либо <</em>>'

Мой APIClient 

protocol APIClientProtocol: class {
    var task: URLSessionDataTask { get set }
    var session: SessionProtocol { get }
    func call<T: Codable>(with request: URLRequest, completion: @escaping (Either<T>) -> Void) -> Void
    func requestRefreshToken<T: Codable>(forRequest failedRequest: URLRequest, completion: @escaping (Either<T>) -> Void) -> Void
}

extension APIClientProtocol {
    func call<T: Codable>(with request: URLRequest, completion: @escaping (Either<T>) -> Void) -> Void {
        task = session.dataTask(with: request, completionHandler: { [weak self] data, response, error in
            guard error == nil else {
                return completion(.error("An unknown error occured with the remote service"))
            }
            guard let response = response as? HTTPURLResponse else {
                return completion(.error("Invalid HTTPURLResponse recieved"))
            }

            switch response.statusCode {
            case 100...299:
                guard let data = data else { return completion(.error("No data in response")) }
                guard let value = try? JSONDecoder().decode(T.self, from: data) else { return completion(.error("Could not decode the JSON response")) }
                completion(.success(value))
            case 300...399: return completion(.error(HTTPResponseStatus.redirection.rawValue))
            case 400: return completion(.error(HTTPResponseStatus.clientError.rawValue))
            case 401: self?.requestRefreshToken(forRequest: request, completion: completion)
            case 402...499: return completion(.error(HTTPResponseStatus.clientError.rawValue))
            case 500...599: return completion(.error(HTTPResponseStatus.serverError.rawValue))
            default:
                return completion(.error("Request failed with a status code of \(response.statusCode)"))
            }
        })
        task.resume()
    }
}

1 Ответ

0 голосов
/ 06 января 2019

Сделайте ваш Either тип успеха перечисления необязательным 

enum Either<T> {
    case success(T?)
    case error(String)
}

Создать случай для 204 статуса ответа, передавая ноль 

    case 204:
        completion(.success(nil))

Затем вы можете использовать typealias и создать что-то общее, например 

typealias NoContentResponse = Either<Bool>

После этого вы сможете включить NoContentResponse.

...