MoyaでAPIへのアクセスを行たい

Question

Moyaでlivedoor天気予報のAPIにアクセスしてjsonを取得したいと考えています。
qiitaの記事やブログ記事を参考に実装してみたのですが、アクセスに失敗してしまいます。

コード

Moyaのenum宣言部分では東京と大阪の２ヶ所を設定しています。

import Moya

enum TenkiAPI {
    case tokyo
    case osaka
}
extension TenkiAPI: TargetType {

    
    var baseURL:URL {
        return URL(string:"http://weather.livedoor.com/forecast/webservice/json/v1")!
    }
    var path: String {
        switch self {
        case .tokyo: return "?city=130010"
        case .osaka: return "?city=270000"
        }
    }
    
    var method: Moya.Method {
            switch  self {
            case .tokyo, .osaka:
                return .get
            }
        }
    
    //テストの際に使うメソッド
    var sampleData: Data {
        return Data()
    }
    
    var task: Task {
        switch self {
        case .osaka, .tokyo:
            return .requestPlain
        }
    }
    
    var headers: [String : String]? {
        return nil
    }
    
}

viewDidLoadでAPIへのアクセスを試みたのですが失敗して.failureに飛んでしまいます。

    override func viewDidLoad() {
        super.viewDidLoad()
        let provider = MoyaProvider<TenkiAPI>()
        provider.request(.tokyo) { result in
            switch result {
            case let .success(moyaResponse):
                do {
                    try moyaResponse.filterSuccessfulStatusCodes()
                    let data = try moyaResponse.mapJSON()
                    // do something with the data
                }
                catch {
                    print("json parse失敗")
                }
            case let .failure(error):
                print("アクセスに失敗しました:(error)")
                break
            }
            
        }
    }

アクセスの方法が間違っているのでしょうか？

Comments

[hayabusabusash]

エラーをプリントしているみたいなので出力されたエラーを見せていただけないでしょうか？

Moyaのenum内のpathにクエリ文字列が入っているのでパラメータを使うようなAPIだと思いますが、
その場合parametersとparameterEncodingを使ったほうがいいと思います。

憶測で申し訳ありませんがpath内の?が別の文字に変換されてしまっているのが原因なんじゃないかなと思います。

[hodoru3sei]

エラー ログは以下の様なものが出力されています
```
2019-06-26 17:44:38.293770+0900 MoyaTest[5093:446654] App Transport Security has blocked a cleartext HTTP (http://) resource load since it is insecure. Temporary exceptions can be configured via your app's Info.plist file.
2019-06-26 17:44:38.293818+0900 MoyaTest[5093:446654] Cannot start load of Task <0DB862E5-A5AB-4186-8213-CCC7AE52F05C>.<1> since it does not conform to ATS policy
2019-06-26 17:44:38.293901+0900 MoyaTest[5093:446652] Task <0DB862E5-A5AB-4186-8213-CCC7AE52F05C>.<1> finished with error - code: -1022
アクセスに失敗しました:underlying(Error Domain=NSURLErrorDomain Code=-1022 "The resource could not be loaded because the App Transport Security policy requires the use of a secure connection." UserInfo={NSUnderlyingError=0x1d4649ba0 {Error Domain=kCFErrorDomainCFNetwork Code=-1022 "(null)"}, NSErrorFailingURLStringKey=http://weather.livedoor.com/forecast/webservice/json/v1/%3Fcity=130010, NSErrorFailingURLKey=http://weather.livedoor.com/forecast/webservice/json/v1/%3Fcity=130010, NSLocalizedDescription=The resource could not be loaded because the App Transport Security policy requires the use of a secure connection.}, nil
```

?はそのままだと変換されてしまうのでしょうか？API通信をするのがほぼ初めてであまり理解していませんでした。
サンプルに以下の様な記載があったので東京と大阪それぞれの番号の前に?city=をつけていました
http://weather.livedoor.com/forecast/webservice/json/v1?city=400040

Answer 1

エラーの記載ありがとうございます！

App Transport Security has blocked a cleartext HTTP (http://) resource load since it is insecure. Temporary exceptions can be configured via your app's Info.plist file.

まずAPIのベースのURLがhttpなので、
http通信を許可するようにinfo.plistに編集が必要です。(こちらを参考にしてください)

NSErrorFailingURLStringKey=http://weather.livedoor.com/forecast/webservice/json/v1/%3Fcity=130010,

パスに指定していた?が%3Fになっていたのも原因のようです。
なので以下のようにしてみました。

import Moya

enum LivedoorWeatherAPI {
    case tokyo
    case osaka
}

extension LivedoorWeatherAPI: TargetType {
    
    var baseURL: URL {
        guard let url = URL(string: "http://weather.livedoor.com/forecast/webservice/json/v1") else {
            fatalError("base URL error")
        }
        return url
    }
    
    var path: String {
        switch self {
        case .tokyo:
            return ""
        case .osaka:
            return ""
        }
    }
    
    var method: Method {
        switch self {
        case .tokyo, .osaka:
            return .get
        }
    }
    
    var sampleData: Data {
        return Data()
    }
    
    // 新しく追加 パラメータ名がキーになります
    var parameters: [String: Any] {
        switch self {
        case .tokyo:
            let parameter = [
                "city": "130010"
            ] as [String: Any]
            return parameter
        case .osaka:
            let parameter = [
                "city": "270000"
            ] as [String: Any]
            return parameter
        }
    }
    
    // 新しく追加 URLにクエリ文字列として組み込む 
    var parameterEncoding: ParameterEncoding {
        switch self {
        case .tokyo, .osaka:
            return Moya.URLEncoding.queryString
        }
    }
    
    // 新しく追加 上で定義した parameters と parameterEncoding を使うに指定
    var task: Task {
        switch self {
        case .tokyo, .osaka:
            return .requestParameters(parameters: parameters, encoding: parameterEncoding)
        }
    }
    
    var headers: [String : String]? {
        return ["Content-type": "application/json"]
    }
}

この状態で同じように通信してみるとエラーではなく、なんらかのレスポンスが返ってきたので、
一応通信は成功したんじゃないかな？と思います。

class ViewController: UIViewController {

    override func viewDidLoad() {
        super.viewDidLoad()
        
        let provider = MoyaProvider<LivedoorWeatherAPI>()
        provider.request(.tokyo) { result in
            switch result {
            case let .success(moyaResponse):
                do {
                    try moyaResponse.filterSuccessfulStatusCodes()
                    let data = try moyaResponse.mapJSON()
                    print(data)
                    // do something with the data
                }
                catch {
                    print("json parse失敗")
                }
            case let .failure(error):
                print("アクセスに失敗しました:(error)")
                break
            }
        }
    }


}

Comments

[hodoru3sei]

丁寧にありがとうございます！
すごく参考になりました！！！

[hayabusabusash]

よかったです！
せっかくなので今日の天気を表示するところまでサンプルを作ってみました。
雑な部分もありますが、よければご確認ください。
( https://github.com/hayabusabusa/TeratailMoyaTest )

追記:
URLが変だったので修正しました。

[hodoru3sei]

ありがとうございます！！