Почему view's перерисовывается в cell?

Question

[MarsFM]

Подскажите плз, почему мои три вьюхи перерисовываются(когда скроллю или поворачиваю экран), которые находятся в кастомном stackView, а данные не изменяются?
(изображения в низу)

ViewController:

let cells: [String] = ["StepCell", "GoalCell"]
var steps: [Step] = [] {
        didSet {
            tableView.reloadData()
        }
    }
    
    override func viewDidLoad() {
        super.viewDidLoad()

        spinner.startAnimating()
        
        Network.shared.loadDataUser { (response) in
            switch response {
            case .success(let steps):
                DispatchQueue.main.async {
                    self.steps = steps
                    self.spinner.stopAnimating()
                }
            case .refuse(let error):
                print(error)
            }
        }
    }

extension StepViewController: UITableViewDataSource {
    
    func numberOfSections(in tableView: UITableView) -> Int {
        return steps.count
    }
    
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return cells.count
    }
    
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        
        let cellIdentifier = cells[indexPath.row]
        let step = steps[indexPath.section]
        
        switch cellIdentifier {
        case "StepCell":
            let cell = tableView.dequeueReusableCell(withIdentifier: cellIdentifier, for: indexPath) as! StepTableViewCell
            cell.configure(step)
            return cell
        case "GoalCell":
            let cell = tableView.dequeueReusableCell(withIdentifier: cellIdentifier, for: indexPath) as! GoalTableViewCell
            return cell
        default:
            let cell = UITableViewCell()
            return cell
        }
    }

}

StepTableViewCell:

@IBOutlet weak var customStackView: CustomStackView!
    
    var width: Int = 0
    
    func configure(_ step: Step) {
        dateLabel.text = getStringDate(for: step)
        myStepLabel.text = "\(step.getAllStep())"
        
        walkLabel.text = step.walk.description
        aerobicLabel.text = step.aerobic.description
        runLabel.text = step.run.description
    
        let allStep = step.getAllStep()
        let walk = getProportion(for: step.walk, allStep: allStep)
        let aerobic = getProportion(for: step.aerobic, allStep: allStep)
        let run  = getProportion(for: step.run, allStep: allStep)
        
        customStackView.setValues(walk, aerobic, run)
    }
    
    private func getStringDate(for step: Step) -> String {
        let formatter = DateFormatter.ddMMyyyy
        return formatter.string(from: step.date)
    }
    
    private func getProportion(for value: Int, allStep: Int) -> Double {
        return 100.0 * Double(value) / Double(allStep)
    }

CusttomStackView:

required init(coder: NSCoder) {
        super.init(coder: coder)
        
        distribution = .fillProportionally
        axis = .horizontal
        spacing = 5.0
        
        addArrangedSubview(walk)
        addArrangedSubview(aerobic)
        addArrangedSubview(run)
    }
    
    func setValues(_ walk: Double, _ aerobic: Double, _ run: Double) {
        self.walk.proportion = walk
        self.aerobic.proportion = aerobic
        self.run.proportion = run
    }

CustomProgressBar:

class CustomProgressBar: UIView {
    
    var proportion = 1.0
    
    override var intrinsicContentSize: CGSize {
        return CGSize(width: proportion, height: 1.0)
    }

}

Answer 1

[doublench21]

class CustomProgressBar: UIView {
    
    var proportion = 1.0 { 
      didSet { invalidateIntrinsicContentSize() }
    }

    
    override var intrinsicContentSize: CGSize {
        return CGSize(width: proportion, height: 1.0)
    }

}

Comments

[MarsFM]

