Связывание view model с view controller используя замыкания. Корректен ли данный подход в MVVM?

Question

[Tkas]

В моем приложении ячейка удаляется с collectionView после долгого нажатия на нее во вью контроллере. Код удаления написан во вью модели и выглядит вот так:

func deleteFood(forIndexPath indexPath: IndexPath, completion: @escaping ([FoodBySections], [FoodBySections]) -> ()) {
        let objectToDeleteName = self.foodBySections[indexPath.section][indexPath.row].name!
        
        let oldData = foodBySections
        CoreDataHelper.sharedInstance.deleteFoodFromFridge(foodName: objectToDeleteName)
        foods = CoreDataHelper.sharedInstance.fetchFoods()
        //newData
        foodBySections = FoodBySections.split(foods: foods!)
        
        completion(oldData, foodBySections)
    }

Код контроллера выглядит так:

@objc func deleteFood(gesture: UILongPressGestureRecognizer!) {
        if gesture.state != .ended {
            return
        }
        
        let point = gesture.location(in: self.collectionView)
        
        if let indexPath = self.collectionView?.indexPathForItem(at: point) {
            self.foodViewModel?.deleteFood(forIndexPath: indexPath, completion: { [weak self] (oldData, newData) in
                guard let self = self else { return }
                self.collectionView.animateItemAndSectionChanges(oldData: oldData, newData: newData)
            })
        }
    }

Таким образом, мы получили необходимые данные для обновления collectionView через замыкания. Является ли данный подход в MVVM корректным? Или в любом случае всегда нужно использовать реактивщину?

Comments

[andrew8712]

Я бы вообще советовал вам не использовать замыкания, а, как минимум - Promises. Как максимум - реакт

[Tkas]

andrew8712, спасибо за совет. А чем плохи замыкания в данном случае?

[andrew8712]

Плохая читаемость, вложенность контекстов. В будущем еще добавится проблема обработки ошибок - придется передавать optional параметр error

Answer 1

[Вячеслав Бельтюков]

У меня был опыт использования MVVM без реактивщины. Немного длиннее код, но в целом нормально. Большая часть реактивщины избыточна, если вам нужен только байндинг.

Что касается конкретно этого кода, то все таки в парадигме MVVM не надо связывать действие пользователя с результатом. То есть у вас есть отдельно метод deleteFood(IndexPath) и отдельно

var animatableChanges: ((OldData, NewData) -> Void)?

куда View может засунуть свой байндинг. Такой подход позволяет ViewModel обновлять вьюху тогда, когда она посчитает нужным, а с другой стороны держать код всех байндингов в одном месте (к примеру, в специальном методе, вызываемом из viewDidLoad).

Comments

[Tkas]

Спасибо за развернутый ответ! "куда View может засунуть свой байндинг." Можете описать как примерно это сделать? Для обновления коллекции я использую библиотеку Differ, оттуда и взялся метод self.collectionView.animateItemAndSectionChanges(oldData, newData), который явно вызывается при удалении ячейки коллекции.

[Вячеслав Бельтюков]

Если схематично, то как-то вот так:

class ViewModel {
  var animatableChanges: ((OldData, NewData) -> Void)?
  
  func deleteFood(_ ip: IndexPath) {
    //блаблабла бизнес логика, которая генерит новые данные
    animatableChanges?(oldData, newData)
  }
}

class View {
  func viewDidLoad() {
    viewModel. animatableChanges = { [weak self] oldData, newData in
      self?.collectionView.animateItemAndSectionChanges(oldData: oldData, newData: newData)
    }
  }
}

[Вячеслав Бельтюков]

а чтобы ваш код легче читался, рекомендую освоить выражение guard. Почитать можно много где, к примеру тут: https://oxozle.com/2016/01/10/nachalo-swift-guard-...

[Tkas]

Вячеслав Бельтюков, благодарю вас за решение! Все работает. Долго ломал голову над архитектурой