Android开发中在Activity/Fragment直接给ViewModel属性赋值是否是反模式

首先给出明确结论：在Activity/Fragment中直接通过viewmodel.myData = myData的方式修改ViewModel的成员变量，属于典型的反模式。

• 打破封装原则：ViewModel的核心职责是内部管理自身状态，对外暴露可直接修改的公共属性，等于把状态变更的控制权完全交给了View层。后续如果需要在数据赋值时新增校验、格式转换、关联更新其他状态等逻辑，你需要逐一修改所有调用直接赋值的View层代码，维护成本会随项目迭代指数级上升。
• 状态变更不可追溯：如果多个View组件共享同一个ViewModel实例，任意组件都能直接修改myData的值，出现状态异常时你很难快速定位是哪个场景触发了非法赋值，排查问题的难度会大幅提升。
• 违反单向数据流规范：Android官方推荐的应用架构普遍遵循单向数据流约定：状态只能从ViewModel向下流向View层，View层只能通过发送事件通知ViewModel触发状态更新。直接从View层修改ViewModel状态相当于打破了数据流的单向性，架构会随着功能迭代越来越混乱，后续重构成本极高。

你不需要特意定义setMyData2这类命名的方法，只需要遵循「私有可变、对外不可变、公开方法接收事件」的规范即可，示例代码如下：

class MyViewModel : ViewModel() {
    // 内部私有可变属性，仅允许ViewModel自身修改
    private val _myData = MutableStateFlow<MyData?>(null)
    // 对外暴露不可变属性，仅允许View层观察状态变化
    val myData: StateFlow<MyData?> = _myData.asStateFlow()

    // 公开的状态更新入口，所有赋值逻辑收敛在这里
    fun setMyData(newData: MyData) {
        // 可在此处统一添加校验、数据转换、关联其他逻辑
        if (newData.checkValid()) {
            _myData.value = newData
        }
    }
}

View层只需要调用viewmodel.setMyData(myData)即可触发状态更新，后续如果需要修改赋值逻辑，仅需要修改ViewModel内部的setMyData方法，不需要调整任何View层代码。

如果你开发的是临时Demo、极小体量不会长期迭代的项目，且myData没有任何附加的赋值逻辑，直接赋值不会带来明显问题。但只要项目需要长期维护迭代，遵守上述规范可以大幅降低后期的维护成本。

内容的提问来源于stack exchange，提问作者Charles