在 MVVM 中设置委托的正确方法

Question

我想知道在 Swift 的 MVVM 模式中的 ViewModel 中设置委托的正确方法是什么。

我正在从另一个类实例化 ViewController：

let viewModel = DashboardViewModel()
let viewController = DashboardViewController(viewModel: viewModel)

我的视图模型：

protocol DashboardViewModelType {
    var items: [Item] { get }
    var reloadDelegate: DashboardDataReloadDelegate? { get set }
}

protocol DashboardDataReloadDelegate: class {
    func reloadData()
}

class DashboardViewModel: DashboardViewModelType {

    var items: [Item] = []

    weak var reloadDelegate: DashboardDataReloadDelegate?

    init() {
        loadItems()
    }

    func loadItems() {
        let databaseFetcher = DatabaseDaysFetcher()
        databaseFetcher.getDays(onData: { (items) in
            self.items = items
            reloadDelegate?.reloadData() //delegate is nil here
        }) { (error) in
            print(error)
        }
    }
}

和视图控制器：

class DashboardViewController: UIViewController {

    var viewModel: DashboardViewModelType?

    init(viewModel: DashboardViewModelType) {
        self.viewModel = viewModel
        super.init(nibName: nil, bundle: nil)
        self.viewModel!.reloadDelegate = self // it is executed after 
    }

    required init?(coder aDecoder: NSCoder) {
        fatalError("init(coder:) has not been implemented")
    }

    override func viewDidLoad() {
        super.viewDidLoad() 
    }

}

extension DashboardViewController: DashboardDataReloadDelegate {
    func reloadData() {
        print("data reloaded")
    }
}

所以主要问题是，如果我想在另一个类中注入 viewModel，我会在委托尚未设置时实例化 viewModel。在DashboardViewModelType 协议中声明loadItems，然后从ViewController 中的init 或viewDidLoad 调用这个函数会更好吗？

Answer 1

是的，您可以在init 中为DashboardViewModel 注入DatabaseDaysFetcher，然后如您所说，将loadItems 移动到DashboardViewModelType 协议中。

然后当你调用loadItems 时，它应该回调给调用者。 然后在loadItems回调中使用[weak self]。

这样就不需要delegate。

protocol DashboardViewModelType {
    init(databaseFetcher: DatabaseDaysFetcher)
    func loadItems(completion: ([Item]) -> Void, error: (Error) -> Void)
}

final class DashboardViewModel: DashboardViewModelType {

    private var databaseFetcher: DatabaseDaysFetcher

    init(databaseFetcher: DatabaseDaysFetcher) {
        self.databaseFetcher = databaseFetcher
    }

    func loadItems(completion: ([Item]) -> Void, onError: (Error) -> Void) {
        self.databaseFetcher.getDays(onData: { (items) in
            completion(items)
        }) { (error) in
            onError(error)
        }
    }
}