【Swift5】AlamofireでOpenWeatherMapのAPIを叩く際に読み取った値をプロパティに入れる方法

Question

前提・実現したいこと

Swift5の環境でAlamofireを利用してOpenWeatherMapのAPIを叩いてクラスのプロパティに取ってきた値をセットしたいです。

• Alamofire: ver.5.0.0-rc.3'

発生している問題とソースコード

メインのViewControllerでWeatherインスタンスを作成してプロパティを呼び出しても更新されておらず、Weatherクラス内の初期化されている値が呼び出される。

import Foundation
import Alamofire

class Weather {
    
    var city: String = ""
    var coord: [String: Double] = [:]
    var weather: String = ""
    var description: String = ""
    var icon: String = ""
    var tempreture: [String: Double] = [:]
    
    
    init(city: String) {
        self.city = city
    }
    
　　 // APIを叩いてデータを取得
    func getWeatherData() {

        AF.request("https://api.openweathermap.org/data/2.5/weather?q=(self.city)&APPID=XXXXXXXXXXXXXX")
            .validate(statusCode: 200..<300)
            .validate(contentType: ["application/json"])
            .responseJSON {response in
                let data = response.value as! [String: Any]
                let weather = data["weather"] as! NSArray
                guard let weatherDetail = weather[0] as? [String: Any] else {
                    print("failed")
                    return
                }

                // ここで値をセットしているつもり
                self.coord = data["coord"] as! [String: Double]
                self.weather = weatherDetail["main"] as! String
                self.description = weatherDetail["description"] as! String
                self.icon = weatherDetail["icon"] as! String
                self.tempreture = data["main"] as! [String: Double]
                print(self.icon)　　　// 想定した値が帰ってくる
        }     
    }
}

import UIKit
import Alamofire

class WeatherViewController: UIViewController {

    @IBOutlet weak var cityNameLabel: UILabel!
    override func viewDidLoad() {
        super.viewDidLoad()
        let tokyo = Weather(city: "Tokyo")
        tokyo.getWeatherData()      // ここでプロパティが更新されている想定
        print(tokyo.icon)           // 空文字が帰ってくる
        print(tokyo.coord)          // [:]が帰ってくる
    }  
}

試したこと

APIを使うのが今回で初なので予想なのですが、GetWeather関数が終わるのを待ってからViewControllerで呼び出さないといけないのかと思いました。

ちなみに

都市名"London"でAPIを叩くと以下のように取得されます。

{
"coord": {
"lon": -0.13,
"lat": 51.51
},
"weather": [
{
"id": 300,
"main": "Drizzle",
"description": "light intensity drizzle",
"icon": "09d"
}
],
"base": "stations",
"main": {
"temp": 280.32,
"pressure": 1012,
"humidity": 81,
"temp_min": 279.15,
"temp_max": 281.15
},
"visibility": 10000,
"wind": {
"speed": 4.1,
"deg": 80
},
"clouds": {
"all": 90
},
"dt": 1485789600,
"sys": {
"type": 1,
"id": 5091,
"message": 0.0103,
"country": "GB",
"sunrise": 1485762037,
"sunset": 1485794875
},
"id": 2643743,
"name": "London",
"cod": 200
}

Comments

[nakasho_dev]

viewへの反映をせずにプロパティをprint文で出力しようとして何も出ないということでしょうか。非同期処理については理解されていますでしょうか。

Answer 1

クラスの作り方をCodableを用いて以下のように変えてみてはいかがでしょうか？

コード
class Weather {

let coord: Coord
let Coord:Codable{
   let lon: String
   let lat: String
}
let weather: [Weather] 
let Weather: Cobable {
   let id: String
   let main: String
   let description: String
   let icon: String
}

...（以下省略）
}