为什么在向其添加多个对象后,我的数组大小始终等于零?

时间:2017-08-19 00:35:26

标签: ios swift multithreading asynchronous ios-multithreading

我已经创建了两个函数,它们使用HTTP GET请求调用两个不同的rest API。我做了一个类调用“ExchangeObject”,它将从API检索到的数据封装到一个对象中。在每次调用中,我将一个ExchangeObject类的对象追加到一个名为exchangesArray的数组中。

ExchangeObject类的一个属性称为“price”,它是一个int值。在viewDidLoad中,我调用另一个函数来计算数组中两个对象的平均价格。但是,数组总是空的!我不知道为什么。我认为这是因为我正在进行异步调用,但我不知道如何修复它。但是,我的tableview不是空的。它显示数组中对象的两个价格。

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
     }

 }

1 个答案:

答案 0 :(得分:3)

您误解了异步方法的工作原理。您的方法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以及用于创建输出ExchangeObject的名称之外,您的两种方法看起来几乎完全相同。创建通用函数会更有意义:

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()
  }
}