tableView.reloadData()をMVCのModelから実行したいのですが

Question

前提・実現したいこと

現在以下の動画のようなアプリを作っていて、そのコードをMVCで書き直しています。
そこで動画の最後の方のtableViewのセルをスワイプで削除したときに、一番上の合計獲得標高の更新をすることができずにいます。

発生している問題・エラーメッセージ

Modelでセルの生成と表示はできています。
スワイプで削除する操作もModelで行っていて、削除後にtableViewの更新をし合計獲得標高の値を変更することができません。ModelからtableViewの更新はできないので、Controllerで更新をするように伝えるような方法はないでしょうか？

該当のソースコード

import UIKit
import RealmSwift

class RecordListViewController: UIViewController {

    let realm = try! Realm()
    
    private let dataSource = ElevationDataSource()
    
    var model: Results<RealmData>?

    @IBOutlet fileprivate weak var tableView: UITableView! {
        didSet {
            tableView.dataSource = dataSource
            tableView.delegate = dataSource

            tableView.register(UINib(nibName: "ElevationListTableViewCell", bundle: nil), forCellReuseIdentifier: "ElevationListTableViewCell")
            tableView.register(UINib(nibName: "ElevationSumTableViewCell", bundle: nil), forCellReuseIdentifier: "ElevationSumTableViewCell")
        }

    }


    override func viewDidLoad() {
        super.viewDidLoad()

        model = realm.objects(RealmData.self).sorted(byKeyPath: "id", ascending: true)
        dataSource.setupModel(model)
    }
}

import UIKit
import RealmSwift
import SwipeCellKit

class ElevationDataSource: NSObject {

    let realm = try! Realm()

    var model: Results<RealmData>?

    var sum: Double = 0.0

    func setupModel(_ model: Results<RealmData>?) {
        self.model = model
        sum = model?.sum(ofProperty: "elevation") ?? 0.0
    }
}

extension ElevationDataSource: UITableViewDataSource, SwipeTableViewCellDelegate {
    
    //スワイプした際に表示される文字や画像を設定。またタップした際の処理も設定
    func tableView(_ tableView: UITableView, editActionsForRowAt indexPath: IndexPath, for orientation: SwipeActionsOrientation) -> [SwipeAction]? {
        guard orientation == .right else { return nil }
        
        let deleteAction = SwipeAction(style: .destructive, title: "Delete") { action, indexPath in
            
            self.updateModel(indexPath)
        }
        
        deleteAction.image = UIImage(named: "delete-Icon")
        
        return [deleteAction]
    }
    
    func updateModel(_ indexPath: IndexPath) {
        if let elevationForDeletion = self.model?[indexPath.row] {
            do {
                try self.realm.write {
                    self.realm.delete(elevationForDeletion)
                }
                model = realm.objects(RealmData.self).sorted(byKeyPath: "id", ascending: true)
                setupModel(model)
//                self.loadView()
//                self.viewDidLoad()
//                このあたりでtableViewを更新する操作をしたいです
            } catch {
                print("Error deleting elevation (error)")
            }
        }
    }
    
    func tableView(_ tableView: UITableView, editActionsOptionsForRowAt indexPath: IndexPath, for orientation: SwipeActionsOrientation) -> SwipeOptions {
        var options = SwipeOptions()
        options.expansionStyle = .destructive
        return options
    }
    
    func numberOfSections(in tableView: UITableView) -> Int {
        2
    }
    
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        if section == 0 {
            return 1
        } else {
            return model?.count ?? 1
        }
    }
    
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        if indexPath.section == 0 {
            guard let cell = tableView.dequeueReusableCell(withIdentifier: "ElevationSumTableViewCell", for: indexPath) as? ElevationSumTableViewCell else {
                return UITableViewCell()
            }
            cell.bodyLabel.text = String(format: "%.2fm", sum)
            return cell
        } else {
            guard let cell = tableView.dequeueReusableCell(withIdentifier: "ElevationListTableViewCell", for: indexPath) as? ElevationListTableViewCell else {
                return UITableViewCell()
            }
            
            cell.delegate = self

            if let elevation = model?[indexPath.row] {
                cell.dateLabel.text = elevation.date
                cell.bodyLabel.text = String(format: "%.2fm", elevation.elevation)
            }
            return cell
        }
    } 
}

extension ElevationDataSource: UITableViewDelegate {
    func tableView(_ tableView: UITableView, heightForRowAt indexPath: IndexPath) -> CGFloat {

        return 100
    }
}

試したこと

ModelでControllerのインスタンスを生成しさえすれば、簡単だと思うのですが。それだとMVCで書いている意味がなくなってしまうような気がして他の方法を探しましたが、見つかりませんでした。

Answer 1

updateModel に tableView のインスタンスを渡してあげれば実現できるのではないでしょうか。
コンパイルせずに書いているので不都合があるかもしれませんが、考え方としては次の通りです。

たとえば、

    func tableView(_ tableView: UITableView, editActionsForRowAt indexPath: IndexPath, for orientation: SwipeActionsOrientation) -> [SwipeAction]? {
        guard orientation == .right else { return nil }

        let deleteAction = SwipeAction(style: .destructive, title: "Delete") { action, indexPath in

            // ここ
            self.updateModel(tableView, for: indexPath)
        }
        deleteAction.image = UIImage(named: "delete-Icon")

        return [deleteAction]
    }

という感じにして、updateModel を

    // 引数の個数とラベルを変更（ラベル名はお好みで）
    func updateModel(_ tableView, for indexPath: IndexPath) {
        if let elevationForDeletion = self.model?[indexPath.row] {
            do {
                try self.realm.write {
                    self.realm.delete(elevationForDeletion)
                }
                model = realm.objects(RealmData.self).sorted(byKeyPath: "id", ascending: true)
                setupModel(model)
//                self.loadView()
//                self.viewDidLoad()
//                このあたりでtableViewを更新する操作をしたいです
                // ここで更新
                self.tablewVoew.reloadData()
            } catch {
                print("Error deleting elevation (error)")
            }
        }
    }

こんな感じになるのではないでしょうか。

Comments

[Raleigh]

なるほど！tableViewが引数にあることに気づきませんでした！ありがとうございます。
ちなみにdelegateを使ってModelからViewControllerに更新を依頼する方法も思いついたのですが、どちらがMVC的な書き方として正しいのでしょうか？
頂いた回答のほうがコードは短くて済みそうですが

[TsukubaDepot]

多くの例でも @IBOutlet で tableView, delegate や DataSource のメソッドの第一引数も tableView としてあるため、結構ここに気づかないことが多いような気がします。

私自身、MVC には詳しくないのでなんとも言えませんが、どれが正しいということはないかと思います。
ただ、現状では、RecordListViewController が tableView の Outlet を持っていて、一方 ElevationDataSource が tableView を直接操作しているので、正直なところ一瞬わかりにくいようにも思えます。Model の中で tableView という View を操作していますから、一瞬 Model の操作なのか、 View の操作なのか混乱してしまいそうです（頑張って作っているところ、申し訳ないのですが...）。

delegate を作る方法も一瞬思いついたのですが、現状のモデルだとメリットがあまりないような感じもします。tableView の操作は Model 側で行っているのに、tableView の更新を Controller で行わせるメリットがあまり感じられないためです。

適切にコメントできなくて申し訳ないです。

[Raleigh]

改めて考えてみると、Modelがセルのテキストを操作しているのはおかしなことだと気づきました。
データを取得する部分はModelにまかせて、セルの操作やtableViewの操作はViewに任せるように改良します！助けてもらってばかりで、本当にありがとうございます。