reloadData () вызывается, когда прокрутка влияет на внутренние данные - PullRequest
1 голос
/ 04 мая 2019

Похоже, что табличное представление в Swift динамически перезагружает данные на экран в зависимости от положения прокрутки.Однако в моем случае я не хочу, чтобы механизм работал таким образом, поскольку элементы в моем табличном представлении обновляют общую стоимость заказа пользователя и излишне портят общую стоимость.Когда я прокручиваю вверх или вниз, ячейки, в которых вызываются данные перезагрузки, воспроизводят математику и приводят к неверной итоговой цене.

Мое желание - пересчитать цену, используя функцию reloadData(), только когда нажатие UIStepper на определенной ячейке, а не на прокрутке также.Я сделал это с помощью функции IBAction, которая вызывает reloadData при нажатии на шаговый регулятор.Проблема с математическими и внутренними данными возникает, когда табличное представление прокручивается и неоднократно вызывает функцию reloadData при изменении видимых ячеек, что я не хочу, чтобы это делалось для какой-либо формы прокрутки.

func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let add_on = JSON(self.add_ons[indexPath.row])
        let cell = tableView.dequeueReusableCell(withIdentifier: "AddOnCell", for: indexPath) as! AddOnTableViewCell

        quantities[indexPath.row] = cell.quantity
        //cell.quantityLabel.text = String(quantities[indexPath.row])

        let price = Double(add_on["price"].int!)/100.0
        let quantity:Double = Double(quantityLabel.text!)!

        //when user tries to increase add-on quantity beyond ticket quantity
        if(Int(quantity) < cell.quantity){
            print("QUANTITY UPDATED")
            cell.quantity -= 1
            quantities[indexPath.row] = cell.quantity
        }
        self.addOnPrice = updateTotal(quantities: self.quantities)
        totalPriceLabel.text = (Double(event_subtotal * quantity) + addOnPrice).dollarRound()

        cell.addonLabel?.text = add_on["name"].string! + " (+$\(price.roundTo(places: 2))" + ")"

        return cell
}

func updateTotal(quantities: [Int]) -> Double{
    var result:Double = 0.0
    for i in 0..<quantities.count{
        let curr_addon = JSON(self.add_ons[i])
        let price = Double(curr_addon["price"].int!)/100.0
        result += Double(quantities[i]) * price
    }
    return result.dollarRoundDouble()
}

@IBAction func stepperClicked(_ sender: UIStepper) {
    self.addOnPrice = 0.0
    AddOnTableView.reloadData()
}

Ниже приведена модель данных ячейки:

class AddOnTableViewCell: UITableViewCell{

    @IBOutlet weak var addonLabel: UILabel!
    @IBOutlet weak var quantityLabel: UILabel!
    @IBOutlet weak var quantityStepper: UIStepper!
    var quantity : Int  = 0 {
        didSet{
            self.quantityLabel.text = String(quantity)
            self.quantityStepper.value = Double(quantity)
        }
    }

    //ref: https://stackoverflow.com/questions/42876739/swift-increment-label-with-stepper-in-tableview-cell
    @IBAction func quantityStep(_ sender: UIStepper) {
        self.quantity = Int(sender.value)
        self.quantityLabel.text = String(quantity)
    } 
}

Поведение: при прокрутке общая рассчитанная цена изменяется без взаимодействия со ступенями.На основе позиции прокрутки количество для 1 элемента «обменивается» значением количества с другим элементом.Первое количество товара, равное 1, по какой-то причине обновляет количество последнего товара.Невозможно отладить это поведение.

Пользовательский интерфейс для справки enter image description here

1 Ответ

2 голосов
/ 05 мая 2019

В UIKit ячейки используются повторно при прокрутке списка. Предположим, что значения ячеек просто приходят и уходят. Он не должен содержать долгосрочные значения. Расчет суммы не должен производиться методом cellForRow. Прежде всего, вам не хватает модели данных для хранения всех данных. Способ сделать это - создать структуру, которая представляет собой дополнение (имеет количество, цену и т. Д.). Когда json получен, вы анализируете все это и сохраняете его в массиве. Массив затем используется в качестве основного источника. cellForRow должен использовать данные там и передать структуру в AddOnTableViewCell, чтобы все было хорошо разделено. Когда пользователь нажимает шаг количества, он должен вызвать созданный вами класс делегата, который реализован в контроллере представления, который обновляет массив с обновленным количеством, а затем выполняет суммирование. Предположим, что значения ячеек просто приходят и уходят.

Итак, создайте новый класс:

protocol AddOnUpdated: class {
    func quantityUpdated(label: String, value: Int)
}

Затем добавьте члена в свою ячейку {

class AddOnTableViewCell: UITableViewCell{
  weak var delegate: AddOnUpdated?

  func quantityStep {
     .. same code
     delegste?.quantityUpdated(label, value)
  }
}

Тогда в контроллере представления не обновляйте итоги при создании ячейки. Просто реализуйте делегат и установите значение delegate при создании ячейки на self.

Итак, теперь делегат должен вызываться всякий раз, когда обновляется значение любой ячейки. В этот момент вам нужно обновить массив значений и обновить итоговое значение.

Надеюсь, все это имеет смысл.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...