iOS 中的集合视图

Question

我试图使用 swift 4 和 openWeather API 构建一个天气应用程序，但我卡在了这个函数中：

func fetchWeather(){
    let weatherGetter = GetWeather()
    self.cityWeather = [Temperature]()
            for city in cities {
                    weatherGetter.getWeather(city: city) { (temp) in
                        guard let temp = temp else {return}
                        print(temp)
                        self.cityWeather?.append(temp)
                }
            }
    print(self.cityWeather?.count)
    self.collectionView?.reloadData()
}

所以这个函数在控制台中给了我这个

可选(0) 温度（城市：“伦敦”，cityTemperature：“294.06”，tempIcon：“01d”） 温度（城市：“东京”，cityTemperature：“297.33”，tempIcon：“09n”） 温度（城市：“巴黎”，cityTemperature：“298.0”，tempIcon：“01d”） 温度（城市：“莫斯科”，cityTemperature：“291.41”，tempIcon：“01n”）

这对我来说真的很奇怪，因为 print(self.cityWeather?.count) 是在 for 循环之后使用的，但在控制台中 Optional(0) 是第一位的。

所以如果我用这个方法

override func collectionView(_ collectionView: UICollectionView, numberOfItemsInSection section: Int) -> Int {
    return cityWeather?.count ?? 0
}

我的集合视图单元格不会显示，因为它返回 0。

但是如果在 self.cityWeather?.append(temp) 之后使用 print(self.cityWeather) 我明白了！ （我删除了所有其他打印语句）：

可选([WhatsWeather.Temperature(city: "Moscow", cityTemperature: "291.41", tempIcon: "01n")]) 可选（[WhatsWeather.Temperature（城市：“莫斯科”，cityTemperature：“291.41”，tempIcon：“01n”），WhatsWeather.Temperature（城市：“伦敦”，cityTemperature：“294.05”，tempIcon：“01d”）]） 可选（[WhatsWeather.Temperature（城市：“莫斯科”，cityTemperature：“291.41”，tempIcon：“01n”），WhatsWeather.Temperature（城市：“伦敦”，cityTemperature：“294.05”，tempIcon：“01d”），WhatsWeather .Temperature（城市：“巴黎”，cityTemperature：“298.0”，tempIcon：“01d”）]） 可选（[WhatsWeather.Temperature（城市：“莫斯科”，cityTemperature：“291.41”，tempIcon：“01n”），WhatsWeather.Temperature（城市：“伦敦”，cityTemperature：“294.05”，tempIcon：“01d”），WhatsWeather .Temperature(city: "Paris", cityTemperature: "298.0", tempIcon: "01d"), WhatsWeather.Temperature(city: "Tokyo", cityTemperature: "297.38", tempIcon: "09n")])

Answer 1

只有在所有城市数据都存储在您的数据源中之后，您才应该重新加载collectionView，并且由于这是一个 API 调用，因此接收每个响应都需要一些时间。所以你应该做的是检查完成内部是否已收到所有数据，然后重新加载collectionView。

func fetchWeather(){
    let weatherGetter = GetWeather()
    self.cityWeather = [Temperature]()
    var networkCallsDone = 0
    for city in cities {
        weatherGetter.getWeather(city: city) { (temp) in
            networkCallsDone += 1
            guard let temp = temp else {
                if networkCallsDone == cities.count {
                    DispatchQueue.main.async {
                        collectionView.reloadData()    //In case the last network call to finish does not return valid data
                    }        
                }
                return
            }
            print(temp)
            self.cityWeather?.append(temp)
            if networkCallsDone == cities.count {
                 DispatchQueue.main.async {
                     collectionView.reloadData()
                 }
            }
        }
    }
}

编辑：感谢 matt 在检查 count 和线程问题时指出逻辑错误。

Answer 2

那是因为您在weatherGetter.getWeather 块中附加了cityWeather，这是异步操作并同步重新加载collectionView（在数据从服务器返回之前）。

试试这个：

func fetchWeather(){
    let weatherGetter = GetWeather()
    self.cityWeather = [Temperature]()
    for city in cities {
        weatherGetter.getWeather(city: city) { (temp) in
            guard let temp = temp else {return}
            self.cityWeather?.append(temp)
            self.collectionView?.reloadData()
        }
    }
}