0

I'm trying to convert my AF request to Router structures for a cleaner project. I'm getting an error for:

Value of protocol type 'Any' cannot conform to 'Encodable'; only struct/enum/class types can conform to protocols.

Please help me to fix the code. THANK YOU!

My URL will have a placeholder for username and the password will be sent in body. The response will be Bool (success), username and bearer token.

Under is my AF request:

let username = usernameTextField.text
let password = passwordTextField.text
let loginParams = ["password":"\(password)"]

AF.request("https://example.com/users/\(username)/login", 
            method: .post, 
            parameters: loginParams, 
            encoder: JSONParameterEncoder.default, 
            headers: nil, interceptor: nil).response { response in

                switch response.result {
                case .success:
                    if let data = response.data {

                        do {
                            let userLogin = try JSONDecoder().decode(UsersLogin.self, from: data)

                            if userLogin.success == true {
                                defaults.set(username, forKey: "username")
                                defaults.set(password, forKey: "password")
                                defaults.set(userLogin.token, forKey: "token")
                                print("Successfully get token.")

                            } else {
                                //show alert
                                print("Failed to get token with incorrect login info.")
                            }
                        } catch {
                            print("Error: \(error)")
                        }
                    }

                case .failure(let error):
                    //show alert
                    print("Failed to get token.")
                    print(error.errorDescription as Any)
                }
            }

What I have so far for converting to AF Router structures:

import Foundation
import Alamofire

enum Router: URLRequestConvertible {
    case login(username: String, password: String)

    var method: HTTPMethod {
        switch self {
        case .login:
            return .post
        }
    }

    var path: String {
        switch self {
        case .login(let username):
            return "/users/\(username)/login"
        }
    }

    var parameters: Parameters? {
        switch self {
        case .login(let password):
            return ["password": password]

        }
    }

    // MARK: - URLRequestConvertible
    func asURLRequest() throws -> URLRequest {
        let url = try Constants.ProductionServer.baseURL.asURL()
        var request = URLRequest(url: url.appendingPathComponent(path))

        // HTTP Method
        request.httpMethod = method.rawValue

        // Common Headers
        request.setValue(ContentType.json.rawValue, forHTTPHeaderField: HTTPHeaderField.acceptType.rawValue)
        request.setValue(ContentType.json.rawValue, forHTTPHeaderField: HTTPHeaderField.contentType.rawValue)

        // Parameters
        switch self {
        case .login(let password):
            request = try JSONParameterEncoder().encode(parameters, into: request) //where I got the error
        }

        return request
    }
}

class APIClient {
        static func login(password: String, username: String, completion: @escaping (Result<UsersLogin, AFError>) -> Void) {
            AF.request(Router.login(username: username, password: password)).responseDecodable { (response: DataResponse<UsersLogin, AFError>) in
                completion(response.result)
            }
        }
    }

LoginViewController Class (where I replaced the AF.request code)