также осталось(

Может ячейки не правильно создаю?



Ещё использую let step = steps[indexPath.section], и пердаю в cell.configure(step), может тут что то не так?

[doublench21]

MarsFM, Выделите на скрине, что у вас не так. Может я чего не понял. https://habrastorage.org/webt/5c/3c/65/5c3c65e642a...

[MarsFM]

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



На втором рисунке, после скролла или поворота экрана, полоски поменялись в ячейках, а данные такие же остались, получается step в cell.configure(step) передаётся правильно, или нет?

[doublench21]

MarsFM, не знаю. Выведите данные в cell.configure(...)

Answer 2

[briahas]

Покажите что у вас в cell.configure(step)

(upd)
"Помучил" ваш код немного. Мдааа... очччень интересная ситуация вырисовалась. Часа два проваландался.
Оказывается, что даже если удалять прогрессбары из стэквьюхи, а затем добавлять - все равно не меняется их размер.

Но, именно такое решение я и нашел - удалять из стэквьюхи все. Просто, я добавлял в не правильном месте.

Короче. Ваш код хорош, просто добавьте

arrangedSubviews.forEach { $0.removeFromSuperview() }

первой строчкой в вашем методе setValues

(upd2)
Подумал тут утром, и понял, что лучше чтоб логика была в одном методе, а не разбросанная по нескольким, поэтому, я бы сделал так:
- убрал из CustomStackView override func layoutSubviews()
- переписал setValues следующим образом:

func setValues(_ walkValue: Double, _ aerobicValue: Double, _ runValue: Double) {
        
        arrangedSubviews.forEach { $0.removeFromSuperview() }
        
        setNeedsLayout()
        layoutIfNeeded()
        
        walk.proportion = walkValue
        aerobic.proportion = aerobicValue
        run.proportion = runValue

        addArrangedSubview(walk)
        addArrangedSubview(aerobic)
        addArrangedSubview(run)
    }

...у меня работает.

Comments

[MarsFM]

данные при переиспользование ячеек отображаются правильно(в lable's), а вот три полоски меняются, так и не понял почему

class StepTableViewCell: UITableViewCell {
    
    @IBOutlet weak var dateLabel: UILabel!
    @IBOutlet weak var myStepLabel: UILabel!
    @IBOutlet weak var goalStepLabel: UILabel!
    
    @IBOutlet weak var walkLabel: UILabel!
    @IBOutlet weak var aerobicLabel: UILabel!
    @IBOutlet weak var runLabel: UILabel!
    
    @IBOutlet weak var customStackView: CustomStackView!

    var width: Int = 0
    
    func configure(_ step: Step) {
        print(step.walk)
        dateLabel.text = getStringDate(for: step)
        myStepLabel.text = "\(step.getAllStep())"
        
        walkLabel.text = step.walk.description
        aerobicLabel.text = step.aerobic.description
        runLabel.text = step.run.description
        
        let allStep = step.getAllStep()
        let walk = getProportion(for: step.walk, allStep: allStep)
        let aerobic = getProportion(for: step.aerobic, allStep: allStep)
        let run  = getProportion(for: step.run, allStep: allStep)
        customStackView.setValues(walk, aerobic, run)
    }
    
    private func getStringDate(for step: Step) -> String {
        let formatter = DateFormatter.ddMMyyyy
        return formatter.string(from: step.date)
    }
    
    private func getProportion(for value: Int, allStep: Int) -> Double {
        return 100.0 * Double(value) / Double(allStep)
    }
    
}

[briahas]

Я так понимаю self.walk.proportion = walk и есть код который выставляет новую ширину полоски? Тогда надо, как и указал doublench21 , надо после установки новой пропорции перерисовывать полоску. В вашем коде этого нет.

Попробуйте перерисовывать таблицу после поворота экрана.

[MarsFM]

Да создаёт новую ширину. У меня не только при повороте экрана, но и при скролле вниз происходит перерисовка полосок(это зависит не от изменение ширины экрана, а от переиспользование ячеек).

"Что значит перерисовывать таблицу после поворота экрана?"
Я правильно понял, надо переопределить layoutSubview и там установить пропорции, но тогда как в сам layoutSubview передам step?)

в CustomProgressBar добавил invalidateIntrinsicContentSize(), но это не сработало, как сказал doublench21

class CustomProgressBar: UIView {
    
    var proportion = 1.0 {
        didSet {
            invalidateIntrinsicContentSize()
        }
    }
    
    override var intrinsicContentSize: CGSize {
        return CGSize(width: proportion, height: 1.0)
    }

}

[briahas]

А зачем вы работаете с intrinsicContentSize? Почему не с bounds? Вы пробовали и это не работает?

(upd) Перечитал доку. Да, понятно почему работаете с 'intrinsicContentSize'. Попробуйте вместо invalidateIntrinsicContentSize использовать sizeToFit(); layoutIfNeeded()

[MarsFM]

У меня было с bounds, там кстати такая же проблема была, тоже после скрола менялась ширина. Потом doublench21, показал этот пример с intrinsicContentSize, выглядет красивее и легче) Но проблема с переиспользованием ячеек для полосок не решилась.

Попробовал sizeToFit(); layoutIfNeeded(); тоже самое(

Ссылка на проект, если интересно) https://my-files.ru/so1o31