Почему размер моего массива всегда равен нулю после добавления к нему нескольких объектов?

Question

Я сделал две функции, которые вызывают два разных API-интерфейса остальных с HTTP-запросом GET. Я сделал вызов класса "ExchangeObject", который инкапсулирует данные, полученные из API, в объект. При каждом вызове я добавляю объект класса ExchangeObject в массив с именем exchangesArray.

Одно свойство класса ExchangeObject называется "цена", и это значение типа int. В viewDidLoad я вызываю другую функцию, которая вычисляет среднее значение цен двух объектов в массиве. Тем не менее, массив всегда пуст! Я не знаю почему. Я думаю, это потому, что я делаю асинхронные вызовы, но я не знаю, как это исправить. Тем не менее, мой стол не пуст. Он отображает две цены объектов в массиве.

class ViewController: UIViewController, UITableViewDataSource{


     @IBOutlet weak var exchangesTableView: UITableView!

     var exchangesArray = [ExchangeObject]()


     override func viewDidLoad() {
         super.viewDidLoad()
         exchangesTableView.dataSource = self

         gemini()
         bitfinex()
         average()

     }


     func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int{

         return exchangesArray.count

     }



     func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell{

         let cell = exchangesTableView.dequeueReusableCell(withIdentifier: "cell")
         cell?.textLabel?.text = exchangesArray[indexPath.row].name
         cell?.detailTextLabel?.text = exchangesArray[indexPath.row].price

         return cell!

     }


     func gemini(){
         let url = "https://api.gemini.com/v1/pubticker/ethusd"
         var request = URLRequest(url: URL(string: url)!)
         request.httpMethod = "GET"

         let configuration = URLSessionConfiguration.default
         let session = URLSession(configuration: configuration, delegate: nil, delegateQueue: OperationQueue.main)

         let task = session.dataTask(with: request) { (data, response, error) in

             if(error != nil){
                 print("Error")
             }
             else{
                 do{
                     let fetchedData = try JSONSerialization.jsonObject(with: data!, options: .allowFragments) as AnyObject
                     let price = fetchedData["last"] as! String
                     self.exchangesArray.append(ExchangeObject(name: "Gemini", price: price))
                     DispatchQueue.main.async {
                         self.exchangesTableView.reloadData()
                     }

                 }
                 catch{
                     print("Error")
                 }
             }
         }
         task.resume()
     }


     func bitfinex(){
         let url = "https://api.bitfinex.com/v2/ticker/tETHUSD"
         var request = URLRequest(url: URL(string: url)!)
         request.httpMethod = "GET"

         let configuration = URLSessionConfiguration.default
         let session = URLSession(configuration: configuration, delegate: nil, delegateQueue: OperationQueue.main)

         let task = session.dataTask(with: request) { (data, response, error) in

             if(error != nil){
                 print("Error")
             }
             else{
                 do{
                     let fetchedData = try JSONSerialization.jsonObject(with: data!, options: .allowFragments) as! NSArray
                     let priceInt = fetchedData[6]
                     let price = String(describing: priceInt)
                     self.exchangesArray.append(ExchangeObject(name: "Bitfinex", price: price))
                     DispatchQueue.main.async {
                         self.exchangesTableView.reloadData()
                     }
                 }
                 catch{
                     print("Error")
                 }
             }
         }
         task.resume()
     }




     func average() -> Void{

         var total: Int = 0
         var average: Int

         for eth in self.exchangesArray{

             total = total + Int(eth.price)!

         }

         average = total/self.exchangesArray.count
         print("The average is: \(average)")
      }



 }


 class ExchangeObject{

     var name: String
     var price: String

     init(name: String, price: String){
         self.name = name
         self.price = price
     }

 }

Answer 1

Вы неправильно понимаете, как работают асинхронные методы. Ваши методы gemini() а также bitfinex() оба асинхронны. Они оба запускают задачи с данными в фоновом потоке и возвращаются немедленно, прежде чем задачи с данными даже начали выполняться. Через некоторое время задача с данными завершается, и вызывается метод завершения задачи с данными. К тому времени, однако, ваш average() метод уже был вызван.

Вы должны переписать оба этих метода, чтобы получить обработчики завершения:

func bitfinex(completion: () -> void) {
    let url = "https://api.bitfinex.com/v2/ticker/tETHUSD"
    var request = URLRequest(url: URL(string: url)!)
    request.httpMethod = "GET"

    let configuration = URLSessionConfiguration.default
    let session = URLSession(configuration: configuration, delegate: nil, delegateQueue: OperationQueue.main)

    let task = session.dataTask(with: request) { (data, response, error) in
        if(error != nil) {
             print("Error")
        }
        else {
             do {
                 let fetchedData = try JSONSerialization.jsonObject(with: data!, options: .allowFragments) as! NSArray
                 let priceInt = fetchedData[6]
                 let price = String(describing: priceInt)
                 self.exchangesArray.append(ExchangeObject(name: "Bitfinex", price: price))
                 DispatchQueue.main.async {
                     completion()
                 }
             }
             catch {
                 print("Error")
             }
        }
    }
    task.resume()
}

Рефакторинг gemini() метод таким же образом, а затем вызвать их так:

gemini() {
  self.bitfinex() {
     self.tableView.reloadData()
     self.average()
  }
}

Что вы сделали, чтобы настроить gemini() а также bitfinex() методы, так что каждый из них принимает блок кода в качестве параметра. Они делают свое дело, а затем запускают этот код после завершения асинхронной работы.

Блок завершения для gemini() вызовы функций bitfinex()и блок завершения для bitfinex() перезагружает табличное представление и затем вызывает ваш средний метод.

Я пропустил некоторые детали, например, что делать, если задача данных возвращает ошибку, но этого должно быть достаточно, чтобы начать работу.

Также обратите внимание, что похоже, что ваши два метода в значительной степени идентичны, кроме URL-адресов, на которые они указывают, и имен, которые они используют для создания выходных данных. ExchangeObjects. Было бы разумнее создать функцию общего назначения:

func getStock(symbol: String, remoteURL: String, completion: () -> void)

И затем вызовите это дважды с различными параметрами:

getStock(symbol: "gemini", 
    remoteURL: "https://api.gemini.com/v1/pubticker/ethusd") {
  self.getStock(symbol: "Bitfinex", remoteURL: "https://api.bitfinex.com/v2/ticker/tETHUSD") {
    self.tableView.reloadData()
    self.average()
  }
}