APIClient.login(password: password, username: username) { result in
                    switch result {
                    case .success(let user):
                        print(user)
                    case .failure(let error):
                        print(error.localizedDescription)
                    }

Codable UsersLogin model

struct UsersLogin: Codable {
    let success: Bool
    let username: String
    let token: String?

    enum CodingKeys: String, CodingKey {
        case success = "success"
        case username = "username"
        case token = "token"
    }
}

3 Answers 3

1

Took me a while but finally fixed it. I also clean up the code too.

enum Router: URLRequestConvertible {
    case login([String: String], String) 

    var baseURL: URL {
        return URL(string: "https://example.com")!
    }

    var method: HTTPMethod {
        switch self {
        case .login:
            return .post
        }
    }

    var path: String {
        switch self {
        case .login(_, let username):
            return "/users/\(username)/login"
        }
    }

    func asURLRequest() throws -> URLRequest {
        print(path)
        let urlString = baseURL.appendingPathComponent(path).absoluteString.removingPercentEncoding!
        let url = URL(string: urlString)
        var request = URLRequest(url: url!)
        request.method = method

        switch self {
        case let .login(parameters, _):
            request = try JSONParameterEncoder().encode(parameters, into: request)
        }

        return request
    }
}

Usage

let username = usernameTextField.text

AF.request(Router.login(["password": password], username)).responseDecodable(of: UsersLogin.self) { (response) in

   if let userLogin = response.value {
      switch userLogin.success {
      case true:
           print("Successfully get token.")
      case false:
           print("Failed to get token with incorrect login info.")
      }
   } else {
      print("Failed to get token.")
   }
}
1

I solved a similar problem in this way. I created a protocol Routable

enum EncodeMode {
    case encoding(parameterEncoding: ParameterEncoding, parameters: Parameters?)
    case encoder(parameterEncoder: ParameterEncoder, parameter: Encodable)
}

protocol Routeable: URLRequestConvertible {
    
    var baseURL: URL { get }
    var path: String { get }
    var method: HTTPMethod { get }
    var encodeMode: EncodeMode { get }
}

extension Routeable {
    
    // MARK: - URLRequestConvertible
    
    func asURLRequest() throws -> URLRequest {
        let url = baseURL.appendingPathComponent(path)
        var urlRequest: URLRequest
        
        switch encodeMode {
        case .encoding(let parameterEncoding, let parameters):
            urlRequest = try parameterEncoding.encode(URLRequest(url: url), with: parameters)
        case .encoder(let parameterEncoder, let parameter):
            urlRequest = URLRequest(url: url)
            urlRequest = try parameterEncoder.encode(AnyEncodable(parameter), into: urlRequest)
        }

        urlRequest.method = method
        return urlRequest
    }
}

And my routers look like this one

enum WifiInterfacesRouter: Routeable {
    
    case listActive(installationId: Int16?)
    case insert(interface: WifiInterface)
    
    var encodeMode: EncodeMode {
        
        switch self {
        case .listActive(let installationId):
            guard let installationId = installationId else {
                return .encoding(parameterEncoding: URLEncoding.default, parameters: nil)
            }
            return .encoding(parameterEncoding: URLEncoding.default, parameters: ["idInstallation": installationId])
        case .insert(let interface):
            return .encoder(parameterEncoder: JSONParameterEncoder.default, parameter: interface)
        }
        
    }
    
    var baseURL: URL {
        return URL(string: "www.example.com/wifiInterfaces")!
    }
    
    var method: HTTPMethod {
        switch self {
        case .listActive: return .get
        case .insert: return .post
        }
    }
    
    var path: String {
        switch self {
        case .listActive: return "listActive"
        case .insert: return "manage"
        }
    }
}

To solve the build error

Protocol 'Encodable' as a type cannot conform to the protocol itself

I used the useful AnyCodable library. A type erased implementation of Codable.

0

You can't use Parameters dictionaries with Encodable types, as a dictionary of [String: Encodable] is not Encodable, like the error says. I suggest moving that step of the asURLRequest process into a separate function, such as:

func encodeParameters(into request: inout URLRequest) {
    switch self {
    case let .login(parameters):
        request = try JSONParameterEncoder().encode(parameters, into: request)
    }
}

Unfortunately this doesn't scale that well for routers with many routes, so I usually break up my routes into small enums and move my parameters into separate types which are combined with the router to produce the URLRequest.

3
  • Hi @Jon Shier, Could you go into more details with your last sentence? I would like to use Alamofire router mechanism to send POST with Encodable parameter as body request and I can't figure how to do that in a generic and scalable way. Thanks
    – Giorgio
    Nov 11, 2021 at 12:07
  • I simply meant that any single router with a lot of routes will have a lot of individual cases handled in an encode method like this. It works, it's just a bit of a pain to implement since you need to explicitly capture every route's parameters separately.
    – Jon Shier
    Nov 11, 2021 at 20:49
  • Thanks @Jon Shier. I'd like to have your opinion about my posted answer.
    – Giorgio
    Nov 12, 2021 at 11:59

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service, privacy policy and cookie policy

Not the answer you're looking for? Browse other questions tagged or ask your own question.