Swift-使用Alamofire进行多个GET请求时出现问题

时间:2019-04-17 18:53:33

标签: ios arrays swift api alamofire

我在使用Alamofire快速从GET请求加载数据以将项目加载到UITableView时遇到问题。

我有2种方法fetchAllBeerOrdersfetchAllCocktailOrders都可以正常工作并获取正确的项目。我遇到的问题是在viewWillAppear方法中,我在这两个方法中都调用了这两个提取方法并重新加载tableView。按照我拥有的顺序,只有fetchAllCocktailOrders方法中的项目正在加载到tableView中,并且我已经通过切换顺序并从fetchAllBeerOrders获取项目来进行验证。


class DrinkOrdersTableViewController: UITableViewController { 
    var orders: [Order] = [] 
    override func viewDidLoad() {
        super.viewDidLoad()
        navigationItem.title = "Current Orders"

    }

    override func viewWillAppear(_ animated: Bool) {

        fetchAllBeerOrders { orders in
            self.orders = orders!
            //print("Beer fetch: ", self.orders)
            self.tableView.reloadData()
        }

        fetchAllCocktailOrders { orders in
            self.orders = orders!
            //print("Cocktail fetch: ", self.orders)
            self.tableView.reloadData()
        }

    }

    private func fetchAllCocktailOrders(completion: @escaping([Order]?) -> Void) {
        Alamofire.request("http://127.0.0.1:4000/orders", method: .get)
            .validate()
            .responseJSON { response in
                guard response.result.isSuccess else { return completion(nil) }
                guard let rawInventory = response.result.value as? [[String: Any]?] else { return completion(nil) }
                let currentOrders = rawInventory.compactMap { ordersDict -> Order? in
                    guard let orderId = ordersDict!["id"] as? String,
                        let orderStatus = ordersDict!["status"] as? String,
                        var pizza = ordersDict!["cocktail"] as? [String: Any] else { return nil }
                    pizza["image"] = UIImage(named: pizza["image"] as! String)

                    return Order(
                        id: orderId,
                        pizza: Pizza(data: pizza),
                        status: OrderStatus(rawValue: orderStatus)!
                    )

                }
                completion(currentOrders)
        }

    }

    private func fetchAllBeerOrders(completion: @escaping([Order]?) -> Void) {
        Alamofire.request("http://127.0.0.1:4000/orders", method: .get)
            .validate()
            .responseJSON { response in
                guard response.result.isSuccess else { return completion(nil) }
                guard let rawInventory = response.result.value as? [[String: Any]?] else { return completion(nil) }
                let currentOrders = rawInventory.compactMap { ordersDict -> Order? in
                    guard let orderId = ordersDict!["id"] as? String,
                        let orderStatus = ordersDict!["status"] as? String,
                        var pizza = ordersDict!["pizza"] as? [String: Any] else { return nil }
                    pizza["image"] = UIImage(named: pizza["image"] as! String)

                    return Order(
                        id: orderId,
                        pizza: Pizza(data: pizza),
                        status: OrderStatus(rawValue: orderStatus)!
                    )

                }
                completion(currentOrders)
        }
    }


    override func numberOfSections(in tableView: UITableView) -> Int {
        return 1
    }
    override func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        print("Debugging ROWS", orders.count)
        return orders.count
    }
    override func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = tableView.dequeueReusableCell(withIdentifier: "order", for: indexPath)
        let order = orders[indexPath.row]        
        cell.textLabel?.text = order.pizza.name
        cell.imageView?.image = order.pizza.image
        cell.detailTextLabel?.text = "$\(order.pizza.amount) - \(order.status.rawValue)"

        return cell
    }

    override func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
        performSegue(withIdentifier: "orderSegue", sender: orders[indexPath.row] as Order)
    }
    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
        if segue.identifier == "orderSegue" {
            guard let vc = segue.destination as? OrderViewController else { return }
            vc.order = sender as? Order
        }
    }
}

我不确定是否需要将GET请求合并为单个请求,或者我的方法逻辑是否错误,但是我需要tableView从fetchAllBeerOrdersfetchAllCocktailOrders加载订单。请帮助

3 个答案:

答案 0 :(得分:1)

就像Levi Yonder所说的那样,您的第二个请求替换了order的结果确实是一个问题,但是他提供的答案并不是最佳的。

您必须记住,网络请求是异步的。这意味着fetchAllCocktailOrders可以在完成fetchAllBeerOrders请求之前完成。在那种情况下,会发生相同的问题:

  1. 同时请求开火
  2. fetchAllCocktailOrders首先完成,将数据追加到订单中
  3. fetchAllBeerOrders完成,并用此请求的结果替换当前的数据批次。

解决方案:

fetchAllBeerOrders { orders in
    self.orders.append(orders)
    self.tableView.reloadData()
}

fetchAllCocktailOrders { orders in
    self.orders.append(orders)
    self.tableView.reloadData()
}

答案 1 :(得分:0)

出现此问题的原因是,fetchAllCocktailOrders会替换orders最初获取的fetchAllBeerOrders中的所有内容,而不是附加到其后。

因此,viewWillAppear正在调用fetchAllBeerOrders,后者将填充orders,然后fetchAllCocktailOrders将替换orders中的所有内容,而不是附加到{{1 }}。

要解决此问题,请在调用orders时,在fetchAllCocktailOrders后面附加函数返回的内容,而不要替换:

orders

答案 2 :(得分:0)

更新

要实现从两个GET请求中加载数据的目标,我需要使用.append(contentsOf: orders!)方法而不是self.orders = orders!将数据附加到orders数组。在viewWillAppear方法中,对fetchCocktailOrders的第二次调用将始终覆盖第一次对fetchBeerOrders的调用中的内容。显然我是个敏捷的新秀!

        fetchAllBeerOrders { orders in
            //self.orders.append(orders)
            self.orders.append(contentsOf: orders!)
            self.tableView.reloadData()
        }

        fetchAllCocktailOrders { orders in
            //self.orders.append(orders)
            self.orders.append(contentsOf: orders!)
            self.tableView.reloadData()
        }

这是正确的代码。