Android正确实现MVVM（三）：向ViewModel传递动态创建View是否合规

你当前将接收ImageView作为参数的方法迁移到ViewModel中的操作，严格不符合MVVM的设计规范，不建议这么做，原因如下：

• Android ViewModel的核心设计要求就是完全与视图层解耦，不得持有任何View、Context相关的引用，否则会直接引发内存泄漏风险：ViewModel的生命周期远长于Activity/Fragment及其中的View实例，当配置变更（如屏幕旋转）导致视图销毁重建时，ViewModel持有的旧View引用会无法被GC回收，直接造成内存泄漏。
• 你提供的代码中存在大量直接操作View属性的逻辑：包括调用cbAnswer.getTag()、cbAnswer.setTag()，以及切换CheckBox选中状态的方法，这些都属于纯视图层操作，不属于业务逻辑范畴，不应该放在ViewModel中处理。

优化方案 #

你只需要将数据逻辑和视图操作拆分即可，拆分原则如下：

1. ViewModel层仅处理纯数据运算逻辑：仅保留下标计算、选中状态判断的纯数据逻辑，输入参数和返回值都不涉及任何View对象，示例代码如下：

// ViewModel层的纯数据逻辑
public SparseBooleanArray calculateTestModeCheckBoxState(List<AnswerDTO> answers, int pageNumber, int cbSelectedId, String currentCbTag) {
    SparseBooleanArray checkBoxStateResult = new SparseBooleanArray();
    for (int j = 0; j < answers.size(); j++) {
        int cbIndex = answers.size() * pageNumber + j;
        if (cbIndex != cbSelectedId) {
            checkBoxStateResult.put(cbIndex, false);
        } else {
            String stateFlag = currentCbTag.split("@")[0];
            checkBoxStateResult.put(cbIndex, !stateFlag.equals("1"));
        }
    }
    return checkBoxStateResult;
}

2. 视图层（Activity/Fragment）负责接收ViewModel返回的状态结果，执行对应的视图操作：拿到每个CheckBox下标对应的选中状态后，找到对应的动态创建的ImageView实例，自行执行setTag、切换选中状态的操作即可。

这样拆分既符合MVVM的解耦要求，也保留了原有的逻辑，同时规避了内存泄漏风险。

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